thinking-sphinx 3.0.6 → 3.1.0

data/spec/acceptance/indexing_spec.rb

@@ -0,0 +1,27 @@
+require 'acceptance/spec_helper'
+
+describe 'Indexing', :live => true do
+  it "does not index files where the temp file exists" do
+    path = Rails.root.join('db/sphinx/test/ts-article_core.tmp')
+    FileUtils.mkdir_p Rails.root.join('db/sphinx/test')
+    FileUtils.touch path
+
+    article = Article.create! :title => 'Pancakes'
+    index 'article_core'
+    Article.search.should be_empty
+
+    FileUtils.rm path
+  end
+
+  it "indexes files when other indices are already being processed" do
+    path = Rails.root.join('db/sphinx/test/ts-book_core.tmp')
+    FileUtils.mkdir_p Rails.root.join('db/sphinx/test')
+    FileUtils.touch path
+
+    article = Article.create! :title => 'Pancakes'
+    index 'article_core'
+    Article.search.should_not be_empty
+
+    FileUtils.rm path
+  end
+end

data/spec/acceptance/remove_deleted_records_spec.rb

@@ -30,4 +30,12 @@ describe 'Hiding deleted records from search results', :live => true do
 
     Product.search_for_ids('Shiny').should be_empty
   end
+
+  it "deletes STI child classes from parent indices" do
+    duck = Bird.create :name => 'Duck'
+    index
+    duck.destroy
+
+    expect(Bird.search_for_ids('duck')).to be_empty
+  end
 end

data/spec/acceptance/searching_with_sti_spec.rb

@@ -52,4 +52,11 @@ describe 'Searching across STI models', :live => true do
 
     Bird.search(nil, :skip_sti => true, :classes=>[Bird, FlightlessBird]).to_a.should == [duck, emu]
   end
+
+  it 'finds root objects when type is blank' do
+    animal = Animal.create :name => 'Animal', type: ''
+    index
+
+    Animal.search.to_a.should == [animal]
+  end
 end

data/spec/acceptance/searching_within_a_model_spec.rb

@@ -41,6 +41,14 @@ describe 'Searching within a model', :live => true do
     articles.to_a.should == [article]
   end
 
+  it "allows for searching on distributed indices" do
+    article = Article.create :title => 'Pancakes'
+    index
+
+    articles = Article.search('pancake', :indices => ['article'])
+    articles.to_a.should == [article]
+  end
+
   it "can search on namespaced models" do
     person = Admin::Person.create :name => 'James Bond'
     index

data/spec/acceptance/sorting_search_results_spec.rb

@@ -42,7 +42,7 @@ describe 'Sorting search results', :live => true do
     index
 
     Book.search(
-      :select => 'year MOD 2004 as mod_year', :order => 'mod_year ASC'
+      :select => '*, year MOD 2004 as mod_year', :order => 'mod_year ASC'
     ).to_a.should == [boys, grave, gods]
   end
 end

data/spec/acceptance/spec_helper.rb

@@ -2,3 +2,16 @@ require 'spec_helper'
 
 root = File.expand_path File.dirname(__FILE__)
 Dir["#{root}/support/**/*.rb"].each { |file| require file }
+
+if ENV['SPHINX_VERSION'].try :[], /2.0.\d/
+  ThinkingSphinx::SphinxQL.variables!
+
+  ThinkingSphinx::Middlewares::DEFAULT.insert_after(
+    ThinkingSphinx::Middlewares::Inquirer,
+    ThinkingSphinx::Middlewares::UTF8
+  )
+  ThinkingSphinx::Middlewares::RAW_ONLY.insert_after(
+    ThinkingSphinx::Middlewares::Inquirer,
+    ThinkingSphinx::Middlewares::UTF8
+  )
+end

data/spec/acceptance/specifying_sql_spec.rb

@@ -128,7 +128,7 @@ describe 'specifying SQL for index definitions' do
     query.should match(/articles\..title., books\..title./)
   end
 
-  it "concatenates references where that have column" do
+  it "concatenates references that have column" do
     index = ThinkingSphinx::ActiveRecord::Index.new(:event)
     index.definition_block = Proc.new {
       indexes eventable.title, :as => :title
@@ -138,8 +138,8 @@ describe 'specifying SQL for index definitions' do
 
     query = index.sources.first.sql_query
     query.should match(/LEFT OUTER JOIN .articles. ON .articles.\..id. = .events.\..eventable_id. AND .events.\..eventable_type. = 'Article'/)
-    query.should match(/LEFT OUTER JOIN .users. ON .users.\..id. = .events.\..eventable_id. AND .events.\..eventable_type. = 'User'/)
     query.should_not match(/articles\..title., users\..title./)
+    query.should match(/articles\..title./)
   end
 
   it "respects deeper associations through polymorphic joins" do

data/spec/acceptance/support/sphinx_controller.rb

@@ -5,21 +5,21 @@ class SphinxController
 
   def setup
     FileUtils.mkdir_p config.indices_location
+    config.controller.bin_path = ENV['SPHINX_BIN'] || ''
     config.render_to_file && index
 
     ThinkingSphinx::Configuration.reset
 
-    if ENV['SPHINX_VERSION'].try :[], /2.1.\d/
-      ThinkingSphinx::SphinxQL.functions!
-      ThinkingSphinx::Configuration.instance.settings['utf8'] = true
-    end
-
     ActiveSupport::Dependencies.loaded.each do |path|
       $LOADED_FEATURES.delete "#{path}.rb"
     end
 
     ActiveSupport::Dependencies.clear
 
+    if ENV['SPHINX_VERSION'].try :[], /2.0.\d/
+      ThinkingSphinx::Configuration.instance.settings['utf8'] = false
+    end
+
     config.searchd.mysql41 = 9307
     config.settings['quiet_deltas']      = true
     config.settings['attribute_updates'] = true

data/spec/acceptance/support/sphinx_helpers.rb

@@ -4,10 +4,13 @@ module SphinxHelpers
   end
 
   def index(*indices)
+    sleep 0.5 if ENV['TRAVIS']
+
     yield if block_given?
 
     sphinx.index *indices
     sleep 0.25
+    sleep 0.5 if ENV['TRAVIS']
   end
 end
 

data/spec/acceptance/suspended_deltas_spec.rb

@@ -17,4 +17,38 @@ describe 'Suspend deltas for a given action', :live => true do
     sleep 0.25
     Book.search('Terry').to_a.should == [book]
   end
+
+  it "returns core records even though they are no longer valid" do
+    book = Book.create :title => 'Night Watch', :author => 'Harry Pritchett'
+    index
+
+    Book.search('Harry').to_a.should == [book]
+
+    ThinkingSphinx::Deltas.suspend :book do
+      book.reload.update_attributes(:author => 'Terry Pratchett')
+      sleep 0.25
+
+      Book.search('Terry').to_a.should == []
+    end
+
+    sleep 0.25
+    Book.search('Harry').to_a.should == [book]
+  end
+
+  it "marks core records as deleted" do
+    book = Book.create :title => 'Night Watch', :author => 'Harry Pritchett'
+    index
+
+    Book.search('Harry').to_a.should == [book]
+
+    ThinkingSphinx::Deltas.suspend_and_update :book do
+      book.reload.update_attributes(:author => 'Terry Pratchett')
+      sleep 0.25
+
+      Book.search('Terry').to_a.should == []
+    end
+
+    sleep 0.25
+    Book.search('Harry').to_a.should be_empty
+  end
 end

data/spec/internal/config/database.yml

@@ -3,3 +3,4 @@ test:
   database:     thinking_sphinx
   username:     <%= ENV['DATABASE'] == 'postgresql' ? ENV['USER'] : 'root' %>
   min_messages: warning
+  encoding:     utf8

data/spec/thinking_sphinx/active_record/callbacks/delete_callbacks_spec.rb

@@ -31,18 +31,25 @@ describe ThinkingSphinx::ActiveRecord::Callbacks::DeleteCallbacks do
   end
 
   describe '#after_destroy' do
-    let(:config)     { double('config', :indices_for_references => [index],
-      :preload_indices => true) }
+    let(:index_set)  { double 'index set', :to_a => [index] }
     let(:index)      { double('index', :name => 'foo_core',
-      :document_id_for_key => 14, :type => 'plain') }
-    let(:instance)   { double('instance', :id => 7) }
+      :document_id_for_key => 14, :type => 'plain', :distributed? => false) }
+    let(:instance)   { double('instance', :id => 7, :new_record? => false) }
 
     before :each do
-      ThinkingSphinx::Configuration.stub :instance => config
+      ThinkingSphinx::IndexSet.stub :new => index_set
     end
 
     it "performs the deletion for the index and instance" do
-      ThinkingSphinx::Deletion.should_receive(:perform).with(index, instance)
+      ThinkingSphinx::Deletion.should_receive(:perform).with(index, 7)
+
+      callbacks.after_destroy
+    end
+
+    it "doesn't do anything if the instance is a new record" do
+      instance.stub :new_record? => true
+
+      ThinkingSphinx::Deletion.should_not_receive(:perform)
 
       callbacks.after_destroy
     end

data/spec/thinking_sphinx/active_record/callbacks/update_callbacks_spec.rb

@@ -19,8 +19,8 @@ describe ThinkingSphinx::ActiveRecord::Callbacks::UpdateCallbacks do
       :settings => {'attribute_updates' => true},
       :indices_for_references => [index]) }
     let(:connection)    { double('connection', :execute => '') }
-    let(:index)         { double('index', :name => 'article_core',
-      :sources => [source], :document_id_for_key => 3) }
+    let(:index)         { double 'index', :name => 'article_core',
+      :sources => [source], :document_id_for_key => 3, :distributed? => false }
     let(:source)        { double('source', :attributes => []) }
 
     before :each do

data/spec/thinking_sphinx/active_record/database_adapters/mysql_adapter_spec.rb

@@ -40,6 +40,13 @@ describe ThinkingSphinx::ActiveRecord::DatabaseAdapters::MySQLAdapter do
     end
   end
 
+  describe '#convert_blank' do
+    it "translates arguments to a COALESCE NULLIF SQL call" do
+      adapter.convert_blank('id', 5).should == "COALESCE(NULLIF(id, ''), 5)"
+    end
+  end
+
+
   describe '#group_concatenate' do
     it "group concatenates the clause with the given separator" do
       adapter.group_concatenate('foo', ',').

data/spec/thinking_sphinx/active_record/database_adapters/postgresql_adapter_spec.rb

@@ -49,6 +49,12 @@ describe ThinkingSphinx::ActiveRecord::DatabaseAdapters::PostgreSQLAdapter do
     end
   end
 
+  describe '#convert_blank' do
+    it "translates arguments to a COALESCE NULLIF SQL call" do
+      adapter.convert_blank('id', 5).should == "COALESCE(NULLIF(id, ''), 5)"
+    end
+  end
+
   describe '#group_concatenate' do
     it "group concatenates the clause with the given separator" do
       adapter.group_concatenate('foo', ',').

data/spec/thinking_sphinx/active_record/interpreter_spec.rb

@@ -218,6 +218,33 @@ describe ThinkingSphinx::ActiveRecord::Interpreter do
     end
   end
 
+  describe '#set_database' do
+    before :each do
+      source.stub :set_database_settings => true
+
+      stub_const 'ActiveRecord::Base',
+        double(:configurations => {'other' => {:baz => :qux}})
+    end
+
+    it "sends through a hash if provided" do
+      source.should_receive(:set_database_settings).with(:foo => :bar)
+
+      instance.set_database :foo => :bar
+    end
+
+    it "finds the environment settings if given a string key" do
+      source.should_receive(:set_database_settings).with(:baz => :qux)
+
+      instance.set_database 'other'
+    end
+
+    it "finds the environment settings if given a symbol key" do
+      source.should_receive(:set_database_settings).with(:baz => :qux)
+
+      instance.set_database :other
+    end
+  end
+
   describe '#set_property' do
     before :each do
       index.class.stub  :settings => [:morphology]

data/spec/thinking_sphinx/active_record/property_sql_presenter_spec.rb

@@ -2,12 +2,14 @@ require 'spec_helper'
 
 describe ThinkingSphinx::ActiveRecord::PropertySQLPresenter do
   let(:adapter)      { double 'adapter' }
-  let(:associations) { double 'associations', :alias_for => 'articles',
-    :aggregate_for? => false, :model_for => model }
+  let(:associations) { double 'associations', :alias_for => 'articles' }
   let(:model)        { double :column_names => ['title', 'created_at'] }
+  let(:path)         { double :aggregate? => false, :model => model }
 
   before :each do
     adapter.stub(:quote) { |column| column }
+
+    stub_const 'Joiner::Path', double(:new => path)
   end
 
   context 'with a field' do
@@ -17,7 +19,7 @@ describe ThinkingSphinx::ActiveRecord::PropertySQLPresenter do
       )
     }
     let(:field)     { double('field', :name => 'title', :columns => [column],
-      :type => nil, :multi? => false, :source_type => nil) }
+      :type => nil, :multi? => false, :source_type => nil, :model => double) }
     let(:column)    { double('column', :string? => false, :__stack => [],
       :__name => 'title') }
 
@@ -36,7 +38,7 @@ describe ThinkingSphinx::ActiveRecord::PropertySQLPresenter do
       end
 
       it "returns nil if the property is an aggregate" do
-        associations.stub! :aggregate_for? => true
+        path.stub! :aggregate? => true
 
         presenter.to_group.should be_nil
       end
@@ -73,7 +75,7 @@ describe ThinkingSphinx::ActiveRecord::PropertySQLPresenter do
           "GROUP_CONCAT(#{clause} SEPARATOR '#{separator}')"
         end
 
-        associations.stub! :aggregate_for? => true
+        path.stub! :aggregate? => true
 
         presenter.to_select.
           should == "GROUP_CONCAT(articles.title SEPARATOR ' ') AS title"
@@ -124,7 +126,7 @@ describe ThinkingSphinx::ActiveRecord::PropertySQLPresenter do
     }
     let(:attribute) { double('attribute', :name => 'created_at',
       :columns => [column], :type => :integer, :multi? => false,
-      :source_type => nil) }
+      :source_type => nil, :model => double) }
     let(:column)    { double('column', :string? => false, :__stack => [],
       :__name => 'created_at') }
 
@@ -155,7 +157,7 @@ describe ThinkingSphinx::ActiveRecord::PropertySQLPresenter do
       end
 
       it "returns nil if the property is an aggregate" do
-        associations.stub! :aggregate_for? => true
+        path.stub! :aggregate? => true
 
         presenter.to_group.should be_nil
       end
@@ -235,6 +237,15 @@ describe ThinkingSphinx::ActiveRecord::PropertySQLPresenter do
         presenter.to_select.should == "CONCAT_WS(',', CAST(UNIX_TIMESTAMP(articles.created_at) AS varchar), CAST(UNIX_TIMESTAMP(articles.created_at) AS varchar)) AS created_at"
       end
 
+      it "does not split attribute clause for timestamp casting if it looks like a function call" do
+        column.stub :__name => "COALESCE(articles.updated_at, articles.created_at)", :string? => true
+
+        attribute.stub :name => 'mod_date', :columns => [column],
+          :type => :timestamp
+
+        presenter.to_select.should == "UNIX_TIMESTAMP(COALESCE(articles.updated_at, articles.created_at)) AS mod_date"
+      end
+
       it "returns nil for query sourced attributes" do
         attribute.stub :source_type => :query
 

data/spec/thinking_sphinx/active_record/sql_builder_spec.rb

@@ -24,7 +24,7 @@ describe ThinkingSphinx::ActiveRecord::SQLBuilder do
   before :each do
     ThinkingSphinx::Configuration.stub! :instance => config
     ThinkingSphinx::ActiveRecord::PropertySQLPresenter.stub! :new => presenter
-    ThinkingSphinx::ActiveRecord::Associations.stub! :new => associations
+    Joiner::Joins.stub! :new => associations
     relation.stub! :select => relation, :where => relation, :group => relation,
       :order => relation, :joins => relation, :to_sql => ''
     connection.stub!(:quote_column_name) { |column| "`#{column}`"}
@@ -545,6 +545,22 @@ describe ThinkingSphinx::ActiveRecord::SQLBuilder do
     end
   end
 
+  describe 'sql_query_post_index' do
+    let(:processor) { double('processor', :reset_query => 'RESET DELTAS') }
+
+    it "adds a reset delta query if there is a delta processor and this is the core source" do
+      source.stub :delta_processor => processor, :delta? => false
+
+      builder.sql_query_post_index.should include('RESET DELTAS')
+    end
+
+    it "adds no reset delta query if there is a delta processor and this is the delta source" do
+      source.stub :delta_processor => processor, :delta? => true
+
+      builder.sql_query_post_index.should_not include('RESET DELTAS')
+    end
+  end
+
   describe 'sql_query_pre' do
     let(:processor) { double('processor', :reset_query => 'RESET DELTAS') }
 
@@ -553,12 +569,6 @@ describe ThinkingSphinx::ActiveRecord::SQLBuilder do
       adapter.stub :utf8_query_pre => ['SET UTF8']
     end
 
-    it "adds a reset delta query if there is a delta processor and this is the core source" do
-      source.stub :delta_processor => processor
-
-      builder.sql_query_pre.should include('RESET DELTAS')
-    end
-
     it "does not add a reset query if there is no delta processor" do
       builder.sql_query_pre.should_not include('RESET DELTAS')
     end

data/spec/thinking_sphinx/active_record/sql_source_spec.rb

@@ -1,8 +1,9 @@
 require 'spec_helper'
 
 describe ThinkingSphinx::ActiveRecord::SQLSource do
-  let(:model)      { double('model', :name => 'User', :column_names => [],
-    :inheritance_column => 'type', :primary_key => :id) }
+  let(:model)      { double('model', :connection => connection,
+    :name => 'User', :column_names => [], :inheritance_column => 'type',
+    :primary_key => :id) }
   let(:connection) {
     double('connection', :instance_variable_get => db_config) }
   let(:db_config)  { {:host => 'localhost', :user => 'root',
@@ -12,8 +13,6 @@ describe ThinkingSphinx::ActiveRecord::SQLSource do
   let(:adapter)    { double('adapter') }
 
   before :each do
-    stub_const 'ActiveRecord::Base', double(:connection => connection)
-
     ThinkingSphinx::ActiveRecord::DatabaseAdapters::MySQLAdapter.
       stub!(:=== => true)
     ThinkingSphinx::ActiveRecord::DatabaseAdapters.
@@ -56,9 +55,14 @@ describe ThinkingSphinx::ActiveRecord::SQLSource do
       ThinkingSphinx::ActiveRecord::SQLSource.new model,
         :delta_processor => processor_class
     }
+    let(:source_with_options)    {
+      ThinkingSphinx::ActiveRecord::SQLSource.new model,
+        :delta_processor => processor_class,
+        :delta_options   => { :opt_key => :opt_value }
+    }
 
     it "loads the processor with the adapter" do
-      processor_class.should_receive(:try).with(:new, adapter).
+      processor_class.should_receive(:try).with(:new, adapter, {}).
         and_return processor
 
       source.delta_processor
@@ -67,6 +71,11 @@ describe ThinkingSphinx::ActiveRecord::SQLSource do
     it "returns the given processor" do
       source.delta_processor.should == processor
     end
+
+    it "passes given options to the processor" do
+      processor_class.should_receive(:try).with(:new, adapter, {:opt_key => :opt_value})
+      source_with_options.delta_processor
+    end
   end
 
   describe '#delta?' do
@@ -101,15 +110,15 @@ describe ThinkingSphinx::ActiveRecord::SQLSource do
 
     it "uses the inheritance column if it exists for the sphinx class field" do
       adapter.stub :quoted_table_name => '"users"', :quote => '"type"'
-      adapter.stub(:convert_nulls) { |clause, default|
-        "ifnull(#{clause}, #{default})"
+      adapter.stub(:convert_blank) { |clause, default|
+        "coalesce(nullif(#{clause}, ''), #{default})"
       }
       model.stub :column_names => ['type'], :sti_name => 'User'
 
       source.fields.detect { |field|
         field.name == 'sphinx_internal_class_name'
       }.columns.first.__name.
-        should == "ifnull(\"users\".\"type\", 'User')"
+        should == "coalesce(nullif(\"users\".\"type\", ''), 'User')"
     end
   end
 
@@ -147,7 +156,9 @@ describe ThinkingSphinx::ActiveRecord::SQLSource do
   end
 
   describe '#render' do
-    let(:builder)   { double('builder', :sql_query_pre => []).as_null_object }
+    let(:builder)   { double('builder', :sql_query_pre => [],
+      :sql_query_post_index => [], :sql_query => 'query',
+      :sql_query_range => 'range', :sql_query_info => 'info') }
     let(:config)    { double('config', :settings => {}) }
     let(:presenter) { double('presenter', :collection_type => :uint) }
     let(:template)  { double('template', :apply => true) }
@@ -159,79 +170,6 @@ describe ThinkingSphinx::ActiveRecord::SQLSource do
       ThinkingSphinx::Configuration.stub :instance => config
     end
 
-    it "sets the sql_host setting from the model's database settings" do
-      db_config[:host] = '12.34.56.78'
-
-      source.render
-
-      source.sql_host.should == '12.34.56.78'
-    end
-
-    it "defaults sql_host to localhost if the model has no host" do
-      db_config[:host] = nil
-
-      source.render
-
-      source.sql_host.should == 'localhost'
-    end
-
-    it "sets the sql_user setting from the model's database settings" do
-      db_config[:username] = 'pat'
-
-      source.render
-
-      source.sql_user.should == 'pat'
-    end
-
-    it "uses the user setting if username is not set in the model" do
-      db_config[:username] = nil
-      db_config[:user]     = 'pat'
-
-      source.render
-
-      source.sql_user.should == 'pat'
-    end
-
-    it "sets the sql_pass setting from the model's database settings" do
-      db_config[:password] = 'swordfish'
-
-      source.render
-
-      source.sql_pass.should == 'swordfish'
-    end
-
-    it "escapes hashes in the password for sql_pass" do
-      db_config[:password] = 'sword#fish'
-
-      source.render
-
-      source.sql_pass.should == 'sword\#fish'
-    end
-
-    it "sets the sql_db setting from the model's database settings" do
-      db_config[:database] = 'rails_app'
-
-      source.render
-
-      source.sql_db.should == 'rails_app'
-    end
-
-    it "sets the sql_port setting from the model's database settings" do
-      db_config[:port] = 5432
-
-      source.render
-
-      source.sql_port.should == 5432
-    end
-
-    it "sets the sql_sock setting from the model's database settings" do
-      db_config[:socket] = '/unix/socket'
-
-      source.render
-
-      source.sql_sock.should == '/unix/socket'
-    end
-
     it "uses the builder's sql_query value" do
       builder.stub! :sql_query => 'select * from table'
 
@@ -264,6 +202,14 @@ describe ThinkingSphinx::ActiveRecord::SQLSource do
       source.sql_query_pre.should == ['Change Setting']
     end
 
+    it "appends the builder's sql_query_post_index value" do
+      builder.stub! :sql_query_post_index => ['RESET DELTAS']
+
+      source.render
+
+      source.sql_query_post_index.should include('RESET DELTAS')
+    end
+
     it "adds fields with attributes to sql_field_string" do
       source.fields << double('field', :name => 'title', :source_type => nil,
         :with_attribute? => true, :file? => false, :wordcount? => false)
@@ -397,6 +343,62 @@ describe ThinkingSphinx::ActiveRecord::SQLSource do
     end
   end
 
+  describe '#set_database_settings' do
+    it "sets the sql_host setting from the model's database settings" do
+      source.set_database_settings :host => '12.34.56.78'
+
+      source.sql_host.should == '12.34.56.78'
+    end
+
+    it "defaults sql_host to localhost if the model has no host" do
+      source.set_database_settings :host => nil
+
+      source.sql_host.should == 'localhost'
+    end
+
+    it "sets the sql_user setting from the model's database settings" do
+      source.set_database_settings :username => 'pat'
+
+      source.sql_user.should == 'pat'
+    end
+
+    it "uses the user setting if username is not set in the model" do
+      source.set_database_settings :username => nil, :user => 'pat'
+
+      source.sql_user.should == 'pat'
+    end
+
+    it "sets the sql_pass setting from the model's database settings" do
+      source.set_database_settings :password => 'swordfish'
+
+      source.sql_pass.should == 'swordfish'
+    end
+
+    it "escapes hashes in the password for sql_pass" do
+      source.set_database_settings :password => 'sword#fish'
+
+      source.sql_pass.should == 'sword\#fish'
+    end
+
+    it "sets the sql_db setting from the model's database settings" do
+      source.set_database_settings :database => 'rails_app'
+
+      source.sql_db.should == 'rails_app'
+    end
+
+    it "sets the sql_port setting from the model's database settings" do
+      source.set_database_settings :port => 5432
+
+      source.sql_port.should == 5432
+    end
+
+    it "sets the sql_sock setting from the model's database settings" do
+      source.set_database_settings :socket => '/unix/socket'
+
+      source.sql_sock.should == '/unix/socket'
+    end
+  end
+
   describe '#type' do
     it "is mysql when using the MySQL Adapter" do
       ThinkingSphinx::ActiveRecord::DatabaseAdapters::MySQLAdapter.