Created pattern for testing index options more easily [skip ci]

Andrew
1 parent 64a827a0
Showing 3 changed files with 24 additions and 7 deletions Show diff stats
test/language_test.rb
test/reindex_test.rb
test/test_helper.rb
@@ -7,10 +7,12 @@ class LanguageTest &lt; Minitest::Test
   def test_chinese
     skip unless ENV["CHINESE"]
-    Song.reindex
-    store_names ["中华人民共和国国歌"], Song
-    assert_search "中华人民共和国", ["中华人民共和国国歌"], {}, Song
-    assert_search "国歌", ["中华人民共和国国歌"], {}, Song
-    assert_search "人", [], {}, Song
+
+    with_options(Song, language: "chinese") do
+      store_names ["中华人民共和国国歌"], Song
+      assert_search "中华人民共和国", ["中华人民共和国国歌"], {}, Song
+      assert_search "国歌", ["中华人民共和国国歌"], {}, Song
+      assert_search "人", [], {}, Song
+    end
   end
 end
 require_relative "test_helper"
 class ReindexTest < Minitest::Test
+  def setup
+    super
+    Sku.destroy_all
+  end
+
   def test_scoped
     skip if nobrainer? || cequel?
@@ -528,7 +528,7 @@ class Sku
 end
 class Song
-  searchkick language: "chinese"
+  searchkick
 end
 Product.searchkick_index.delete if Product.searchkick_index.exists?
@@ -547,7 +547,6 @@ class Minitest::Test
     Store.destroy_all
     Animal.destroy_all
     Speaker.destroy_all
-    Sku.destroy_all
   end
   protected
@@ -579,4 +578,15 @@ class Minitest::Test
   def assert_first(term, expected, options = {}, klass = Product)
     assert_equal expected, klass.search(term, options).map(&:name).first
   end
+
+  def with_options(model, options)
+    previous_options = model.searchkick_options.dup
+    begin
+      model.searchkick_options.merge!(options)
+      model.reindex
+    ensure
+      model.searchkick_options.clear
+      model.searchkick_options.merge!(previous_options)
+    end
+  end
 end