sunspot_matchers 1.3.0.2 → 2.0.0.0

data/.gitignore

@@ -1,3 +1,4 @@
 pkg/*
 *.gem
 .bundle
+.idea

data/Gemfile.lock

@@ -1,17 +1,16 @@
 PATH
   remote: .
   specs:
-    sunspot_matchers (1.3.0.2)
+    sunspot_matchers (2.0.0)
 
 GEM
   remote: http://rubygems.org/
   specs:
-    builder (3.0.0)
+    builder (3.2.0)
     diff-lcs (1.1.2)
-    escape (0.0.4)
     pr_geohash (1.0.0)
     rake (0.8.7)
-    rsolr (1.0.6)
+    rsolr (1.0.9)
       builder (>= 2.1.2)
     rspec (2.4.0)
       rspec-core (~> 2.4.0)
@@ -21,10 +20,9 @@ GEM
     rspec-expectations (2.4.0)
       diff-lcs (~> 1.1.2)
     rspec-mocks (2.4.0)
-    sunspot (1.3.0)
-      escape (~> 0.0.4)
+    sunspot (2.0.0)
       pr_geohash (~> 1.0)
-      rsolr (~> 1.0.6)
+      rsolr (~> 1.0.7)
 
 PLATFORMS
   ruby
@@ -33,5 +31,5 @@ DEPENDENCIES
   bundler (>= 1.0.0)
   rake
   rspec (~> 2.4.0)
-  sunspot (~> 1.3.0)
+  sunspot (~> 2.0.0)
   sunspot_matchers!

data/README.markdown

@@ -60,6 +60,12 @@ If you want to verify that a model has a searchable field, you can use this matc
 
 `Post.should have_searchable_field(:name)`
 
+You can also use the shorthand syntax:
+
+    describe User do
+      it { should have_searchable_field(:name) }
+    end
+
 ## have_search_params
 
 This is where the bulk of the functionality lies.  There are seven types of search matches you can perform: `keywords` or `fulltext`,
@@ -70,11 +76,15 @@ have a dozen `with` restrictions on a search and still write an expectation on a
 
 Negative expectations also work correctly.  `should_not` will fail if the search actually includes the expectation.
 
-With all matchers, you can specify a `Proc` as the second argument, and perform multi statement expectations inside the
-Proc.  Keep in mind, that only the expectation type specified in the first argument will actually be checked.  So if
-you specify `keywords` and `with` restrictions in the same Proc, but you said `have_search_params(:keywords, ...`
+With all matchers, you can pass a block and perform multi statement expectations inside the
+block.  Keep in mind, that only the expectation type specified as the argument will actually be checked.  So if
+you specify `keywords` and `with` restrictions in the same block, but you said `have_search_params(:keywords, ...`
 the `with` restrictions are simply ignored.
 
+Without creative usage of parentheses you can not use do...end blocks to pass multi statement expectations to the matcher
+though.  Because do...end have such a low binding precedent, the block is passed to the wrong method.  Curly syntax blocks
+will work though {...}, or you can pass a Proc as the last argument.
+
 ### wildcard matching
 
 keywords, with, without, and order_by support wildcard expectations using the `any_param` parameter:
@@ -109,7 +119,7 @@ You can match against a with restriction:
 
     Sunspot.session.should have_search_params(:with, :author_name, 'Mark Twain')
 
-Complex conditions can be matched by using a Proc instead of a value.  Be aware that order does matter, not for
+Complex conditions can be matched by using a block instead of a value.  Be aware that order does matter, not for
 the actual results that would come out of Solr, but the matcher will fail of the order of `with` restrictions is
 different.
 
@@ -120,12 +130,12 @@ different.
       end
     end
 
-    Sunspot.session.should have_search_params(:with, Proc.new {
+    Sunspot.session.should have_search_params(:with) {
       any_of do
         with :category_ids, 1
         with :category_ids, 2
       end
-    })
+    }
 
 ### :without
 
@@ -149,7 +159,7 @@ You can also specify only page or per_page, both are not required.
 
 ### :order_by
 
-Expectations on multiple orderings are supported using using the Proc format mentioned above.
+Expectations on multiple orderings are supported using using the block format mentioned above.
 
     Sunspot.search(Post) do
       order_by :published_at, :desc
@@ -174,10 +184,10 @@ Faceting where a query is excluded:
       facet(:category_ids, :exclude => category_filter)
     end
 
-    Sunspot.session.should have_search_params(:facet, Proc.new {
+    Sunspot.session.should have_search_params(:facet) {
       category_filter = with(:category_ids, 2)
       facet(:category_ids, :exclude => category_filter)
-    })
+    }
 
 Query faceting:
 
@@ -192,7 +202,7 @@ Query faceting:
       end
     end
     
-    Sunspot.session.should have_search_params(:facet, Proc.new {
+    Sunspot.session.should have_search_params(:facet) {
       facet(:average_rating) do
         row(1.0..2.0) do
           with(:average_rating, 1.0..2.0)
@@ -201,7 +211,7 @@ Query faceting:
           with(:average_rating, 2.0..3.0)
         end
       end
-    })
+    }
 
 ### :boost
 
@@ -213,11 +223,11 @@ Field boost matching:
       end
     end
 
-    Sunspot.session.should have_search_params(:boost, Proc.new {
+    Sunspot.session.should have_search_params(:boost) {
       keywords 'great pizza' do
         boost_fields :body => 2.0
       end
-    })
+    }
 
 Boost query matching:
 
@@ -229,13 +239,13 @@ Boost query matching:
       end
     end
 
-    Sunspot.session.should have_search_params(:boost, Proc.new {
+    Sunspot.session.should have_search_params(:boost) {
       keywords 'great pizza' do
         boost(2.0) do
           with :blog_id, 4
         end
       end
-    })
+    }
 
 Boost function matching:
 
@@ -245,11 +255,11 @@ Boost function matching:
       end
     end
 
-    Sunspot.session.should have_search_params(:boost, Proc.new {
+    Sunspot.session.should have_search_params(:boost) {
       keywords 'great pizza' do
         boost(function { sum(:average_rating, product(:popularity, 10)) })
       end
-    })
+    }
 
 # Assertions
 
@@ -270,7 +280,7 @@ These are used like:
     Sunspot.search(Post) do
       with :category_ids, 1..3
     end
-    assert_has_search_params Sunspot.session, :with, Proc.new {
+    assert_has_search_params Sunspot.session, :with {
       with :category_ids, 1..3
     }
     

data/lib/sunspot_matchers/matchers.rb

@@ -123,9 +123,9 @@ module SunspotMatchers
   end
 
   class HaveSearchParams
-    def initialize(method, *args)
+    def initialize(method, *args, &block)
       @method = method
-      @args = args
+      @args = [*args, block].compact
     end
 
     def matches?(actual)
@@ -158,12 +158,14 @@ module SunspotMatchers
           OrderByMatcher
         when :paginate
           PaginationMatcher
+        when :group
+          GroupMatcher
       end
     end
   end
 
-  def have_search_params(method, *args)
-    HaveSearchParams.new(method, *args)
+  def have_search_params(method, *args, &block)
+    HaveSearchParams.new(method, *args, &block)
   end
 
   class WithMatcher < BaseMatcher
@@ -273,6 +275,16 @@ module SunspotMatchers
     end
   end
 
+  class GroupMatcher < BaseMatcher
+    def search_method
+      :group
+    end
+
+    def keys_to_compare
+      comparison_params.keys.select {|key| /group/ =~ key.to_s}
+    end
+  end
+
   class BeASearchFor
     def initialize(expected_class)
       @expected_class = expected_class
@@ -312,11 +324,10 @@ module SunspotMatchers
       @field = field
     end
 
-    def matches?(klass)
-      @klass = klass
-      Sunspot::Setup.for(@klass).text_field_factories.find do |field|
-        field.name == @field
-      end
+    def matches?(klass_or_object)
+      @klass = klass_or_object.class.name == 'Class' ? klass_or_object : klass_or_object.class
+      @sunspot = Sunspot::Setup.for(@klass)
+      (@sunspot.all_text_fields + @sunspot.fields).collect(&:name).include?(@field)
     end
 
     def failure_message_for_should

data/lib/sunspot_matchers/test_helper.rb

@@ -6,10 +6,10 @@ module SunspotMatchers
   class HaveSearchParamsForSession
       include Test::Unit::Assertions
 
-      def initialize(session, method, *args)
+      def initialize(session, method, *args, &block)
         @session = session
         @method = method
-        @args = args
+        @args = [*args, block].compact
       end
 
       def get_matcher
@@ -63,15 +63,15 @@ module SunspotMatchers
     end
   end
   module TestHelper
-    def assert_has_search_params(session, *method_and_args)
+    def assert_has_search_params(session, *method_and_args, &block)
       method, *args = method_and_args
-      matcher = HaveSearchParamsForSession.new(session, method, *args).get_matcher
+      matcher = HaveSearchParamsForSession.new(session, method, *args, &block).get_matcher
       assert matcher.match?, matcher.missing_param_error_message
     end
 
-    def assert_has_no_search_params(session, *method_and_args)
+    def assert_has_no_search_params(session, *method_and_args, &block)
       method, *args = method_and_args
-      matcher = HaveSearchParamsForSession.new(session, method, *args).get_matcher
+      matcher = HaveSearchParamsForSession.new(session, method, *args, &block).get_matcher
       assert !matcher.match?, matcher.unexpected_match_error_message
     end
     

data/lib/sunspot_matchers/version.rb

@@ -1,3 +1,3 @@
 module SunspotMatchers
-  VERSION = "1.3.0.2"
+  VERSION = "2.0.0.0"
 end