freelancing-god-thinking-sphinx 0.9.5

data/spec/unit/thinking_sphinx/active_record/delta_spec.rb

@@ -0,0 +1,184 @@
+require 'spec/spec_helper'
+
+describe "ThinkingSphinx::ActiveRecord::Delta" do
+  describe "after_commit callback" do
+    before :each do
+      Person.stub_method(:write_inheritable_array => true)
+    end
+    
+    after :each do
+      Person.unstub_method(:write_inheritable_array)
+    end
+    
+    it "should add callbacks" do
+      Person.after_commit :toggle_delta
+      
+      Person.should have_received(:write_inheritable_array).with(
+        :after_commit, [:toggle_delta]
+      )
+    end
+  end
+  
+  describe "save_with_after_commit_callback method" do
+    before :each do
+      @person = Person.new
+      @person.stub_methods(
+        :save_without_after_commit_callback => true,
+        :callback                           => true
+      )
+    end
+    
+    it "should call the normal save method" do
+      @person.save
+      
+      @person.should have_received(:save_without_after_commit_callback)
+    end
+    
+    it "should call the callbacks if the save was successful" do
+      @person.save
+      
+      @person.should have_received(:callback).with(:after_commit)
+    end
+    
+    it "shouldn't call the callbacks if the save failed" do
+      @person.stub_method(:save_without_after_commit_callback => false)
+      
+      @person.save
+      
+      @person.should_not have_received(:callback)
+    end
+    
+    it "should return the normal save's result" do
+      @person.save.should be_true
+      
+      @person.stub_method(:save_without_after_commit_callback => false)
+      
+      @person.save.should be_false
+    end
+  end
+  
+  describe "save_with_after_commit_callback! method" do
+    before :each do
+      @person = Person.new
+      @person.stub_methods(
+        :save_without_after_commit_callback! => true,
+        :callback                            => true
+      )
+    end
+    
+    it "should call the normal save! method" do
+      @person.save!
+      
+      @person.should have_received(:save_without_after_commit_callback!)
+    end
+    
+    it "should call the callbacks if the save! was successful" do
+      @person.save!
+      
+      @person.should have_received(:callback).with(:after_commit)
+    end
+    
+    it "shouldn't call the callbacks if the save! failed" do
+      @person.stub_method(:save_without_after_commit_callback! => false)
+      
+      @person.save!
+      
+      @person.should_not have_received(:callback)
+    end
+    
+    it "should return the normal save's result" do
+      @person.save!.should be_true
+      
+      @person.stub_method(:save_without_after_commit_callback! => false)
+      
+      @person.save!.should be_false
+    end
+  end
+  
+  describe "destroy_with_after_commit_callback method" do
+    before :each do
+      @person = Person.new
+      @person.stub_methods(
+        :destroy_without_after_commit_callback  => true,
+        :callback                               => true
+      )
+    end
+    
+    it "should call the normal destroy method" do
+      @person.destroy
+      
+      @person.should have_received(:destroy_without_after_commit_callback)
+    end
+    
+    it "should call the callbacks if the destroy was successful" do
+      @person.destroy
+      
+      @person.should have_received(:callback).with(:after_commit)
+    end
+    
+    it "shouldn't call the callbacks if the destroy failed" do
+      @person.stub_method(:destroy_without_after_commit_callback => false)
+      
+      @person.destroy
+      
+      @person.should_not have_received(:callback)
+    end
+    
+    it "should return the normal save's result" do
+      @person.destroy.should be_true
+      
+      @person.stub_method(:destroy_without_after_commit_callback => false)
+      
+      @person.destroy.should be_false
+    end
+  end
+  
+  describe "toggle_delta method" do
+    it "should set the delta value to true" do
+      @person = Person.new
+      
+      @person.delta.should be_false
+      @person.send(:toggle_delta)
+      @person.delta.should be_true
+    end
+  end
+  
+  describe "index_delta method" do
+    before :each do
+      ThinkingSphinx::Configuration.stub_method(:environment => "spec")
+      ThinkingSphinx.stub_method(:deltas_enabled? => true)
+      
+      @person = Person.new
+      @person.stub_method(:system => true)
+    end
+    
+    after :each do
+      ThinkingSphinx::Configuration.unstub_method(:environment)
+      ThinkingSphinx.unstub_method(:deltas_enabled?)
+    end
+    
+    it "shouldn't index if delta indexing is disabled" do
+      ThinkingSphinx.stub_method(:deltas_enabled? => false)
+      
+      @person.send(:index_delta)
+      
+      @person.should_not have_received(:system)
+    end
+    
+    it "shouldn't index if the environment is 'test'" do
+      ThinkingSphinx::Configuration.stub_method(:environment => "test")
+      
+      @person.send(:index_delta)
+      
+      @person.should_not have_received(:system)
+    end
+    
+    it "should call indexer for the delta index" do
+      @person.send(:index_delta)
+      
+      @person.should have_received(:system).with(
+        "indexer --config #{ThinkingSphinx::Configuration.new.config_file} --rotate person_delta"
+      )
+    end
+  end
+end

data/spec/unit/thinking_sphinx/active_record/has_many_association_spec.rb

@@ -0,0 +1,53 @@
+require 'spec/spec_helper'
+
+describe 'ThinkingSphinx::ActiveRecord::HasManyAssociation' do
+  describe "search method" do
+    before :each do
+      Friendship.stub_method(:search => true)
+      
+      @person = Person.find(:first)
+      @index  = Friendship.indexes.first
+    end
+    
+    it "should raise an error if the required attribute doesn't exist" do
+      @index.stub_method(:attributes => [])
+      
+      lambda { @person.friendships.search "test" }.should raise_error(RuntimeError)
+      
+      @index.unstub_method(:attributes)
+    end
+    
+    it "should add a filter for the attribute into a normal search call" do
+      @person.friendships.search "test"
+      
+      Friendship.should have_received(:search).with(
+        "test", :with => {:person_id => @person.id}
+      )
+    end    
+  end
+  
+  describe "search method for has_many :through" do
+    before :each do
+      Person.stub_method(:search => true)
+      
+      @person = Person.find(:first)
+      @index  = Person.indexes.first
+    end
+    
+    it "should raise an error if the required attribute doesn't exist" do
+      @index.stub_method(:attributes => [])
+      
+      lambda { @person.friends.search "test" }.should raise_error(RuntimeError)
+      
+      @index.unstub_method(:attributes)
+    end
+    
+    it "should add a filter for the attribute into a normal search call" do
+      @person.friends.search "test"
+      
+      Person.should have_received(:search).with(
+        "test", :with => {:friendly_ids => @person.id}
+      )
+    end
+  end
+end

data/spec/unit/thinking_sphinx/active_record_spec.rb

@@ -0,0 +1,85 @@
+require 'spec/spec_helper'
+
+describe "ThinkingSphinx::ActiveRecord" do
+  describe "define_index method" do
+    before :each do
+      module TestModule
+        class TestModel < ActiveRecord::Base; end
+      end
+      
+      TestModule::TestModel.stub_methods(
+        :before_save  => true,
+        :after_commit => true
+      )
+      
+      @index = ThinkingSphinx::Index.stub_instance(:delta? => false)
+      ThinkingSphinx::Index.stub_method(:new => @index)
+    end
+    
+    after :each do
+      # Remove the class so we can redefine it
+      TestModule.send(:remove_const, :TestModel)
+    end
+    
+    it "should return nil and do nothing if indexes are disabled" do
+      ThinkingSphinx.stub_method(:define_indexes? => false)
+      
+      TestModule::TestModel.define_index {}.should be_nil
+      ThinkingSphinx::Index.should_not have_received(:new)
+      
+      ThinkingSphinx.unstub_method(:define_indexes?)
+    end
+    
+    it "should add a new index to the model" do
+      TestModule::TestModel.define_index do; end
+      
+      TestModule::TestModel.indexes.length.should == 1
+    end
+    
+    it "should add to ThinkingSphinx.indexed_models if the model doesn't already exist in the array" do
+      TestModule::TestModel.define_index do; end
+      
+      ThinkingSphinx.indexed_models.should include("TestModule::TestModel")
+    end
+    
+    it "shouldn't add to ThinkingSphinx.indexed_models if the model already exists in the array" do
+      TestModule::TestModel.define_index do; end
+      
+      ThinkingSphinx.indexed_models.select { |model|
+        model == "TestModule::TestModel"
+      }.length.should == 1
+      
+      TestModule::TestModel.define_index do; end
+      
+      ThinkingSphinx.indexed_models.select { |model|
+        model == "TestModule::TestModel"
+      }.length.should == 1
+    end
+    
+    it "should add before_save and after_commit hooks to the model if delta indexing is enabled" do
+      @index.stub_method(:delta? => true)
+      
+      TestModule::TestModel.define_index do; end
+      
+      TestModule::TestModel.should have_received(:before_save)
+      TestModule::TestModel.should have_received(:after_commit)
+    end
+    
+    it "should not add before_save and after_commit hooks to the model if delta indexing is disabled" do
+      TestModule::TestModel.define_index do; end
+      
+      TestModule::TestModel.should_not have_received(:before_save)
+      TestModule::TestModel.should_not have_received(:after_commit)
+    end
+    
+    it "should return the new index" do
+      TestModule::TestModel.define_index.should == @index
+    end
+  end
+  
+  describe "to_crc32 method" do
+    it "should return an integer" do
+      Person.to_crc32.should be_a_kind_of(Integer)
+    end
+  end
+end

data/spec/unit/thinking_sphinx/attribute_spec.rb

@@ -0,0 +1,73 @@
+require 'spec/spec_helper'
+
+describe ThinkingSphinx::Attribute do
+  describe "to_select_sql method" do
+    before :each do
+      @index = Person.indexes.first
+      @index.link!
+    end
+    
+    it "should concat with spaces if there's more than one non-integer column" do
+      @index.attributes[0].to_select_sql.should match(/CONCAT_WS\(' ', /)
+    end
+    
+    it "should concat with spaces if there's more than one association for a non-integer column" do
+      @index.attributes[1].to_select_sql.should match(/CONCAT_WS\(' ', /)
+    end
+    
+    it "should concat with commas if there's multiple integer columns" do
+      @index.attributes[2].to_select_sql.should match(/CONCAT_WS\(',', /)
+    end
+    
+    it "should concat with commas if there's more than one association for an integer column" do
+      @index.attributes[3].to_select_sql.should match(/CONCAT_WS\(',', /)
+    end
+    
+    it "should group with spaces if there's string columns from a has_many or has_and_belongs_to_many association" do
+      @index.attributes[4].to_select_sql.should match(/GROUP_CONCAT\(.+ SEPARATOR ' '\)/)
+    end
+    
+    it "should group with commas if there's integer columns from a has_many or has_and_belongs_to_many association" do
+      @index.attributes[5].to_select_sql.should match(/GROUP_CONCAT\(.+ SEPARATOR ','\)/)
+    end
+    
+    it "should convert datetime values to timestamps" do
+      @index.attributes[6].to_select_sql.should match(/UNIX_TIMESTAMP/)
+    end
+  end
+  
+  describe "to_group_sql method" do
+    before :each do
+      @attribute = ThinkingSphinx::Attribute.new([])
+      @attribute.stub_method(:is_many? => false, :is_string? => false)
+      
+      ThinkingSphinx.stub_method(:use_group_by_shortcut? => false)
+    end
+    
+    it "should return nil if is_many?" do
+      @attribute.stub_method(:is_many? => true)
+      
+      @attribute.to_group_sql.should be_nil
+    end
+    
+    it "should return nil if is_string?" do
+      @attribute.stub_method(:is_string? => true)
+      
+      @attribute.to_group_sql.should be_nil
+    end
+    
+    it "should return nil if group_by shortcut is allowed" do
+      ThinkingSphinx.stub_method(:use_group_by_shortcut? => true)
+      
+      @attribute.to_group_sql.should be_nil
+    end
+    
+    it "should return an array if neither is_many? or shortcut allowed" do
+      @attribute.to_group_sql.should be_a_kind_of(Array)
+    end
+    
+    after :each do
+      ThinkingSphinx.unstub_method(:use_group_by_shortcut?)
+    end
+  end
+end

data/spec/unit/thinking_sphinx/configuration_spec.rb

@@ -0,0 +1,7 @@
+require 'spec/spec_helper'
+
+describe ThinkingSphinx::Configuration do
+  describe "build method" do
+    
+  end
+end

data/spec/unit/thinking_sphinx/field_spec.rb

@@ -0,0 +1,51 @@
+require 'spec/spec_helper'
+
+describe ThinkingSphinx::Field do
+  describe "to_select_sql method" do
+    before :each do
+      @index = Person.indexes.first
+      @index.link!
+    end
+    
+    it "should concat with spaces if there are multiple columns" do
+      @index.fields.first.to_select_sql.should match(/CONCAT_WS\(' ', /)
+    end
+    
+    it "should concat with spaces if a column has more than one association" do
+      @index.fields[1].to_select_sql.should match(/CONCAT_WS\(' ', /)
+    end
+    
+    it "should group if any association for any column is a has_many or has_and_belongs_to_many" do
+      @index.fields[2].to_select_sql.should match(/GROUP_CONCAT/)
+    end
+  end
+  
+  describe "to_group_sql method" do
+    before :each do
+      @field = ThinkingSphinx::Field.new([])
+      @field.stub_methods(:is_many? => false)
+      
+      ThinkingSphinx.stub_method(:use_group_by_shortcut? => false)
+    end
+    
+    it "should return nil if is_many?" do
+      @field.stub_method(:is_many? => true)
+      
+      @field.to_group_sql.should be_nil
+    end
+    
+    it "should return nil if group_by shortcut is allowed" do
+      ThinkingSphinx.stub_method(:use_group_by_shortcut? => true)
+      
+      @field.to_group_sql.should be_nil
+    end
+    
+    it "should return an array if neither is_many? or shortcut allowed" do
+      @field.to_group_sql.should be_a_kind_of(Array)
+    end
+    
+    after :each do
+      ThinkingSphinx.unstub_method(:use_group_by_shortcut?)
+    end
+  end
+end