Replace .where with in_order_of for results method (#30)

npezza93 · Aug 28, 2024 · babf321 · babf321
1 parent ca781c4
commit babf321
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/lib/redi_search/search.rb b/lib/redi_search/search.rb
@@ -23,7 +23,7 @@ def results
       if model
         no_content unless loaded?
 
-        model.where(id: to_a.map(&:document_id_without_index))
+        model.in_order_of(:id, to_a.map(&:document_id_without_index))
       else
         to_a
       end

diff --git a/test/unit/redi_search/search_test.rb b/test/unit/redi_search/search_test.rb
@@ -62,11 +62,11 @@ def active_record_relation_double
     end
 
     def search_with_model
-      model = Minitest::Mock.new.expect(:where, active_record_relation_double,
-                                        [], id: ["1"])
-      Search.new(
-        Index.new(:users, model) { text_field :name }, "foo"
-      ).then { |search| yield search }
+      model = Minitest::Mock.new.expect(:in_order_of, 
+                                        active_record_relation_double, 
+                                        [:id, ["1"]])
+      Search.new(Index.new(:users, model) { text_field :name }, 
+                 "foo").then { |search| yield search }
 
       assert_mock model
     end