sunspot 2.2.7 → 2.5.0

data/spec/api/query/connectives_examples.rb

@@ -6,7 +6,7 @@ shared_examples_for "query with connective scope" do
         with :blog_id, 2
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :fq, '(category_ids_im:1 OR blog_id_i:2)'
     )
   end
@@ -21,7 +21,7 @@ shared_examples_for "query with connective scope" do
         end
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :fq,
       '(blog_id_i:2 OR (category_ids_im:1 AND average_rating_ft:{3\.0 TO *}))'
     )
@@ -37,7 +37,7 @@ shared_examples_for "query with connective scope" do
         end
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :fq, '(category_ids_im:1 OR (-average_rating_ft:{3\.0 TO *} AND blog_id_i:1))'
     )
   end
@@ -49,7 +49,7 @@ shared_examples_for "query with connective scope" do
         with :category_ids, 1
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :fq, 'blog_id_i:2', 'category_ids_im:1'
     )
   end
@@ -61,7 +61,7 @@ shared_examples_for "query with connective scope" do
         without(:average_rating).greater_than(3.0)
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :fq, '-(-category_ids_im:1 AND average_rating_ft:{3\.0 TO *})'
     )
   end
@@ -79,7 +79,7 @@ shared_examples_for "query with connective scope" do
         end
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :fq, '-(title_ss:Yes AND -(blog_id_i:1 AND -(-category_ids_im:4 AND average_rating_ft:2\.0)))'
     )
   end
@@ -96,7 +96,7 @@ shared_examples_for "query with connective scope" do
         end
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :fq, '(title_ss:Yes OR (blog_id_i:1 AND -(-category_ids_im:4 AND average_rating_ft:2\.0)))'
     )
   end
@@ -123,7 +123,7 @@ shared_examples_for "query with connective scope" do
         end
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :fq, '(title_ss:Yes OR blog_id_i:1 OR category_ids_im:4)'
     )
   end
@@ -136,7 +136,7 @@ shared_examples_for "query with connective scope" do
         with(:category_ids, 1)
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :fq, "-(id:(Post\\ #{post.id}) AND -category_ids_im:1)"
     )
   end
@@ -148,7 +148,7 @@ shared_examples_for "query with connective scope" do
         with(:average_rating).greater_than(3.0)
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :fq, '-(average_rating_ft:[* TO *] AND -average_rating_ft:{3\.0 TO *})'
     )
   end
@@ -161,7 +161,7 @@ shared_examples_for "query with connective scope" do
         with(:average_rating).greater_than(3.0)
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :fq, "(id:(Post\\ #{post.id}) OR average_rating_ft:{3\\.0 TO *})"
     )
   end
@@ -175,7 +175,7 @@ shared_examples_for "query with connective scope" do
         with(:blog_id, 1)
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :fq, '(title_text:test* OR blog_id_i:1)'
     )
   end
@@ -184,7 +184,7 @@ shared_examples_for "query with connective scope" do
     search do
       any_of {}
     end
-    connection.should_not have_last_search_including(:fq, '')
+    expect(connection).not_to have_last_search_including(:fq, '')
   end
 
   it 'creates a conjunction of in_radius queries' do
@@ -194,8 +194,20 @@ shared_examples_for "query with connective scope" do
         with(:coordinates_new).in_radius(42, 56, 50)
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :fq, '(_query_:"{!geofilt sfield=coordinates_new_ll pt=23,-46 d=100}" OR _query_:"{!geofilt sfield=coordinates_new_ll pt=42,56 d=50}")'
     )
   end
+  
+  it 'creates a conjunction of in_bounding_box queries' do
+    search do
+      any_of do
+        with(:coordinates_new).in_bounding_box([23, -46], [25, -44])
+        with(:coordinates_new).in_bounding_box([42, 56], [43, 58])
+      end
+    end
+    expect(connection).to have_last_search_including(
+      :fq, '(coordinates_new_ll:[23,-46 TO 25,-44] OR coordinates_new_ll:[42,56 TO 43,58])'
+    )
+  end
 end

data/spec/api/query/dsl_spec.rb

@@ -1,28 +1,36 @@
 require File.expand_path('spec_helper', File.dirname(__FILE__))
 
 describe 'query DSL', :type => :query do
+  let(:blog_id) { 1 }
+
   it 'should allow building search using block argument rather than instance_eval' do
-    @blog_id = 1
     session.search Post do |query|
       query.field_list [:blog_id, :title]
-      query.with(:blog_id, @blog_id)
+      query.with(:blog_id, blog_id)
     end
-    connection.should have_last_search_including(:fq, 'blog_id_i:1')
-    connection.should have_last_search_with(fl: [:blog_id, :title])
+    expect(connection).to have_last_search_including(:fq, 'blog_id_i:1')
+    expect(connection).to have_last_search_with(fl: [:id, :blog_id_i, :title_ss])
   end
 
   it 'should allow field_list specified as arguments' do
-    @blog_id = 1
     session.search Post do |query|
       query.field_list :blog_id, :title
-      query.with(:blog_id, @blog_id)
+      query.with(:blog_id, blog_id)
+    end
+    expect(connection).to have_last_search_with(fl: [:id, :blog_id_i, :title_ss])
+  end
+
+  it 'should allow to skip stored fields retrieval' do
+    session.search Post do |query|
+      query.with(:blog_id, blog_id)
+      query.without_stored_fields
     end
-    connection.should have_last_search_with(fl: [:blog_id, :title])
+    expect(connection).to have_last_search_with(fl: [:id])
   end
 
   it 'should accept a block in the #new_search method' do
-    search = session.new_search(Post) { with(:blog_id, 1) }
+    search = session.new_search(Post) { with(:blog_id, blog_id) }
     search.execute
-    connection.should have_last_search_including(:fq, 'blog_id_i:1')
+    expect(connection).to have_last_search_including(:fq, 'blog_id_i:1')
   end
 end

data/spec/api/query/dynamic_fields_examples.rb

@@ -5,7 +5,7 @@ shared_examples_for "query with dynamic field support" do
         with :test, 'string'
       end
     end
-    connection.should have_last_search_including(:fq, 'custom_string\:test_ss:string')
+    expect(connection).to have_last_search_including(:fq, 'custom_string\:test_ss:string')
   end
 
   it 'restricts by dynamic integer field with less than restriction' do
@@ -14,7 +14,7 @@ shared_examples_for "query with dynamic field support" do
         with(:test).less_than(1)
       end
     end
-    connection.should have_last_search_including(:fq, 'custom_integer\:test_i:{* TO 1}')
+    expect(connection).to have_last_search_including(:fq, 'custom_integer\:test_i:{* TO 1}')
   end
 
   it 'restricts by dynamic float field with between restriction' do
@@ -23,7 +23,7 @@ shared_examples_for "query with dynamic field support" do
         with(:test).between(2.2..3.3)
       end
     end
-    connection.should have_last_search_including(:fq, 'custom_float\:test_fm:[2\.2 TO 3\.3]')
+    expect(connection).to have_last_search_including(:fq, 'custom_float\:test_fm:[2\.2 TO 3\.3]')
   end
 
   it 'restricts by dynamic time field with any of restriction' do
@@ -33,7 +33,7 @@ shared_examples_for "query with dynamic field support" do
                             Time.parse('2009-02-13 18:00:00 UTC')])
       end
     end
-    connection.should have_last_search_including(:fq, 'custom_time\:test_d:(2009\-02\-10T14\:00\:00Z OR 2009\-02\-13T18\:00\:00Z)')
+    expect(connection).to have_last_search_including(:fq, 'custom_time\:test_d:(2009\-02\-10T14\:00\:00Z OR 2009\-02\-13T18\:00\:00Z)')
   end
 
   it 'restricts by dynamic boolean field with equality restriction' do
@@ -42,7 +42,7 @@ shared_examples_for "query with dynamic field support" do
         with :test, false
       end
     end
-    connection.should have_last_search_including(:fq, 'custom_boolean\:test_b:false')
+    expect(connection).to have_last_search_including(:fq, 'custom_boolean\:test_b:false')
   end
 
   it 'negates a dynamic field restriction' do
@@ -51,7 +51,7 @@ shared_examples_for "query with dynamic field support" do
         without :test, 'foo'
       end
     end
-    connection.should have_last_search_including(:fq, '-custom_string\:test_ss:foo')
+    expect(connection).to have_last_search_including(:fq, '-custom_string\:test_ss:foo')
   end
 
   it 'scopes by a dynamic field inside a disjunction' do
@@ -63,7 +63,7 @@ shared_examples_for "query with dynamic field support" do
         with :title, 'bar'
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :fq, '(custom_string\:test_ss:foo OR title_ss:bar)'
     )
   end
@@ -74,7 +74,7 @@ shared_examples_for "query with dynamic field support" do
         order_by :test, :desc
       end
     end
-    connection.should have_last_search_with(:sort => 'custom_integer:test_i desc')
+    expect(connection).to have_last_search_with(:sort => 'custom_integer:test_i desc')
   end
 
   it 'orders by a dynamic field and static field, with given precedence' do
@@ -84,25 +84,25 @@ shared_examples_for "query with dynamic field support" do
       end
       order_by :sort_title, :asc
     end
-    connection.should have_last_search_with(:sort => 'custom_integer:test_i desc, sort_title_s asc')
+    expect(connection).to have_last_search_with(:sort => 'custom_integer:test_i desc, sort_title_s asc')
   end
 
   it 'raises an UnrecognizedFieldError if an unknown dynamic field is searched by' do
-    lambda do
+    expect do
       search do
         dynamic(:bogus) { with :some, 'value' }
       end
-    end.should raise_error(Sunspot::UnrecognizedFieldError)
+    end.to raise_error(Sunspot::UnrecognizedFieldError)
   end
 
   it 'raises a NoMethodError if pagination is attempted in a dynamic query' do
-    lambda do
+    expect do
       search do
         dynamic :custom_string do
           paginate :page => 3, :per_page => 10
         end
       end
-    end.should raise_error(NoMethodError)
+    end.to raise_error(NoMethodError)
   end
 
   it 'requests field facet on dynamic field' do
@@ -111,7 +111,7 @@ shared_examples_for "query with dynamic field support" do
         facet(:test)
       end
     end
-    connection.should have_last_search_including(:"facet.field", 'custom_string:test_ss')
+    expect(connection).to have_last_search_including(:"facet.field", 'custom_string:test_ss')
   end
 
   it 'requests named field facet on dynamic field' do
@@ -120,7 +120,7 @@ shared_examples_for "query with dynamic field support" do
         facet(:test, :name => :bogus)
       end
     end
-    connection.should have_last_search_including(:"facet.field", '{!key=bogus}custom_string:test_ss')
+    expect(connection).to have_last_search_including(:"facet.field", '{!key=bogus}custom_string:test_ss')
   end
 
   it 'requests query facet with internal dynamic field' do
@@ -133,7 +133,7 @@ shared_examples_for "query with dynamic field support" do
         end
       end
     end
-    connection.should have_last_search_with(
+    expect(connection).to have_last_search_with(
       :"facet.query" => 'custom_string\:test_ss:foo'
     )
   end
@@ -148,7 +148,7 @@ shared_examples_for "query with dynamic field support" do
         end
       end
     end
-    connection.should have_last_search_including(
+    expect(connection).to have_last_search_including(
       :"facet.query",
       'custom_string\:test_ss:foo'
     )
@@ -160,6 +160,6 @@ shared_examples_for "query with dynamic field support" do
         with(:test, 1.23)
       end
     end
-    connection.should have_last_search_including(:fq, 'custom_float\\:test_fm:1\\.23')
+    expect(connection).to have_last_search_including(:fq, 'custom_float\\:test_fm:1\\.23')
   end
 end