skalee-thinking-sphinx 1.3.14.1

data/spec/thinking_sphinx/index/builder_spec.rb

@@ -0,0 +1,479 @@
+require 'spec/spec_helper'
+
+describe ThinkingSphinx::Index::Builder do
+  describe ".generate without source scope" do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Person) do
+        indexes first_name, last_name
+        has birthday
+        has id, :as => :internal_id
+        
+        set_property :sql_range_step => 1000
+        
+        where "birthday <= NOW()"
+        group_by "first_name"
+      end
+      
+      @source = @index.sources.first
+    end
+    
+    it "should return an index" do
+      @index.should be_a_kind_of(ThinkingSphinx::Index)
+    end
+    
+    it "should have one source for the index" do
+      @index.sources.length.should == 1
+    end
+    
+    it "should have two fields" do
+      @source.fields.length.should == 2
+      @source.fields[0].unique_name.should == :first_name
+      @source.fields[1].unique_name.should == :last_name
+    end
+    
+    it "should have two attributes alongside the four internal ones" do
+      @source.attributes.length.should == 6
+      @source.attributes[4].unique_name.should == :birthday
+      @source.attributes[5].unique_name.should == :internal_id
+    end
+    
+    it "should have one condition" do
+      @source.conditions.length.should == 1
+      @source.conditions.first.should == "birthday <= NOW()"
+    end
+    
+    it "should have one grouping" do
+      @source.groupings.length.should == 1
+      @source.groupings.first.should == "first_name"
+    end
+    
+    it "should have one option" do
+      @source.options.length.should == 1
+      @source.options[:sql_range_step].should == 1000
+    end
+  end
+  
+  describe 'aliased field' do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Person) do
+        indexes first_name, :as => 'name'
+      end
+      
+      @source = @index.sources.first
+    end
+    
+    it "should store the alias as a symbol for consistency" do
+      @source.fields.last.unique_name.should == :name
+    end
+  end
+  
+  describe 'aliased attribute' do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Person) do
+        indexes first_name
+        has :id, :as => 'real_id'
+      end
+      
+      @source = @index.sources.first
+    end
+    
+    it "should store the alias as a symbol for consistency" do
+      @source.attributes.last.unique_name.should == :real_id
+    end
+  end
+  
+  describe "sortable field" do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Person) do
+        indexes first_name, :sortable => true
+      end
+      
+      @source = @index.sources.first
+    end
+    
+    it "should have one field" do
+      @source.fields.length.should == 1
+    end
+    
+    it "should have one attribute alongside the four internal ones" do
+      @source.attributes.length.should == 5
+    end
+    
+    it "should set the attribute name to have the _sort suffix" do
+      @source.attributes.last.unique_name.should == :first_name_sort
+    end
+    
+    it "should set the attribute column to be the same as the field" do
+      @source.attributes.last.columns.length.should == 1
+      @source.attributes.last.columns.first.__name.should == :first_name
+    end
+  end
+  
+  describe "faceted field" do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Person) do
+        indexes first_name, :facet => true
+      end
+      
+      @source = @index.sources.first
+    end
+    
+    after :each do
+      Person.sphinx_facets.delete_at(-1)
+    end
+    
+    it "should have one field" do
+      @source.fields.length.should == 1
+    end
+    
+    it "should have one attribute alongside the four internal ones" do
+      @source.attributes.length.should == 5
+    end
+    
+    it "should set the attribute name to have the _facet suffix" do
+      @source.attributes.last.unique_name.should == :first_name_facet
+    end
+    
+    it "should set the attribute type to integer" do
+      @source.attributes.last.type.should == :integer
+    end
+    
+    it "should set the attribute column to be the same as the field" do
+      @source.attributes.last.columns.length.should == 1
+      @source.attributes.last.columns.first.__name.should == :first_name
+    end
+  end
+  
+  describe "faceted integer attribute" do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Alpha) do
+        indexes :name
+        has value, :facet => true
+      end
+      
+      @source = @index.sources.first
+    end
+    
+    after :each do
+      Alpha.sphinx_facets.delete_at(-1)
+    end
+    
+    it "should have just one attribute alongside the four internal ones" do
+      @source.attributes.length.should == 5
+    end
+  end
+  
+  describe "faceted timestamp attribute" do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Person) do
+        indexes first_name
+        has birthday, :facet => true
+      end
+      
+      @source = @index.sources.first
+    end
+    
+    after :each do
+      Person.sphinx_facets.delete_at(-1)
+    end
+    
+    it "should have just one attribute alongside the four internal ones" do
+      @source.attributes.length.should == 5
+    end
+  end
+  
+  describe "faceted boolean attribute" do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Beta) do
+        indexes :name
+        has delta, :facet => true
+      end
+      
+      @source = @index.sources.first
+    end
+    
+    after :each do
+      Beta.sphinx_facets.delete_at(-1)
+    end
+    
+    it "should have just one attribute alongside the four internal ones" do
+      @source.attributes.length.should == 5
+    end
+  end
+  
+  describe "faceted float attribute" do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Alpha) do
+        indexes :name
+        has cost, :facet => true
+      end
+      
+      @source = @index.sources.first
+    end
+    
+    after :each do
+      Alpha.sphinx_facets.delete_at(-1)
+    end
+    
+    it "should have just one attribute alongside the four internal ones" do
+      @source.attributes.length.should == 5
+    end
+  end
+  
+  describe "faceted string attribute" do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Person) do
+        indexes first_name
+        has last_name, :facet => true
+      end
+      
+      @source = @index.sources.first
+    end
+    
+    after :each do
+      Person.sphinx_facets.delete_at(-1)
+    end
+    
+    it "should have two attributes alongside the four internal ones" do
+      @source.attributes.length.should == 6
+    end
+    
+    it "should set the facet attribute name to have the _facet suffix" do
+      @source.attributes.last.unique_name.should == :last_name_facet
+    end
+    
+    it "should set the attribute type to integer" do
+      @source.attributes.last.type.should == :integer
+    end
+    
+    it "should set the attribute column to be the same as the field" do
+      @source.attributes.last.columns.length.should == 1
+      @source.attributes.last.columns.first.__name.should == :last_name
+    end
+  end
+  
+  describe 'faceted manual MVA' do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Person) do
+        indexes first_name
+        has 'SQL STATEMENT', :type => :multi, :as => :sql, :facet => true
+      end
+      
+      @source = @index.sources.first
+    end
+    
+    after :each do
+      Person.sphinx_facets.delete_at(-1)
+    end
+    
+    it "should have two attributes alongside the four internal ones" do
+      @source.attributes.length.should == 6
+    end
+    
+    it "should set the facet attribute name to have the _facet suffix" do
+      @source.attributes.last.unique_name.should == :sql_facet
+    end
+    
+    it "should keep the original attribute's name set as requested" do
+      @source.attributes[-2].unique_name.should == :sql
+    end
+    
+    it "should set the attribute type to multi" do
+      @source.attributes.last.type.should == :multi
+    end
+    
+    it "should set the attribute column to be the same as the field" do
+      @source.attributes.last.columns.length.should == 1
+      @source.attributes.last.columns.first.__name.should == 'SQL STATEMENT'
+    end
+  end
+  
+  describe 'faceted MVA field' do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Person) do
+        indexes tags(:name), :as => :tags, :facet => true
+      end
+      
+      @source = @index.sources.first
+    end
+    
+    after :each do
+      Person.sphinx_facets.delete_at(-1)
+    end
+    
+    it "should have one field" do
+      @source.fields.length.should == 1
+    end
+    
+    it "should have one attribute alongside the four internal ones" do
+      @source.attributes.length.should == 5
+    end
+    
+    it "should set the attribute name to have the _facet suffix" do
+      @source.attributes.last.unique_name.should == :tags_facet
+    end
+    
+    it "should set the attribute type to multi" do
+      @source.attributes.last.type.should == :multi
+    end
+    
+    it "should set the attribute column to be the same as the field" do
+      @source.attributes.last.columns.length.should == 1
+      @source.attributes.last.columns.first.__name.should == :name
+    end
+  end
+  
+  describe "no fields" do
+    it "should raise an exception" do
+      lambda {
+        ThinkingSphinx::Index::Builder.generate(Person) do
+          #
+        end
+      }.should raise_error
+    end
+  end
+  
+  describe "explicit source" do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Person) do
+        define_source do
+          indexes first_name, last_name
+          has birthday
+          has id, :as => :internal_id
+        
+          set_property :delta => true
+        
+          where "birthday <= NOW()"
+          group_by "first_name"
+        end
+      end
+      
+      @source = @index.sources.first
+    end
+    
+    it "should return an index" do
+      @index.should be_a_kind_of(ThinkingSphinx::Index)
+    end
+    
+    it "should have one source for the index" do
+      @index.sources.length.should == 1
+    end
+    
+    it "should have two fields" do
+      @source.fields.length.should == 2
+      @source.fields[0].unique_name.should == :first_name
+      @source.fields[1].unique_name.should == :last_name
+    end
+    
+    it "should have two attributes alongside the four internal ones" do
+      @source.attributes.length.should == 6
+      @source.attributes[4].unique_name.should == :birthday
+      @source.attributes[5].unique_name.should == :internal_id
+    end
+  end
+  
+  describe "multiple sources" do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Person) do
+        define_source do
+          indexes first_name
+          has birthday
+        end
+        
+        define_source do
+          indexes last_name
+          has :id, :as => :internal_id
+        end
+      end
+    end
+    
+    it "should have two sources" do
+      @index.sources.length.should == 2
+    end
+    
+    it "should have two fields" do
+      @index.fields.length.should == 2
+    end
+    
+    it "should have one field in each source" do
+      @index.sources.each do |source|
+        source.fields.length.should == 1
+      end
+    end
+    
+    it "should have two attributes alongside the eight internal ones" do
+      @index.attributes.length.should == 10
+    end
+    
+    it "should have one attribute in each source alongside the four internal ones" do
+      @index.sources.each do |source|
+        source.attributes.length.should == 5
+      end
+    end
+  end
+  
+  describe "index options" do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Person) do
+        indexes first_name
+        
+        set_property :charset_type => "utf16"
+        set_property :group_concat_max_len => 1024
+      end
+    end
+    
+    it "should store the index setting for the index" do
+      @index.local_options[:charset_type].should == "utf16"
+    end
+    
+    it "should store non-Sphinx settings for the index" do
+      @index.local_options[:group_concat_max_len].should == 1024
+    end
+  end
+  
+  describe "delta options" do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Person) do
+        indexes first_name
+        
+        set_property :delta => true
+      end
+    end
+    
+    it "should not keep the delta setting in source options" do
+      @index.sources.first.options.should be_empty
+    end
+    
+    it "should not keep the delta setting in index options" do
+      @index.local_options.should be_empty
+    end
+    
+    it "should set the index delta object set" do
+      @index.delta_object.should be_a_kind_of(ThinkingSphinx::Deltas::DefaultDelta)
+    end
+  end
+  
+  context 'index options' do
+    before :each do
+      @index = ThinkingSphinx::Index::Builder.generate(Person) do
+        indexes first_name
+        
+        set_property :index_exact_words => true
+      end
+    end
+    
+    it "should track the index_exact_words option to the index" do
+      @index.local_options[:index_exact_words].should be_true
+    end
+  end
+  
+  context 'with an explicit name' do
+    it "should set the index's name using the provided value" do
+      index = ThinkingSphinx::Index::Builder.generate(Person, 'custom') do
+        indexes first_name
+      end
+      
+      index.name.should == 'custom'
+    end
+  end
+end

data/spec/thinking_sphinx/index/faux_column_spec.rb

@@ -0,0 +1,30 @@
+require 'spec/spec_helper'
+
+describe ThinkingSphinx::Index::FauxColumn do  
+  describe "coerce class method" do
+    before :each do
+      @column = stub('column')
+      ThinkingSphinx::Index::FauxColumn.stub!(:new => @column)
+    end
+    
+    it "should return a single faux column if passed a string" do
+      ThinkingSphinx::Index::FauxColumn.coerce("string").should == @column
+    end
+    
+    it "should return a single faux column if passed a symbol" do
+      ThinkingSphinx::Index::FauxColumn.coerce(:string).should == @column
+    end
+    
+    it "should return an array of faux columns if passed an array of strings" do
+      ThinkingSphinx::Index::FauxColumn.coerce(["one", "two"]).should == [
+        @column, @column
+      ]
+    end
+    
+    it "should return an array of faux columns if passed an array of symbols" do
+      ThinkingSphinx::Index::FauxColumn.coerce([:one, :two]).should == [
+        @column, @column
+      ]
+    end
+  end
+end

data/spec/thinking_sphinx/index_spec.rb

@@ -0,0 +1,183 @@
+require 'spec/spec_helper'
+
+describe ThinkingSphinx::Index do
+  describe "prefix_fields method" do
+    before :each do
+      @index = ThinkingSphinx::Index.new(Person)
+      
+      @field_a = stub('field', :prefixes => true)
+      @field_b = stub('field', :prefixes => false)
+      @field_c = stub('field', :prefixes => true)
+      
+      @index.stub!(:fields => [@field_a, @field_b, @field_c])
+    end
+    
+    it "should return fields that are flagged as prefixed" do
+      @index.prefix_fields.should include(@field_a)
+      @index.prefix_fields.should include(@field_c)
+    end
+    
+    it "should not return fields that aren't flagged as prefixed" do
+      @index.prefix_fields.should_not include(@field_b)
+    end
+  end
+  
+  describe "infix_fields method" do
+    before :each do
+      @index = ThinkingSphinx::Index.new(Person)
+      
+      @field_a = stub('field', :infixes => true)
+      @field_b = stub('field', :infixes => false)
+      @field_c = stub('field', :infixes => true)
+      
+      @index.stub!(:fields => [@field_a, @field_b, @field_c])
+    end
+    
+    it "should return fields that are flagged as infixed" do
+      @index.infix_fields.should include(@field_a)
+      @index.infix_fields.should include(@field_c)
+    end
+    
+    it "should not return fields that aren't flagged as infixed" do
+      @index.infix_fields.should_not include(@field_b)
+    end
+  end
+  
+  describe '.name_for' do
+    it "should return the model's name downcased" do
+      ThinkingSphinx::Index.name_for(Alpha).should == 'alpha'
+    end
+    
+    it "should separate words by underscores" do
+      ThinkingSphinx::Index.name_for(ActiveRecord).should == 'active_record'
+    end
+    
+    it "should separate namespaces by underscores" do
+      ThinkingSphinx::Index.name_for(ActiveRecord::Base).
+        should == 'active_record_base'
+    end
+  end
+  
+  describe '#name' do
+    it "should return the downcased name of the index's model" do
+      ThinkingSphinx::Index.new(Alpha).name.should == 'alpha'
+    end
+    
+    it "should return a custom name if one is set" do
+      index = ThinkingSphinx::Index.new(Alpha)
+      index.name = 'custom'
+      index.name.should == 'custom'
+    end
+  end
+  
+  describe '#core_name' do
+    it "should take the index's name and append _core" do
+      ThinkingSphinx::Index.new(Alpha).core_name.should == 'alpha_core'
+    end
+  end
+  
+  describe '#delta_name' do
+    it "should take the index's name and append _delta" do
+      ThinkingSphinx::Index.new(Alpha).delta_name.should == 'alpha_delta'
+    end
+  end
+  
+  describe '#all_names' do
+    it "should return the core index name by default" do
+      ThinkingSphinx::Index.new(Alpha).all_names.should == ['alpha_core']
+    end
+    
+    it "should return both core and delta names if deltas are enabled" do
+      index = ThinkingSphinx::Index.new(Alpha)
+      index.delta_object = stub('delta')
+      
+      index.all_names.should == ['alpha_core', 'alpha_delta']
+    end
+    
+    it "should respect custom names" do
+      index = ThinkingSphinx::Index.new(Alpha)
+      index.name = 'custom'
+      
+      index.all_names.should == ['custom_core']
+    end
+    
+    it "should respect custom names when deltas are enabled" do
+      index = ThinkingSphinx::Index.new(Alpha)
+      index.name = 'custom'
+      index.delta_object = stub('delta')
+      
+      index.all_names.should == ['custom_core', 'custom_delta']
+    end
+  end
+  
+  describe '#to_riddle' do
+    it "should return two Riddle indexes if deltas are disabled" do
+      index = ThinkingSphinx::Index.new(Alpha)
+      
+      index.to_riddle(0).length.should == 2
+    end
+    
+    it "should return three Riddle indexes if deltas are enabled" do
+      index = ThinkingSphinx::Index.new(Beta)
+      index.delta_object = stub('delta')
+      
+      index.to_riddle(0).length.should == 3
+    end
+    
+    it "should include a distributed index" do
+      index = ThinkingSphinx::Index.new(Alpha)
+      
+      index.to_riddle(0).last.
+        should be_a(Riddle::Configuration::DistributedIndex)
+    end
+    
+    context 'core index' do
+      it "should use the core name" do
+        @index = ThinkingSphinx::Index.new(Alpha).to_riddle(0).first
+        @index.name.should == 'alpha_core'
+      end
+      
+      it "should not try to set disable_range on the index" do
+        ThinkingSphinx::Configuration.instance.
+          index_options[:disable_range] = true
+        
+        lambda {
+          @index = ThinkingSphinx::Index.new(Alpha).to_riddle(0).first
+        }.should_not raise_error(NoMethodError)
+      end
+    end
+    
+    context 'delta index' do
+      before :each do
+        index = ThinkingSphinx::Index.new(Beta)
+        index.delta_object = stub('delta')
+        @index = index.to_riddle(0)[1]
+      end
+      
+      it "should use the delta name" do
+        @index.name.should == 'beta_delta'
+      end
+    end
+    
+    context 'distributed index' do
+      it "should use the index's name" do
+        index = ThinkingSphinx::Index.new(Alpha)
+
+        index.to_riddle(0).last.name.should == 'alpha'
+      end
+      
+      it "should add the core index" do
+        index = ThinkingSphinx::Index.new(Alpha)
+
+        index.to_riddle(0).last.local_indexes.should include('alpha_core')
+      end
+      
+      it "should add the delta index if there is one" do
+        index = ThinkingSphinx::Index.new(Beta)
+        index.delta_object = stub('delta')
+
+        index.to_riddle(0).last.local_indexes.should include('beta_delta')
+      end
+    end
+  end
+end