nuatt_sunspot 1.1.0.3

data/spec/api/adapters_spec.rb

@@ -0,0 +1,33 @@
+require File.join(File.dirname(__FILE__), 'spec_helper')
+
+describe Sunspot::Adapters::InstanceAdapter do
+  it "finds adapter by superclass" do
+    Sunspot::Adapters::InstanceAdapter::for(Model).should be(AbstractModelInstanceAdapter)
+  end
+
+  it "finds adapter by mixin" do
+    Sunspot::Adapters::InstanceAdapter::for(MixModel).should be(MixInModelInstanceAdapter)
+  end
+
+  it 'throws NoAdapterError if anonymous module passed in' do
+    lambda do
+      Sunspot::Adapters::InstanceAdapter::for(Module.new)
+    end.should raise_error(Sunspot::NoAdapterError)
+  end
+end
+
+describe Sunspot::Adapters::DataAccessor do
+  it "finds adapter by superclass" do
+    Sunspot::Adapters::DataAccessor::for(Model).should be(AbstractModelDataAccessor)
+  end
+
+  it "finds adapter by mixin" do
+    Sunspot::Adapters::DataAccessor::for(MixModel).should be(MixInModelDataAccessor)
+  end
+
+  it 'throws NoAdapterError if anonymous module passed in' do
+    lambda do
+      Sunspot::Adapters::DataAccessor::for(Module.new)
+    end.should raise_error(Sunspot::NoAdapterError)
+  end
+end

data/spec/api/binding_spec.rb

@@ -0,0 +1,38 @@
+require File.join(File.dirname(__FILE__), 'spec_helper')
+
+describe "DSL bindings" do
+  it 'should give access to calling context\'s methods in search DSL' do
+    value = nil
+    session.search(Post) do
+      value = test_method
+    end
+    value.should == 'value'
+  end
+
+  it 'should give access to calling context\'s methods in nested DSL block' do
+    value = nil
+    session.search(Post) do
+      any_of do
+        value = test_method
+      end
+    end
+    value.should == 'value'
+  end
+
+  it 'should give access to calling context\'s methods in double-nested DSL block' do
+    value = nil
+    session.search(Post) do
+      any_of do
+        all_of do
+          value = test_method
+        end
+      end
+    end
+  end
+
+  private
+
+  def test_method
+    'value'
+  end
+end

data/spec/api/indexer/attributes_spec.rb

@@ -0,0 +1,149 @@
+require File.join(File.dirname(__FILE__), 'spec_helper')
+
+describe 'indexing attribute fields', :type => :indexer do
+  it 'should correctly index a stored string attribute field' do
+    session.index(post(:title => 'A Title'))
+    connection.should have_add_with(:title_ss => 'A Title')
+  end
+
+  it 'should correctly index an integer attribute field' do
+    session.index(post(:blog_id => 4))
+    connection.should have_add_with(:blog_id_i => '4')
+  end
+
+  it 'should correctly index a long attribute field' do
+    session.index(Namespaced::Comment.new(:hash => 2**30))
+    connection.should have_add_with(:hash_l => '1073741824')
+  end
+
+  it 'should correctly index a float attribute field' do
+    session.index(post(:ratings_average => 2.23))
+    connection.should have_add_with(:average_rating_ft => '2.23')
+  end
+
+  it 'should correctly index a double attribute field' do
+    session.index(Namespaced::Comment.new(:average_rating => 2.23))
+    connection.should have_add_with(:average_rating_e => '2.23')
+  end
+
+  it 'should correctly index a trie integer attribute field' do
+    session.index(Photo.new(:size => 104856))
+    connection.should have_add_with(:size_it => '104856')
+  end
+
+  it 'should correctly index a trie float attribute field' do
+    session.index(Photo.new(:average_rating => 2.23))
+    connection.should have_add_with(:average_rating_ft => '2.23')
+  end
+
+  it 'should correctly index a trie time attribute field' do
+    session.index(Photo.new(:created_at => Time.parse('2009-12-16 15:00:00 -0400')))
+    connection.should have_add_with(:created_at_dt => '2009-12-16T19:00:00Z')
+  end
+
+  it 'should allow indexing by a multiple-value field' do
+    session.index(post(:category_ids => [3, 14]))
+    connection.should have_add_with(:category_ids_im => ['3', '14'])
+  end
+
+  it 'should not index a single-value field with newlines as multiple' do
+    session.index(post(:title => "Multi\nLine"))
+    connection.adds.last.first.field_by_name(:title_ss).value.should == "Multi\nLine"
+  end
+
+  it 'should correctly index a time field' do
+    session.index(
+      post(:published_at => Time.parse('1983-07-08 05:00:00 -0400'))
+    )
+    connection.should have_add_with(:published_at_dt => '1983-07-08T09:00:00Z')
+  end
+
+  it 'should correctly index a time field that\'s after 32-bit Y2K' do
+    session.index(
+      post(:published_at => DateTime.parse('2050-07-08 05:00:00 -0400'))
+    )
+    connection.should have_add_with(:published_at_dt => '2050-07-08T09:00:00Z')
+  end
+
+  it 'should correctly index a date field' do
+    session.index(post(:expire_date => Date.new(2009, 07, 13)))
+    connection.should have_add_with(:expire_date_d => '2009-07-13T00:00:00Z')
+  end
+
+  it 'should correctly index a boolean field' do
+    session.index(post(:featured => true))
+    connection.should have_add_with(:featured_b => 'true')
+  end
+
+  it 'should correctly index a false boolean field' do
+    session.index(post(:featured => false))
+    connection.should have_add_with(:featured_b => 'false')
+  end
+
+  it 'should not index a nil boolean field' do
+    session.index(post)
+    connection.should_not have_add_with(:featured_b)
+  end
+
+  it 'should index latitude and longitude as a pair' do
+    session.index(post(:coordinates => [40.7, -73.5]))
+    connection.should have_add_with(:lat => 40.7, :lng => -73.5)
+  end
+
+  [
+    [:lat, :lng],
+    [:lat, :lon],
+    [:lat, :long],
+    [:latitude, :longitude]
+  ].each do |lat_attr, lng_attr|
+    it "should index latitude and longitude from #{lat_attr.inspect}, #{lng_attr.inspect}" do
+      session.index(post(
+          :coordinates => OpenStruct.new(lat_attr => 40.7, lng_attr => -73.5)
+      ))
+      connection.should have_add_with(:lat => 40.7, :lng => -73.5)
+    end
+  end
+
+  it 'should index latitude and longitude from a block' do
+    session.index(Photo.new(:lat => 30, :lng => -60))
+    connection.should have_add_with(:lat => 30.0, :lng => -60.0)
+  end
+
+  it 'should correctly index an attribute field with block access' do
+    session.index(post(:title => 'The Blog Post'))
+    connection.should have_add_with(:sort_title_s => 'blog post')
+  end
+
+  it 'should correctly index an attribute field with instance-external block access' do
+    session.index(post(:category_ids => [1, 2, 3]))
+    connection.should have_add_with(:primary_category_id_i => '1')
+  end
+
+  it 'should correctly index a field that is defined on a superclass' do
+    Sunspot.setup(SuperClass) { string :author_name }
+    session.index(post(:author_name => 'Mat Brown'))
+    connection.should have_add_with(:author_name_s => 'Mat Brown')
+  end
+
+  it 'should throw a NoMethodError only if a nonexistent type is defined' do
+    lambda { Sunspot.setup(Post) { string :author_name }}.should_not raise_error
+    lambda { Sunspot.setup(Post) { bogus :journey }}.should raise_error(NoMethodError)
+  end
+
+  it 'should throw a NoMethodError if a nonexistent field argument is passed' do
+    lambda { Sunspot.setup(Post) { string :author_name, :bogus => :argument }}.should raise_error(ArgumentError)
+  end
+
+  it 'should throw an ArgumentError if single-value field tries to index multiple values' do
+    lambda do
+      Sunspot.setup(Post) { string :author_name }
+      session.index(post(:author_name => ['Mat Brown', 'Matthew Brown']))
+    end.should raise_error(ArgumentError)
+  end
+
+  it 'should throw an ArgumentError if specifying more_like_this on type that does not support it' do
+    lambda do
+      Sunspot.setup(Post) { integer :popularity, :more_like_this => true }
+    end.should raise_error(ArgumentError)
+  end
+end

data/spec/api/indexer/batch_spec.rb

@@ -0,0 +1,46 @@
+require File.join(File.dirname(__FILE__), 'spec_helper')
+
+describe 'batch indexing', :type => :indexer do
+  it 'should send all batched adds in a single request' do
+    posts = Array.new(2) { Post.new }
+    session.batch do
+      for post in posts
+        session.index(post)
+      end
+    end
+    connection.adds.length.should == 1
+  end
+
+  it 'should add all batched adds' do
+    posts = Array.new(2) { Post.new }
+    session.batch do
+      for post in posts
+        session.index(post)
+      end
+    end
+    add = connection.adds.last
+    connection.adds.first.map { |add| add.field_by_name(:id).value }.should ==
+      posts.map { |post| "Post #{post.id}" }
+  end
+
+  it 'should not index changes to models that happen after index call' do
+    post = Post.new
+    session.batch do
+      session.index(post)
+      post.title = 'Title'
+    end
+    connection.adds.first.first.field_by_name(:title_ss).should be_nil
+  end
+
+  it 'should batch an add and a delete' do
+    pending 'batching all operations'
+    connection.should_not_receive(:add)
+    connection.should_not_receive(:remove)
+    posts = Array.new(2) { Post.new }
+    session.batch do
+      session.index(posts[0])
+      session.remove(posts[1])
+    end
+    connection.adds
+  end
+end

data/spec/api/indexer/dynamic_fields_spec.rb

@@ -0,0 +1,42 @@
+require File.join(File.dirname(__FILE__), 'spec_helper')
+
+describe 'indexing dynamic fields' do
+  it 'indexes string data' do
+    session.index(post(:custom_string => { :test => 'string' }))
+    connection.should have_add_with(:"custom_string:test_ss" => 'string')
+  end
+
+  it 'indexes integer data with virtual accessor' do
+    session.index(post(:category_ids => [1, 2]))
+    connection.should have_add_with(:"custom_integer:1_i" => '1', :"custom_integer:2_i" => '1')
+  end
+
+  it 'indexes float data' do
+    session.index(post(:custom_fl => { :test => 1.5 }))
+    connection.should have_add_with(:"custom_float:test_fm" => '1.5')
+  end
+
+  it 'indexes time data' do
+    session.index(post(:custom_time => { :test => Time.parse('2009-05-18 18:05:00 -0400') }))
+    connection.should have_add_with(:"custom_time:test_d" => '2009-05-18T22:05:00Z')
+  end
+
+  it 'indexes boolean data' do
+    session.index(post(:custom_boolean => { :test => false }))
+    connection.should have_add_with(:"custom_boolean:test_b" => 'false')
+  end
+
+  it 'indexes multiple values for a field' do
+    session.index(post(:custom_fl => { :test => [1.0, 2.1, 3.2] }))
+    connection.should have_add_with(:"custom_float:test_fm" => %w(1.0 2.1 3.2))
+  end
+
+  it 'should throw a NoMethodError if dynamic text field defined' do
+    lambda do
+      Sunspot.setup(Post) do
+        dynamic_text :custom_text
+      end
+    end.should raise_error(NoMethodError)
+  end
+end
+

data/spec/api/indexer/fixed_fields_spec.rb

@@ -0,0 +1,57 @@
+require File.join(File.dirname(__FILE__), 'spec_helper')
+
+describe 'indexing fixed fields', :type => :indexer do
+  it 'should index id' do
+    session.index post
+    connection.should have_add_with(:id => "Post #{post.id}")
+  end
+
+  it 'should index type' do
+    session.index post
+    connection.should have_add_with(:type => ['Post', 'SuperClass', 'MockRecord'])
+  end
+
+  it 'should index class name' do
+    session.index post
+    connection.should have_add_with(:class_name => 'Post')
+  end
+
+  it 'should index the array of objects supplied' do
+    posts = Array.new(2) { Post.new }
+    session.index posts
+    connection.should have_add_with(
+      { :id => "Post #{posts.first.id}" },
+      { :id => "Post #{posts.last.id}" }
+    )
+  end
+
+  it 'should index an array containing more than one type of object' do
+    post1, comment, post2 = objects = [Post.new, Namespaced::Comment.new, Post.new]
+    session.index objects
+    connection.should have_add_with(
+      { :id => "Post #{post1.id}", :type => ['Post', 'SuperClass', 'MockRecord'] },
+      { :id => "Namespaced::Comment #{comment.id}", :type => ['Namespaced::Comment', 'MockRecord'] },
+      { :id => "Post #{post2.id}", :type => ['Post', 'SuperClass', 'MockRecord'] }
+    )
+  end
+
+  it 'commits immediately after index! called' do
+    connection.should_receive(:add).ordered
+    connection.should_receive(:commit).ordered
+    session.index!(post)
+  end
+
+  it 'raises an ArgumentError if an attempt is made to index an object that has no configuration' do
+    lambda { session.index(Blog.new) }.should raise_error(Sunspot::NoSetupError)
+  end
+
+  it 'raises a NoAdapterError if class without adapter is indexed' do
+    lambda { session.index(User.new) }.should raise_error(Sunspot::NoAdapterError)
+  end
+
+  it 'raises an ArgumentError if a non-word character is included in the field name' do
+    lambda do
+      Sunspot.setup(Post) { string :"bad name" }
+    end.should raise_error(ArgumentError)
+  end
+end

data/spec/api/indexer/fulltext_spec.rb

@@ -0,0 +1,43 @@
+require File.join(File.dirname(__FILE__), 'spec_helper')
+
+describe 'indexing fulltext fields' do
+  it 'indexes text field' do
+    session.index(post(:title => 'A Title'))
+    connection.should have_add_with(:title_text => 'A Title')
+  end
+
+  it 'indexes stored text field' do
+    session.index(post(:body => 'Test body'))
+    connection.should have_add_with(:body_textsv => 'Test body')
+  end
+
+  it 'indexes text field with boost' do
+    session.index(post(:title => 'A Title'))
+    connection.adds.last.first.field_by_name(:title_text).attrs[:boost].should == 2
+  end
+
+  it 'indexes multiple values for a text field' do
+    session.index(post(:body => %w(some title)))
+    connection.should have_add_with(:body_textsv => %w(some title))
+  end
+
+  it 'indexes text via a block accessor' do
+    session.index(post(:title => 'backwards'))
+    connection.should have_add_with(:backwards_title_text => 'sdrawkcab')
+  end
+
+  it 'indexes document level boost using block' do
+    session.index(post(:ratings_average => 4.0))
+    connection.adds.last.first.attrs[:boost].should == 1.25
+  end
+
+  it 'indexes document level boost using attribute' do
+    session.index(Namespaced::Comment.new(:boost => 1.5))
+    connection.adds.last.first.attrs[:boost].should == 1.5
+  end
+
+  it 'indexes document level boost defined statically' do
+    session.index(Photo.new)
+    connection.adds.last.first.attrs[:boost].should == 0.75
+  end
+end

data/spec/api/indexer/removal_spec.rb

@@ -0,0 +1,53 @@
+require File.join(File.dirname(__FILE__), 'spec_helper')
+
+describe 'document removal', :type => :indexer do
+  it 'removes an object from the index' do
+    session.remove(post)
+    connection.should have_delete("Post #{post.id}")
+  end
+
+  it 'removes an object by type and id' do
+    session.remove_by_id(Post, 1)
+    connection.should have_delete('Post 1')
+  end
+
+  it 'removes an object by type and id and immediately commits' do
+    connection.should_receive(:delete_by_id).with(['Post 1']).ordered
+    connection.should_receive(:commit).ordered
+    session.remove_by_id!(Post, 1)
+  end
+
+  it 'removes an object from the index and immediately commits' do
+    connection.should_receive(:delete_by_id).ordered
+    connection.should_receive(:commit).ordered
+    session.remove!(post)
+  end
+
+  it 'removes everything from the index' do
+    session.remove_all
+    connection.should have_delete_by_query("*:*")
+  end
+
+  it 'removes everything from the index and immediately commits' do
+    connection.should_receive(:delete_by_query).ordered
+    connection.should_receive(:commit).ordered
+    session.remove_all!
+  end
+
+  it 'removes everything of a given class from the index' do
+    session.remove_all(Post)
+    connection.should have_delete_by_query("type:Post")
+  end
+
+  it 'correctly escapes namespaced classes when removing everything from the index' do
+    connection.should_receive(:delete_by_query).with('type:Namespaced\:\:Comment')
+    session.remove_all(Namespaced::Comment)
+  end
+
+  it 'should remove by query' do
+    session.remove(Post) do
+      with(:title, 'monkeys')
+    end
+    connection.should have_delete_by_query("(type:Post AND title_ss:monkeys)")
+  end
+end