ryanb-thinking_sphinx 0.9.8

data/spec/unit/thinking_sphinx/search_spec.rb

@@ -0,0 +1,190 @@
+require 'spec/spec_helper'
+
+describe ThinkingSphinx::Search do
+  describe "search_for_id method" do
+    before :each do
+      @client = Riddle::Client.stub_instance(
+        :filters    => [],
+        :filters=   => true,
+        :id_range=  => true,
+        :query      => {
+          :matches  => []
+        }
+      )
+      
+      ThinkingSphinx::Search.stub_methods(
+        :client_from_options => @client,
+        :search_conditions   => ["", []]
+      )
+    end
+    
+    it "should set the client id range to focus on the given id" do
+      ThinkingSphinx::Search.search_for_id 42, "an_index"
+      
+      @client.should have_received(:id_range=).with(42..42)
+    end
+    
+    it "should query on the given index" do
+      ThinkingSphinx::Search.search_for_id 42, "an_index"
+      
+      @client.should have_received(:query).with("", "an_index")
+    end
+    
+    it "should return true if a record is returned" do
+      @client.stub_method(:query => {
+        :matches => [24]
+      })
+      
+      ThinkingSphinx::Search.search_for_id(42, "an_index").should be_true
+    end
+    
+    it "should return false if no records are returned" do
+      ThinkingSphinx::Search.search_for_id(42, "an_index").should be_false
+    end
+  end
+  
+  describe "instance_from_result method" do
+    before :each do
+      Person.stub_method(:find => true)
+      ThinkingSphinx::Search.stub_method(:class_from_crc => Person)
+    end
+    
+    after :each do
+      Person.unstub_method(:find)
+      ThinkingSphinx::Search.unstub_method(:class_from_crc)
+    end
+
+    it "should honour the :include option" do
+      ThinkingSphinx::Search.send(
+        :instance_from_result,
+        {:doc => 1, :attributes => {"class_crc" => 123}},
+        {:include => :assoc}
+      )
+
+      Person.should have_received(:find).with(1, :include => :assoc, :select => nil)
+    end
+
+    it "should honour the :select option" do
+      ThinkingSphinx::Search.send(
+        :instance_from_result,
+        {:doc => 1, :attributes => {"class_crc" => 123}},
+        {:select => :columns}
+      )
+
+      Person.should have_received(:find).with(1, :include => nil, :select => :columns)
+    end
+
+  end
+
+  describe "instances_from_results method" do
+    before :each do
+      @person_a = Person.stub_instance
+      @person_b = Person.stub_instance
+      @person_c = Person.stub_instance
+
+      @results = [
+        {:doc => @person_a.id},
+        {:doc => @person_b.id},
+        {:doc => @person_c.id}
+      ]
+      
+      Person.stub_method(
+        :find => [@person_c, @person_a, @person_b]
+      )
+      ThinkingSphinx::Search.stub_method(:instance_from_result => true)
+    end
+
+    after :each do
+      Person.unstub_method(:find)
+      ThinkingSphinx::Search.unstub_method(:instance_from_result)
+    end
+
+    it "should pass calls to instance_from_result if no class given" do
+      ThinkingSphinx::Search.send(
+        :instances_from_results, @results
+      )
+
+      ThinkingSphinx::Search.should have_received(:instance_from_result).with(
+        {:doc => @person_a.id}, {}
+      )
+      ThinkingSphinx::Search.should have_received(:instance_from_result).with(
+        {:doc => @person_b.id}, {}
+      )
+      ThinkingSphinx::Search.should have_received(:instance_from_result).with(
+        {:doc => @person_c.id}, {}
+      )
+    end
+
+    it "should call a find on all ids for the class" do
+      ThinkingSphinx::Search.send(
+        :instances_from_results, @results, {}, Person
+      )
+      
+      Person.should have_received(:find).with(
+        :all,
+        :conditions => {:id => [@person_a.id, @person_b.id, @person_c.id]},
+        :include    => nil,
+        :select     => nil
+      )
+    end
+
+    it "should honour the :include option" do
+      ThinkingSphinx::Search.send(
+        :instances_from_results, @results, {:include => :something}, Person
+      )
+      
+      Person.should have_received(:find).with(
+        :all,
+        :conditions => {:id => [@person_a.id, @person_b.id, @person_c.id]},
+        :include    => :something,
+        :select     => nil
+      )
+    end
+
+    it "should honour the :select option" do
+      ThinkingSphinx::Search.send(
+        :instances_from_results, @results, {:select => :fields}, Person
+      )
+      
+      Person.should have_received(:find).with(
+        :all,
+        :conditions => {:id => [@person_a.id, @person_b.id, @person_c.id]},
+        :include    => nil,
+        :select     => :fields
+      )
+    end
+
+    it "should sort the objects the same as the result set" do
+      ThinkingSphinx::Search.send(
+        :instances_from_results, @results, {:select => :fields}, Person
+      ).should == [@person_a, @person_b, @person_c]
+    end
+  end
+
+  describe "count method" do
+    before :each do
+      @client = Riddle::Client.stub_instance(
+        :filters    => [],
+        :filters=   => true,
+        :id_range=  => true,
+        :sort_mode  => :asc,
+        :limit      => 5,
+        :offset=    => 0,
+        :sort_mode= => true,
+        :query      => {
+          :matches  => [],
+          :total    => 50
+        }
+      )
+
+      ThinkingSphinx::Search.stub_methods(
+        :client_from_options => @client,
+        :search_conditions   => ["", []]
+      )
+    end
+
+    it "should return query total" do
+      ThinkingSphinx::Search.count(42, "an_index").should == 50
+    end
+  end
+end

data/spec/unit/thinking_sphinx_spec.rb

@@ -0,0 +1,129 @@
+require 'spec/spec_helper'
+
+describe ThinkingSphinx do
+  describe "indexed_models methods" do
+    it "should contain all the names of models that have indexes" do
+      ThinkingSphinx.indexed_models.should include("Person")
+    end
+  end
+  
+  it "should define indexes by default" do
+    ThinkingSphinx.define_indexes?.should be_true
+  end
+  
+  it "should disable index definition" do
+    ThinkingSphinx.define_indexes = false
+    ThinkingSphinx.define_indexes?.should be_false
+  end
+  
+  it "should enable index definition" do
+    ThinkingSphinx.define_indexes = false
+    ThinkingSphinx.define_indexes?.should be_false
+    ThinkingSphinx.define_indexes = true
+    ThinkingSphinx.define_indexes?.should be_true
+  end
+  
+  it "should index deltas by default" do
+    ThinkingSphinx.deltas_enabled = nil
+    ThinkingSphinx.deltas_enabled?.should be_true
+  end
+  
+  it "should disable delta indexing" do
+    ThinkingSphinx.deltas_enabled = false
+    ThinkingSphinx.deltas_enabled?.should be_false
+  end
+  
+  it "should enable delta indexing" do
+    ThinkingSphinx.deltas_enabled = false
+    ThinkingSphinx.deltas_enabled?.should be_false
+    ThinkingSphinx.deltas_enabled = true
+    ThinkingSphinx.deltas_enabled?.should be_true
+  end
+  
+  it "should update indexes by default" do
+    ThinkingSphinx.updates_enabled = nil
+    ThinkingSphinx.updates_enabled?.should be_true
+  end
+  
+  it "should disable index updating" do
+    ThinkingSphinx.updates_enabled = false
+    ThinkingSphinx.updates_enabled?.should be_false
+  end
+  
+  it "should enable index updating" do
+    ThinkingSphinx.updates_enabled = false
+    ThinkingSphinx.updates_enabled?.should be_false
+    ThinkingSphinx.updates_enabled = true
+    ThinkingSphinx.updates_enabled?.should be_true
+  end
+  
+  describe "use_group_by_shortcut? method" do
+    after :each do
+      ::ActiveRecord::Base.connection.unstub_method(:select_all)
+    end
+    
+    it "should return true if no ONLY_FULL_GROUP_BY" do
+      ::ActiveRecord::Base.connection.stub_method(
+        :select_all => {:a => "OTHER SETTINGS"}
+      )
+      
+      ThinkingSphinx.use_group_by_shortcut?.should be_true
+    end
+  
+    it "should return true if NULL value" do
+      ::ActiveRecord::Base.connection.stub_method(
+        :select_all => {:a => nil}
+      )
+      
+      ThinkingSphinx.use_group_by_shortcut?.should be_true
+    end
+  
+    it "should return false if ONLY_FULL_GROUP_BY is set" do
+      ::ActiveRecord::Base.connection.stub_method(
+        :select_all => {:a => "OTHER SETTINGS,ONLY_FULL_GROUP_BY,blah"}
+      )
+      
+      ThinkingSphinx.use_group_by_shortcut?.should be_false
+    end
+    
+    it "should return false if ONLY_FULL_GROUP_BY is set in any of the values" do
+      ::ActiveRecord::Base.connection.stub_method(
+        :select_all => {
+          :a => "OTHER SETTINGS",
+          :b => "ONLY_FULL_GROUP_BY"
+        }
+      )
+      
+      ThinkingSphinx.use_group_by_shortcut?.should be_false
+    end
+    
+    describe "if not using MySQL" do
+      before :each do
+        unless ::ActiveRecord::ConnectionAdapters.const_defined?(:PostgreSQLAdapter)
+          pending "No PostgreSQL"
+          return
+        end
+        @connection = ::ActiveRecord::ConnectionAdapters::PostgreSQLAdapter.stub_instance(
+          :select_all => true
+        )
+        ::ActiveRecord::Base.stub_method(
+          :connection => @connection
+        )
+      end
+      
+      after :each do
+        ::ActiveRecord::Base.unstub_method(:connection)
+      end
+    
+      it "should return false" do
+        ThinkingSphinx.use_group_by_shortcut?.should be_false
+      end
+    
+      it "should not call select_all" do
+        ThinkingSphinx.use_group_by_shortcut?
+        
+        @connection.should_not have_received(:select_all)
+      end
+    end
+  end
+end

data/tasks/thinking_sphinx_tasks.rake

@@ -0,0 +1 @@
+require 'thinking_sphinx/tasks'

metadata

@@ -0,0 +1,103 @@
+--- !ruby/object:Gem::Specification 
+name: ryanb-thinking_sphinx
+version: !ruby/object:Gem::Version 
+  version: 0.9.8
+platform: ruby
+authors: 
+- Pat Allan
+autorequire: 
+bindir: bin
+cert_chain: []
+
+date: 2008-08-18 00:00:00 -07:00
+default_executable: 
+dependencies: []
+
+description: A concise and easy-to-use Ruby library that connects ActiveRecord to the Sphinx search daemon, managing configuration, indexing and searching.
+email: pat@freelancing-gods.com
+executables: []
+
+extensions: []
+
+extra_rdoc_files: []
+
+files: 
+- lib/riddle/client/filter.rb
+- lib/riddle/client/message.rb
+- lib/riddle/client/response.rb
+- lib/riddle/client.rb
+- lib/riddle.rb
+- lib/test.rb
+- lib/thinking_sphinx/active_record/delta.rb
+- lib/thinking_sphinx/active_record/has_many_association.rb
+- lib/thinking_sphinx/active_record/search.rb
+- lib/thinking_sphinx/active_record.rb
+- lib/thinking_sphinx/association.rb
+- lib/thinking_sphinx/attribute.rb
+- lib/thinking_sphinx/configuration.rb
+- lib/thinking_sphinx/field.rb
+- lib/thinking_sphinx/index/builder.rb
+- lib/thinking_sphinx/index/faux_column.rb
+- lib/thinking_sphinx/index.rb
+- lib/thinking_sphinx/rails_additions.rb
+- lib/thinking_sphinx/search.rb
+- lib/thinking_sphinx/tasks.rb
+- lib/thinking_sphinx.rb
+- LICENCE
+- README
+- tasks/thinking_sphinx_tasks.rake
+- spec/unit/thinking_sphinx/active_record/delta_spec.rb
+- spec/unit/thinking_sphinx/active_record/has_many_association_spec.rb
+- spec/unit/thinking_sphinx/active_record/search_spec.rb
+- spec/unit/thinking_sphinx/active_record_spec.rb
+- spec/unit/thinking_sphinx/association_spec.rb
+- spec/unit/thinking_sphinx/attribute_spec.rb
+- spec/unit/thinking_sphinx/configuration_spec.rb
+- spec/unit/thinking_sphinx/field_spec.rb
+- spec/unit/thinking_sphinx/index/builder_spec.rb
+- spec/unit/thinking_sphinx/index/faux_column_spec.rb
+- spec/unit/thinking_sphinx/index_spec.rb
+- spec/unit/thinking_sphinx/search_spec.rb
+- spec/unit/thinking_sphinx_spec.rb
+has_rdoc: true
+homepage: http://ts.freelancing-gods.com
+post_install_message: 
+rdoc_options: 
+- --title
+- Thinking Sphinx -- Rails/Merb Sphinx Plugin
+- --line-numbers
+require_paths: 
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      version: "0"
+  version: 
+required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      version: "0"
+  version: 
+requirements: []
+
+rubyforge_project: thinking-sphinx
+rubygems_version: 1.2.0
+signing_key: 
+specification_version: 2
+summary: A concise and easy-to-use Ruby library that connects ActiveRecord to the Sphinx search daemon, managing configuration, indexing and searching.
+test_files: 
+- spec/unit/thinking_sphinx/active_record/delta_spec.rb
+- spec/unit/thinking_sphinx/active_record/has_many_association_spec.rb
+- spec/unit/thinking_sphinx/active_record/search_spec.rb
+- spec/unit/thinking_sphinx/active_record_spec.rb
+- spec/unit/thinking_sphinx/association_spec.rb
+- spec/unit/thinking_sphinx/attribute_spec.rb
+- spec/unit/thinking_sphinx/configuration_spec.rb
+- spec/unit/thinking_sphinx/field_spec.rb
+- spec/unit/thinking_sphinx/index/builder_spec.rb
+- spec/unit/thinking_sphinx/index/faux_column_spec.rb
+- spec/unit/thinking_sphinx/index_spec.rb
+- spec/unit/thinking_sphinx/search_spec.rb
+- spec/unit/thinking_sphinx_spec.rb