AO3-5264: Set operator for bookmark search #3195

elzj · Dec 2, 2017

Issue

Fixes bookmark search to use 'AND' as the default operator, to match other search behavior.

Bookmark searches for multiple words/terms should return results that match all of them, not any one of them.

houndci-bot · Dec 2, 2017

Redundant curly braces around a hash parameter.
Space inside } missing.

redsummernight · Dec 2, 2017

Looks good to me!

zz9pzza · Dec 2, 2017

elzj added Awaiting review Priority: Broken on Test (High) labels Dec 2, 2017

redsummernight added Reviewed: Ready to Merge and removed Awaiting review labels Dec 2, 2017

@@ -5,7 +5,7 @@
            it "should allow you to perform a simple search" do
              q = BookmarkQuery.new(query: "unicorns")
              search_body = q.generated_query
-        -    expect(search_body[:query][:bool][:should]).to include({:query_string => { :query => "unicorns" }})
+        +    expect(search_body[:query][:bool][:should]).to include({ query_string: { query: "unicorns", default_operator: "AND" }})