sayso-thinking-sphinx 2.0.3.001

data/spec/thinking_sphinx/auto_version_spec.rb

@@ -0,0 +1,63 @@
+require 'spec_helper'
+
+describe ThinkingSphinx::AutoVersion do
+  describe '.detect' do
+    before :each do
+      @config = ThinkingSphinx::Configuration.instance
+    end
+    
+    it "should require 0.9.8 if that is the detected version" do
+      ThinkingSphinx::AutoVersion.should_receive(:require).
+        with('riddle/0.9.8')
+      
+      @config.stub!(:version => '0.9.8')
+      ThinkingSphinx::AutoVersion.detect
+    end
+    
+    it "should require 0.9.9 if that is the detected version" do
+      ThinkingSphinx::AutoVersion.should_receive(:require).
+        with('riddle/0.9.9')
+      
+      @config.stub!(:version => '0.9.9')
+      ThinkingSphinx::AutoVersion.detect
+    end
+    
+    it "should require 1.10-beta if that is the detected version" do
+      ThinkingSphinx::AutoVersion.should_receive(:require).
+        with('riddle/1.10')
+      
+      @config.stub!(:version => '1.10-beta')
+      ThinkingSphinx::AutoVersion.detect
+    end
+    
+    it "should require 1.10-beta if using 1.10-beta compiled with id64 support" do
+      ThinkingSphinx::AutoVersion.should_receive(:require).
+        with('riddle/1.10')
+      
+      @config.stub!(:version => '1.10-id64-beta')
+      ThinkingSphinx::AutoVersion.detect
+    end
+    
+    it "should require 2.0.1 if using Sphinx 2.0.1 beta" do
+      ThinkingSphinx::AutoVersion.should_receive(:require).
+        with('riddle/2.0.1')
+      
+      @config.stub!(:version => '2.0.1-beta')
+      ThinkingSphinx::AutoVersion.detect
+    end
+    
+    it "should output a warning if the detected version is unsupported" do
+      STDERR.should_receive(:puts).with(/unsupported/i)
+      
+      @config.stub!(:version => '0.9.7')
+      ThinkingSphinx::AutoVersion.detect
+    end
+    
+    it "should output a warning if the version cannot be determined" do
+      STDERR.should_receive(:puts).at_least(:once)
+      
+      @config.stub!(:version => nil)
+      ThinkingSphinx::AutoVersion.detect
+    end
+  end
+end

data/spec/thinking_sphinx/configuration_spec.rb

@@ -0,0 +1,288 @@
+require 'spec_helper'
+
+describe ThinkingSphinx::Configuration do
+  describe "environment class method" do
+    before :each do
+      ThinkingSphinx::Configuration.reset_environment
+      
+      ENV["RAILS_ENV"] = nil
+    end
+
+    it "should use Rails.env if set" do
+      was = Rails.env
+      Rails.env = 'global_rails'
+      
+      ThinkingSphinx::Configuration.environment.should == 'global_rails'
+      
+      Rails.env = was
+    end
+
+    it "should use the Rails environment value if set" do
+      Rails.stub!(:env => "rails_production")
+      ThinkingSphinx::Configuration.environment.should == "rails_production"
+    end
+
+    it "should default to development" do
+      ThinkingSphinx::Configuration.environment.should == "development"
+    end
+  end
+  
+  describe '#version' do
+    before :each do
+      @config = ThinkingSphinx::Configuration.instance
+      @config.reset
+    end
+    
+    it "should use the given version from sphinx.yml if there is one" do
+      open("#{Rails.root}/config/sphinx.yml", "w") do |f|
+        f.write  YAML.dump({'development' => {'version' => '0.9.7'}})
+      end
+      @config.reset
+      
+      @config.version.should == '0.9.7'
+      
+      FileUtils.rm "#{Rails.root}/config/sphinx.yml"
+    end
+    
+    it "should detect the version from Riddle otherwise" do
+      controller = @config.controller
+      controller.stub!(:sphinx_version => '0.9.6')
+      
+      Riddle::Controller.stub!(:new => controller)
+      @config.reset
+      
+      @config.version.should == '0.9.6'
+    end
+  end
+
+  describe "parse_config method" do
+    before :each do
+      @settings = {
+        "development" => {
+          "config_file"       => "tmp/config/development.sphinx.conf",
+          "searchd_log_file"  => "searchd_log_file.log",
+          "query_log_file"    => "query_log_file.log",
+          "pid_file"          => "pid_file.pid",
+          "searchd_file_path" => "searchd/file/path",
+          "address"           => "127.0.0.1",
+          "port"              => 3333,
+          "min_prefix_len"    => 2,
+          "min_infix_len"     => 3,
+          "mem_limit"         => "128M",
+          "max_matches"       => 1001,
+          "morphology"        => "stem_ru",
+          "charset_type"      => "latin1",
+          "charset_table"     => "table",
+          "ignore_chars"      => "e",
+          "searchd_binary_name" => "sphinx-searchd",
+          "indexer_binary_name" => "sphinx-indexer",
+          "index_exact_words" => true,
+          "indexed_models"    => ['Alpha', 'Beta']
+        }
+      }
+
+      open("#{Rails.root}/config/sphinx.yml", "w") do |f|
+        f.write  YAML.dump(@settings)
+      end
+    end
+
+    it "should use the accessors to set the configuration values" do
+      config = ThinkingSphinx::Configuration.instance
+      config.send(:parse_config)
+
+      %w(config_file searchd_log_file query_log_file pid_file searchd_file_path
+        address port searchd_binary_name indexer_binary_name).each do |key|
+        config.send(key).should == @settings["development"][key]
+      end
+    end
+
+    after :each do
+      FileUtils.rm "#{Rails.root}/config/sphinx.yml"
+    end
+  end
+
+  describe "block configuration" do
+    it "should let the user set-up a custom app_root" do
+      ThinkingSphinx::Configuration.configure do |config|
+        config.app_root = "/here/somewhere"
+      end
+      ThinkingSphinx::Configuration.instance.app_root.should == "/here/somewhere"
+      
+      ThinkingSphinx::Configuration.instance.reset
+    end
+  end
+
+  describe "initialisation" do
+    it "should have a default bin_path of nothing" do
+      ThinkingSphinx::Configuration.instance.bin_path.should == ""
+    end
+
+    it "should append a / to bin_path if one is supplied" do
+      @settings = {
+        "development" => {
+          "bin_path" => "path/to/somewhere"
+        }
+      }
+
+      open("#{Rails.root}/config/sphinx.yml", "w") do |f|
+        f.write YAML.dump(@settings)
+      end
+      
+      ThinkingSphinx::Configuration.instance.send(:parse_config)
+      ThinkingSphinx::Configuration.instance.bin_path.should match(/\/$/)
+
+      FileUtils.rm "#{Rails.root}/config/sphinx.yml"
+    end
+  end
+
+  describe "index options" do
+    before :each do
+      @settings = {
+        "development" => {"disable_range" => true}
+      }
+
+      open("#{Rails.root}/config/sphinx.yml", "w") do |f|
+        f.write  YAML.dump(@settings)
+      end
+
+      @config = ThinkingSphinx::Configuration.instance
+      @config.send(:parse_config)
+    end
+
+    it "should collect disable_range" do
+      @config.index_options[:disable_range].should be_true
+    end
+
+    after :each do
+      FileUtils.rm "#{Rails.root}/config/sphinx.yml"
+    end
+  end
+
+  it "should insert set index options into the configuration file" do
+    config = ThinkingSphinx::Configuration.instance
+    
+    ThinkingSphinx::Configuration::IndexOptions.each do |option|
+      config.reset
+      config.index_options[option.to_sym] = "something"
+      config.build
+
+      file = open(config.config_file) { |f| f.read }
+      file.should match(/#{option}\s+= something/)
+
+      config.index_options[option.to_sym] = nil
+    end
+  end
+
+  it "should insert set source options into the configuration file" do
+    config = ThinkingSphinx::Configuration.instance
+    config.reset
+    
+    config.source_options[:sql_query_pre] = ["something"]
+    ThinkingSphinx::Configuration::SourceOptions.each do |option|
+      config.source_options[option.to_sym] ||= "something"
+      config.build
+
+      file = open(config.config_file) { |f| f.read }
+      file.should match(/#{option}\s+= something/)
+
+      config.source_options.delete option.to_sym
+    end
+    
+    config.source_options[:sql_query_pre] = []  
+  end
+  
+  it "should not blow away delta or utf options if sql pre is specified in config" do
+    config = ThinkingSphinx::Configuration.instance
+    config.reset
+    
+    config.source_options[:sql_query_pre] = ["a pre query"]
+    config.build
+    file = open(config.config_file) { |f| f.read }
+    
+    file.should match(/sql_query_pre = a pre query\n\s*sql_query_pre = UPDATE `\w+` SET `delta` = 0 WHERE `delta` = 1/im)
+    file.should match(/sql_query_pre = a pre query\n\s*sql_query_pre = \n/im)
+    
+    config.source_options[:sql_query_pre] = []
+  end
+
+  it "should set any explicit prefixed or infixed fields" do
+    ThinkingSphinx::Configuration.instance.build
+    
+    file = open(ThinkingSphinx::Configuration.instance.config_file) { |f|
+      f.read
+    }
+    file.should match(/prefix_fields\s+= city/)
+    file.should match(/infix_fields\s+= state/)
+  end
+
+  it "should not have prefix fields in indexes where nothing is set" do
+    ThinkingSphinx::Configuration.instance.build
+    
+    file = open(ThinkingSphinx::Configuration.instance.config_file) { |f|
+      f.read
+    }
+    file.should_not match(/index alpha_core\s+\{\s+[^\}]*prefix_fields\s+=[^\}]*\}/m)
+  end
+  
+  describe '#generate' do
+    let(:config) { ThinkingSphinx::Configuration.instance }
+    
+    it "should set all sphinx_internal_id attributes to bigints if one is" do
+      config.reset
+      config.generate
+      
+      config.configuration.indexes.each do |index|
+        next if index.is_a? Riddle::Configuration::DistributedIndex
+        
+        index.sources.each do |source|
+          source.sql_attr_bigint.should include(:sphinx_internal_id)
+        end
+      end
+    end
+  end
+  
+  describe '#client' do
+    before :each do
+      @config = ThinkingSphinx::Configuration.instance
+      @config.address     = 'domain.url'
+      @config.port        = 3333
+      @config.configuration.searchd.max_matches = 100
+      @config.timeout = 1
+    end
+    
+    it "should return an instance of Riddle::Client" do
+      @config.client.should be_a(Riddle::Client)
+    end
+    
+    it "should use the configuration address" do
+      @config.client.server.should == 'domain.url'
+    end
+    
+    it "should use the configuration port" do
+      @config.client.port.should == 3333
+    end
+    
+    it "should use the configuration max matches" do
+      @config.client.max_matches.should == 100
+    end
+
+    it "should use the configuration timeout" do
+      @config.client.timeout.should == 1
+    end
+  end
+  
+  describe '#models_by_crc' do
+    before :each do
+      @config = ThinkingSphinx::Configuration.instance
+    end
+    
+    it "should return a hash" do
+      @config.models_by_crc.should be_a(Hash)
+    end
+    
+    it "should pair class names to their crc codes" do
+      @config.models_by_crc[Person.to_crc32].should == 'Person'
+      @config.models_by_crc[Alpha.to_crc32].should  == 'Alpha'
+    end
+  end
+end

data/spec/thinking_sphinx/context_spec.rb

@@ -0,0 +1,128 @@
+require 'spec_helper'
+
+describe ThinkingSphinx::Context do
+  before :each do
+    @context = ThinkingSphinx::Context.new
+  end
+  
+  describe '#prepare' do
+    before :each do
+      @config = ThinkingSphinx::Configuration.instance
+      @config.model_directories = ['']
+
+      @file_name        = 'a.rb'
+      @model_name_lower = 'a'
+      @class_name       = 'A'
+
+      @file_name.stub!(:gsub).and_return(@model_name_lower)
+      @model_name_lower.stub!(:camelize).and_return(@class_name)
+      Dir.stub(:[]).and_return([@file_name])
+    end
+
+    it "should load the files by guessing the file name" do
+      @class_name.should_receive(:constantize).and_return(true)
+
+      @context.prepare
+    end
+
+    it "should not raise errors if the model name is nil" do
+      @file_name.stub!(:gsub).and_return(nil)
+
+      lambda {
+        @context.prepare
+      }.should_not raise_error
+    end
+
+    it "should not raise errors if the file name does not represent a class name" do
+      @class_name.should_receive(:constantize).and_raise(NameError)
+
+      lambda {
+        @context.prepare
+      }.should_not raise_error
+    end
+
+    # Fails in Ruby 1.9 (or maybe it's an RSpec update). Not sure why.
+    it "should retry if the first pass fails and contains a directory" do
+      @model_name_lower.stub!(:gsub!).and_return(true, nil)
+      @class_name.stub(:constantize).and_raise(LoadError)
+      @model_name_lower.should_receive(:camelize).twice
+
+      lambda {
+        @context.prepare
+      }.should_not raise_error
+    end
+
+    it "should catch database errors with a warning" do
+      @class_name.should_receive(:constantize).and_raise(Mysql::Error)
+      STDERR.stub!(:puts => '')
+      STDERR.should_receive(:puts).with('Warning: Error loading a.rb:')
+      
+      lambda {
+        @context.prepare
+      }.should_not raise_error
+    end
+    
+    it "should not load models if they're explicitly set in the configuration" do
+      @config.indexed_models = ['Alpha', 'Beta']
+      @context.prepare
+      
+      @context.indexed_models.should == ['Alpha', 'Beta']
+    end
+  end
+  
+  describe '#define_indexes' do
+    it "should call define_indexes on all known indexed models" do
+      @context.stub!(:indexed_models => ['Alpha', 'Beta'])
+      Alpha.should_receive(:define_indexes)
+      Beta.should_receive(:define_indexes)
+      
+      @context.define_indexes
+    end
+  end
+  
+  describe '#add_indexed_model' do
+    before :each do
+      @context.indexed_models.clear
+    end
+    
+    it "should add the model to the collection" do
+      @context.add_indexed_model 'Alpha'
+      
+      @context.indexed_models.should == ['Alpha']
+    end
+    
+    it "should not duplicate models in the collection" do
+      @context.add_indexed_model 'Alpha'
+      @context.add_indexed_model 'Alpha'
+      
+      @context.indexed_models.should == ['Alpha']
+    end
+    
+    it "should keep the collection in alphabetical order" do
+      @context.add_indexed_model 'Beta'
+      @context.add_indexed_model 'Alpha'
+      
+      @context.indexed_models.should == ['Alpha', 'Beta']
+    end
+    
+    it "should translate classes to their names" do
+      @context.add_indexed_model Alpha
+      
+      @context.indexed_models.should == ['Alpha']
+    end
+  end
+  
+  describe '#superclass_indexed_models' do
+    it "should return indexed model names" do
+      @context.stub!(:indexed_models => ['Alpha', 'Beta'])
+      
+      @context.superclass_indexed_models.should == ['Alpha', 'Beta']
+    end
+    
+    it "should not include classes which have indexed superclasses" do
+      @context.stub!(:indexed_models => ['Parent', 'Person'])
+      
+      @context.superclass_indexed_models.should == ['Person']
+    end
+  end
+end