join knn filter terms with AND (#716)

wellcomecollection · Sep 15, 2023 · d7a705c · d7a705c
1 parent 513246a
commit d7a705c
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 2 deletions.
diff --git a/search/src/main/scala/weco/api/search/services/TemplateSearchBuilder.scala b/search/src/main/scala/weco/api/search/services/TemplateSearchBuilder.scala
@@ -43,8 +43,9 @@ trait TemplateSearchBuilder extends Encoders {
   // on whether the document contains the term in any searchable field.
   // If there are fields that should be ignored, then this default will
   // be inappropriate, and can be overridden.
-  protected val knnFilter: String = """{
-  |   "multi_match": { "query": "{{query}}" }
+  protected val knnFilter: String =
+    """{
+  |   "multi_match": { "query": "{{query}}", "operator": "AND", "type": "cross_fields"}
   | }""".stripMargin
 
   // Importantly, this is *not* JSON, due to the `{{#` sequences, so must be created and sent as a string.

diff --git a/search/src/test/scala/weco/api/search/images/ImagesFiltersTest.scala b/search/src/test/scala/weco/api/search/images/ImagesFiltersTest.scala
@@ -354,6 +354,7 @@ class ImagesFiltersTest extends AnyFunSpec with ApiImagesTestBase {
             imagesIndex,
             "images.examples.color-filter-tests.even-less-blue-foot",
             "images.examples.color-filter-tests.blue-foot",
+            "images.examples.color-filter-tests.blue",
             "images.examples.color-filter-tests.orange-foot",
             "images.examples.color-filter-tests.slightly-less-blue-foot"
           )
@@ -374,5 +375,28 @@ class ImagesFiltersTest extends AnyFunSpec with ApiImagesTestBase {
           }
       }
     }
+    it("combines multi-token query terms with AND") {
+      withImagesApi {
+        case (imagesIndex, routes) =>
+          indexTestDocuments(
+            imagesIndex,
+            "images.examples.color-filter-tests.blue", // Green + Dye (subjects)
+            "images.examples.color-filter-tests.blue-foot", // Dye but not Green
+            "images.examples.color-filter-tests.even-less-blue-foot" // Neither Dye nor Green
+          )
+
+          assertJsonResponse(
+            routes,
+            path = f"$rootPath/images?query=green+dye&color=22bbff"
+          ) {
+            Status.OK -> imagesListResponse(
+              ids = Seq(
+                "images.examples.color-filter-tests.blue"
+              ),
+              strictOrdering = true
+            )
+          }
+      }
+    }
   }
 }