DrMark-thinking-sphinx 0.9.7

data/spec/unit/thinking_sphinx/search_spec.rb

@@ -0,0 +1,163 @@
+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
+end

data/spec/unit/thinking_sphinx_spec.rb

@@ -0,0 +1,107 @@
+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?.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
+  
+  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
+        @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 File.join(File.dirname(__FILE__), 'thinking_sphinx_tasks')

data/tasks/thinking_sphinx_tasks.rb

@@ -0,0 +1,86 @@
+require 'fileutils'
+
+namespace :thinking_sphinx do
+  task :app_env do
+    Rake::Task[:environment].invoke if defined?(RAILS_ROOT)
+    Rake::Task[:merb_env].invoke    if defined?(Merb)
+  end
+  
+  desc "Start a Sphinx searchd daemon using Thinking Sphinx's settings"
+  task :start => :app_env do
+    config = ThinkingSphinx::Configuration.new
+    
+    FileUtils.mkdir_p config.searchd_file_path
+    raise RuntimeError, "searchd is already running." if sphinx_running?
+    
+    Dir["#{config.searchd_file_path}/*.spl"].each { |file| File.delete(file) }
+    
+    cmd = "searchd --config #{config.config_file}"
+    puts cmd
+    system cmd
+    
+    sleep(2)
+    
+    if sphinx_running?
+      puts "Started successfully (pid #{sphinx_pid})."
+    else
+      puts "Failed to start searchd daemon. Check #{config.searchd_log_file}."
+    end
+  end
+  
+  desc "Stop Sphinx using Thinking Sphinx's settings"
+  task :stop => :app_env do
+    raise RuntimeError, "searchd is not running." unless sphinx_running?
+    pid = sphinx_pid
+    system "kill #{pid}"
+    puts "Stopped search daemon (pid #{pid})."
+  end
+  
+  desc "Restart Sphinx"
+  task :restart => [:app_env, :stop, :start]
+  
+  desc "Generate the Sphinx configuration file using Thinking Sphinx's settings"
+  task :configure => :app_env do
+    ThinkingSphinx::Configuration.new.build
+  end
+  
+  desc "Index data for Sphinx using Thinking Sphinx's settings"
+  task :index => [:app_env, :configure] do
+    config = ThinkingSphinx::Configuration.new
+    
+    FileUtils.mkdir_p config.searchd_file_path
+    cmd = "indexer --config #{config.config_file} --all"
+    cmd << " --rotate" if sphinx_running?
+    puts cmd
+    system cmd
+  end
+end
+
+namespace :ts do
+  desc "Start a Sphinx searchd daemon using Thinking Sphinx's settings"
+  task :start   => "thinking_sphinx:start"
+  desc "Stop Sphinx using Thinking Sphinx's settings"
+  task :stop    => "thinking_sphinx:stop"
+  desc "Index data for Sphinx using Thinking Sphinx's settings"
+  task :in      => "thinking_sphinx:index"
+  desc "Index data for Sphinx using Thinking Sphinx's settings"
+  task :index   => "thinking_sphinx:index"
+  desc "Restart Sphinx"
+  task :restart => "thinking_sphinx:restart"
+  desc "Generate the Sphinx configuration file using Thinking Sphinx's settings"
+  task :config  => "thinking_sphinx:configure"
+end
+
+def sphinx_pid
+  config = ThinkingSphinx::Configuration.new
+  
+  if File.exists?(config.pid_file)
+    `cat #{config.pid_file}`[/\d+/]
+  else
+    nil
+  end
+end
+
+def sphinx_running?
+  sphinx_pid && `ps -p #{sphinx_pid} | wc -l`.to_i > 1
+end

metadata

@@ -0,0 +1,90 @@
+--- !ruby/object:Gem::Specification 
+name: DrMark-thinking-sphinx
+version: !ruby/object:Gem::Version 
+  version: 0.9.7
+platform: ruby
+authors: 
+- Pat Allan
+autorequire: 
+bindir: bin
+cert_chain: []
+
+date: 2008-06-25 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.rb
+- LICENCE
+- README
+- tasks/thinking_sphinx_tasks.rb
+- tasks/thinking_sphinx_tasks.rake
+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