sunspot 2.2.7 → 2.2.8

data/spec/integration/field_lists_spec.rb

@@ -0,0 +1,41 @@
+require File.expand_path('../spec_helper', File.dirname(__FILE__))
+
+describe 'fields lists' do
+  before :all do
+    Sunspot.remove_all
+    @post = Post.new(title: 'A Title', body: 'A Body', featured: true, tags: ['tag'])
+    Sunspot.index!(@post)
+  end
+
+  let(:stored_field_names) do
+    (Sunspot::Setup.for(Post).fields + Sunspot::Setup.for(Post).all_text_fields)
+      .select { |f| f.stored? }
+      .map { |f| f.name }
+  end
+
+  it 'loads all stored fields by dafault' do
+    hit = Sunspot.search(Post).hits.first
+
+    stored_field_names.each do |field|
+      expect(hit.stored(field)).not_to be_nil
+    end
+  end
+
+  it 'loads only filtered fields' do
+    hit = Sunspot.search(Post) { field_list(:title) }.hits.first
+
+    expect(hit.stored(:title)).to eq(@post.title)
+
+    (stored_field_names - [:title]).each do |field|
+      expect(hit.stored(field)).to be_nil
+    end
+  end
+
+  it 'does not load any stored fields' do
+    hit = Sunspot.search(Post) { without_stored_fields }.hits.first
+
+    stored_field_names.each do |field|
+      expect(hit.stored(field)).to be_nil
+    end
+  end
+end

data/spec/integration/geospatial_spec.rb

@@ -15,7 +15,7 @@ describe "geospatial search" do
         with(:coordinates_new).in_radius(32, -68, 1)
       }.results
 
-      results.should include(@post)
+      expect(results).to include(@post)
     end
 
     it "filters out posts not in the radius" do
@@ -23,10 +23,10 @@ describe "geospatial search" do
         with(:coordinates_new).in_radius(33, -68, 1)
       }.results
 
-      results.should_not include(@post)
+      expect(results).not_to include(@post)
     end
 
-    it "allows conjunction queries" do
+    it "allows conjunction queries with radius" do
       results = Sunspot.search(Post) {
         any_of do
           with(:coordinates_new).in_radius(32, -68, 1)
@@ -34,7 +34,18 @@ describe "geospatial search" do
         end
       }.results
 
-      results.should include(@post)
+      expect(results).to include(@post)
+    end
+
+    it "allows conjunction queries with bounding box" do
+      results = Sunspot.search(Post) {
+        any_of do
+          with(:coordinates_new).in_bounding_box([31, -69], [33, -67])
+          with(:coordinates_new).in_bounding_box([35, 68], [36, 69])
+        end
+      }.results
+
+      expect(results).to include(@post)
     end
   end
 
@@ -52,7 +63,7 @@ describe "geospatial search" do
         with(:coordinates_new).in_bounding_box [31, -69], [33, -67]
       }.results
 
-      results.should include(@post)
+      expect(results).to include(@post)
     end
 
     it "filters out posts not in the bounding box" do
@@ -60,7 +71,7 @@ describe "geospatial search" do
         with(:coordinates_new).in_bounding_box [20, -70], [21, -69]
       }.results
 
-      results.should_not include(@post)
+      expect(results).not_to include(@post)
     end
   end
 
@@ -82,7 +93,7 @@ describe "geospatial search" do
         order_by_geodist(:coordinates_new, 32, -68)
       }.results
 
-      results.should == @posts.reverse
+      expect(results).to eq(@posts.reverse)
     end
 
     it "orders posts by distance descending" do
@@ -90,7 +101,7 @@ describe "geospatial search" do
         order_by_geodist(:coordinates_new, 32, -68, :desc)
       }.results
 
-      results.should == @posts
+      expect(results).to eq(@posts)
     end
   end
 end

data/spec/integration/highlighting_spec.rb

@@ -11,16 +11,16 @@ describe 'keyword highlighting' do
   end
   
   it 'should include highlights in the results' do
-    @search_result.hits.first.highlights.length.should == 1
+    expect(@search_result.hits.first.highlights.length).to eq(1)
   end
   
   it 'should return formatted highlight fragments' do
-    @search_result.hits.first.highlights(:body).first.format.should == 'And the <em>fox</em> laughed'
+    expect(@search_result.hits.first.highlights(:body).first.format).to eq('And the <em>fox</em> laughed')
   end
   
   it 'should be empty for non-keyword searches' do
     search_result = Sunspot.search(Post){ with :blog_id, 1 }
-    search_result.hits.first.highlights.should be_empty
+    expect(search_result.hits.first.highlights).to be_empty
   end
   
   it "should process multiple keyword request on different fields with highlights correctly" do
@@ -35,8 +35,8 @@ describe 'keyword highlighting' do
         end
       end
     end.to_not raise_error
-    search_results.results.length.should eq(1)
-    search_results.results.first.should eq(@posts.last)
+    expect(search_results.results.length).to eq(1)
+    expect(search_results.results.first).to eq(@posts.last)
     # this one might be a Solr bug, therefore not related to Sunspot itself
     # search_results.hits.first.highlights.should_not be_empty
   end

data/spec/integration/indexing_spec.rb

@@ -2,23 +2,23 @@ require File.expand_path('../spec_helper', File.dirname(__FILE__))
 
 describe 'indexing' do
   it 'should index non-multivalued field with newlines' do
-    lambda do
+    expect do
       Sunspot.index!(Post.new(:title => "A\nTitle"))
-    end.should_not raise_error
+    end.not_to raise_error
   end
 
   it 'should correctly remove by model instance' do
     post = Post.new(:title => 'test post')
     Sunspot.index!(post)
     Sunspot.remove!(post)
-    Sunspot.search(Post) { with(:title, 'test post') }.results.should be_empty
+    expect(Sunspot.search(Post) { with(:title, 'test post') }.results).to be_empty
   end
 
   it 'should correctly delete by ID' do
     post = Post.new(:title => 'test post')
     Sunspot.index!(post)
     Sunspot.remove_by_id!(Post, post.id)
-    Sunspot.search(Post) { with(:title, 'test post') }.results.should be_empty
+    expect(Sunspot.search(Post) { with(:title, 'test post') }.results).to be_empty
   end
 
   it 'removes documents by query' do
@@ -29,7 +29,7 @@ describe 'indexing' do
     Sunspot.remove!(Post) do
       with(:title, 'birds')
     end
-    Sunspot.search(Post).should have(1).results
+    expect(Sunspot.search(Post).results.size).to eq(1)
   end
 
 

data/spec/integration/keyword_search_spec.rb

@@ -19,8 +19,8 @@ describe 'keyword search' do
     context 'edismax' do
       it 'matches with wildcards' do
         results = Sunspot.search(Post) { keywords '*oas*' }.results
-        [0,2].each { |i| results.should include(@posts[i])}
-        [1].each { |i| results.should_not include(@posts[i])}
+        [0,2].each { |i| expect(results).to include(@posts[i])}
+        [1].each { |i| expect(results).not_to include(@posts[i])}
       end
 
       it 'matches multiple keywords on different fields with wildcards using subqueries' do
@@ -28,52 +28,52 @@ describe 'keyword search' do
           keywords 'insuffic*',:fields=>[:title]
           keywords 'win*',:fields=>[:body]
         end.results
-        [0].each {|i| results.should include(@posts[i])}
-        [1,2].each {|i| results.should_not include(@posts[i])}
+        [0].each {|i| expect(results).to include(@posts[i])}
+        [1,2].each {|i| expect(results).not_to include(@posts[i])}
       end
 
       it 'matches with proximity' do
         results = Sunspot.search(Post) { keywords '"wind buffer"~4' }.results
-        [0,1].each {|i| results.should_not include(@posts[i])}
-        [2].each {|i| results.should include(@posts[i])}
+        [0,1].each {|i| expect(results).not_to include(@posts[i])}
+        [2].each {|i| expect(results).to include(@posts[i])}
       end
 
       it 'does not match if not within proximity' do
         results = Sunspot.search(Post) { keywords '"wind buffer"~1' }.results
-        results.should == []
+        expect(results).to eq([])
       end
     end
 
     it 'matches a single keyword out of a single field' do
       results = Sunspot.search(Post) { keywords 'toast' }.results
-      [0, 2].each { |i| results.should include(@posts[i]) }
-      [1].each { |i| results.should_not include(@posts[i]) }
+      [0, 2].each { |i| expect(results).to include(@posts[i]) }
+      [1].each { |i| expect(results).not_to include(@posts[i]) }
     end
 
     it 'matches multiple words out of a single field' do
       results = Sunspot.search(Post) { keywords 'elects toast' }.results
-      results.should == [@posts[0]]
+      expect(results).to eq([@posts[0]])
     end
 
     it 'matches multiple words in multiple fields' do
       results = Sunspot.search(Post) { keywords 'toast wind' }.results
-      [0, 2].each { |i| results.should include(@posts[i]) }
-      [1].each { |i| results.should_not include(@posts[i]) }
+      [0, 2].each { |i| expect(results).to include(@posts[i]) }
+      [1].each { |i| expect(results).not_to include(@posts[i]) }
     end
 
     it 'matches multiple types' do
       results = Sunspot.search(Post, Namespaced::Comment) do
         keywords 'toast'
       end.results
-      [@posts[0], @posts[2], @comment].each  { |obj| results.should include(obj) }
-      results.should_not include(@posts[1])
+      [@posts[0], @posts[2], @comment].each  { |obj| expect(results).to include(obj) }
+      expect(results).not_to include(@posts[1])
     end
 
     it 'matches keywords from only the fields specified' do
       results = Sunspot.search(Post) do
         keywords 'moron', :fields => [:title]
       end.results
-      results.should == [@posts[1]]
+      expect(results).to eq([@posts[1]])
     end
 
     it 'matches multiple keywords on different fields using subqueries' do
@@ -81,13 +81,13 @@ describe 'keyword search' do
         keywords 'moron', :fields => [:title]
         keywords 'wind',  :fields => [:body]
       end
-      search.results.should == []
+      expect(search.results).to eq([])
 
       search = Sunspot.search(Post) do
         keywords 'moron',   :fields => [:title]
         keywords 'buffer',  :fields => [:body]
       end
-      search.results.should == [@posts[1]]
+      expect(search.results).to eq([@posts[1]])
     end
 
     it 'matches multiple keywords with escaped characters' do
@@ -95,7 +95,7 @@ describe 'keyword search' do
         keywords 'spirit',   :fields => [:title]
         keywords 'host\'s',  :fields => [:body]
       end
-      search.results.should == [@posts[2]]
+      expect(search.results).to eq([@posts[2]])
     end
 
     it 'matches multiple keywords with phrase-based search' do
@@ -104,7 +104,7 @@ describe 'keyword search' do
         keywords '"interpret the buffer"', :fields => [:body]
         keywords '"does the"', :fields => [:body]
       end
-      search.results.should == [@posts[2]]
+      expect(search.results).to eq([@posts[2]])
     end
 
     it 'matches multiple keywords different options' do
@@ -112,7 +112,7 @@ describe 'keyword search' do
         keywords 'insufficient nonexistent', :fields => [:title], :minimum_match => 1
         keywords 'wind does', :fields => [:body], :minimum_match => 2
       end
-      search.results.should == [@posts[0]]
+      expect(search.results).to eq([@posts[0]])
     end
   end
 
@@ -125,9 +125,10 @@ describe 'keyword search' do
 
     it 'should assign a higher score to the result matching the higher-boosted field' do
       search = Sunspot.search(Post) { keywords 'rhinoceros' }
-      search.hits.map { |hit| hit.primary_key }.should ==
+      expect(search.hits.map { |hit| hit.primary_key }).to eq(
         @posts.map { |post| post.id.to_s }
-      search.hits.first.score.should > search.hits.last.score
+      )
+      expect(search.hits.first.score).to be > search.hits.last.score
     end
   end
 
@@ -142,9 +143,10 @@ describe 'keyword search' do
 
     it 'should assign a higher score to the higher-boosted document' do
       search = Sunspot.search(Post) { keywords 'test' }
-      search.hits.map { |hit| hit.primary_key }.should ==
+      expect(search.hits.map { |hit| hit.primary_key }).to eq(
         @posts.map { |post| post.id.to_s }
-      search.hits.first.score.should > search.hits.last.score
+      )
+      expect(search.hits.first.score).to be > search.hits.last.score
     end
   end
 
@@ -164,8 +166,8 @@ describe 'keyword search' do
           phrase_fields :body => 2.0
         end
       end.hits
-      hits.first.instance.should == @comments.first
-      hits.first.score.should > hits.last.score
+      expect(hits.first.instance).to eq(@comments.first)
+      expect(hits.first.score).to be > hits.last.score
     end
 
     it 'assigns a higher score to documents in which the search terms appear in a boosted field' do
@@ -174,8 +176,8 @@ describe 'keyword search' do
           fields :body => 2.0, :author_name => 0.75
         end
       end.hits
-      hits.first.instance.should == @comments.first
-      hits.first.score.should > hits.last.score
+      expect(hits.first.instance).to eq(@comments.first)
+      expect(hits.first.score).to be > hits.last.score
     end
 
     it 'assigns a higher score to documents in which the search terms appear in a higher boosted phrase field' do
@@ -184,8 +186,8 @@ describe 'keyword search' do
           phrase_fields :body => 2.0, :author_name => 0.75
         end
       end.hits
-      hits.first.instance.should == @comments.first
-      hits.first.score.should > hits.last.score
+      expect(hits.first.instance).to eq(@comments.first)
+      expect(hits.first.score).to be > hits.last.score
     end
   end
 
@@ -210,8 +212,8 @@ describe 'keyword search' do
         end
         query.without(@posts[1])
       end
-      search.results.should == [@posts[0], @posts[2]]
-      search.hits[0].score.should > search.hits[1].score
+      expect(search.results).to eq([@posts[0], @posts[2]])
+      expect(search.hits[0].score).to be > search.hits[1].score
     end
 
     it 'should assign scores in order of multiple boost query match' do
@@ -221,9 +223,9 @@ describe 'keyword search' do
           boost(1.5) { with(:average_rating).greater_than(3.0) }
         end
       end
-      search.results.should == @posts
-      search.hits[0].score.should > search.hits[1].score
-      search.hits[1].score.should > search.hits[2].score
+      expect(search.results).to eq(@posts)
+      expect(search.hits[0].score).to be > search.hits[1].score
+      expect(search.hits[1].score).to be > search.hits[2].score
     end
   end
 
@@ -241,11 +243,11 @@ describe 'keyword search' do
     end
 
     it 'should match documents that contain the minimum_match number of search terms' do
-      @search.results.should include(@posts[0])
+      expect(@search.results).to include(@posts[0])
     end
 
     it 'should not match documents that do not contain the minimum_match number of search terms' do
-      @search.results.should_not include(@posts[1])
+      expect(@search.results).not_to include(@posts[1])
     end
   end
 
@@ -264,15 +266,15 @@ describe 'keyword search' do
     end
 
     it 'should match exact phrase' do
-      @search.results.should include(@posts[0])
+      expect(@search.results).to include(@posts[0])
     end
 
     it 'should match phrase divided by query phrase slop terms' do
-      @search.results.should include(@posts[1])
+      expect(@search.results).to include(@posts[1])
     end
 
     it 'should not match phrase divided by more than query phrase slop terms' do
-      @search.results.should_not include(@posts[2])
+      expect(@search.results).not_to include(@posts[2])
     end
   end
 
@@ -298,15 +300,15 @@ describe 'keyword search' do
     end
 
     it 'should give phrase field boost to exact match' do
-      @sorted_hits[0].score.should > @sorted_hits[1].score
+      expect(@sorted_hits[0].score).to be > @sorted_hits[1].score
     end
 
     it 'should give phrase field boost to match within slop' do
-      @sorted_hits[2].score.should > @sorted_hits[3].score
+      expect(@sorted_hits[2].score).to be > @sorted_hits[3].score
     end
 
     it 'should not give phrase field boost to match beyond slop' do
-      @sorted_hits[4].score.should == @sorted_hits[5].score
+      expect(@sorted_hits[4].score).to eq(@sorted_hits[5].score)
     end
   end
 
@@ -316,8 +318,8 @@ describe 'keyword search' do
     end
 
     after :each do
-      @search.results.should == @posts
-      @search.hits.first.score.should > @search.hits.last.score
+      expect(@search.results).to eq(@posts)
+      expect(@search.hits.first.score).to be > @search.hits.last.score
     end
 
     it 'boosts via function query with float' do

data/spec/integration/local_search_spec.rb

@@ -25,13 +25,13 @@ describe 'local search' do
     end
 
     it 'should return results in geo order' do
-      @search.results.should == @posts
+      expect(@search.results).to eq(@posts)
     end
 
     it 'should asssign higher score to closer locations' do
       hits = @search.hits
       hits[1..-1].each_with_index do |hit, i|
-        hit.score.should < hits[i].score
+        expect(hit.score).to be < hits[i].score
       end
     end
   end
@@ -51,13 +51,13 @@ describe 'local search' do
     end
 
     it 'should take both fulltext and distance into account in ordering' do
-      @search.results.should == @posts
+      expect(@search.results).to eq(@posts)
     end
 
     it 'should take both fulltext and distance into account in scoring' do
       hits = @search.hits
       hits[1..-1].each_with_index do |hit, i|
-        hit.score.should < hits[i].score
+        expect(hit.score).to be < hits[i].score
       end
     end
   end