freelancing-god-thinking-sphinx 1.1.24 → 1.2.0

data/spec/unit/thinking_sphinx/active_record/search_spec.rb

@@ -1,107 +0,0 @@
-require 'spec/spec_helper'
-
-describe "ThinkingSphinx::ActiveRecord::Search" do
-  it "should add search_for_ids to ActiveRecord::Base" do
-    ActiveRecord::Base.should respond_to("search_for_ids")
-  end
-  
-  it "should add search_for_ids to ActiveRecord::Base" do
-    ActiveRecord::Base.should respond_to("search")
-  end
-  
-  it "should add search_count to ActiveRecord::Base" do
-    ActiveRecord::Base.should respond_to("search_count")
-  end
-
-  it "should add search_for_id to ActiveRecord::Base" do
-    ActiveRecord::Base.should respond_to("search_for_id")
-  end
-  
-  describe "search_for_ids method" do
-    before :each do
-      ThinkingSphinx::Search.stub_method(:search_for_ids => true)
-    end
-    
-    it "should call ThinkingSphinx::Search#search_for_ids with the class option set" do
-      Person.search_for_ids("search")
-      
-      ThinkingSphinx::Search.should have_received(:search_for_ids).with(
-        "search", :class => Person
-      )
-    end
-    
-    it "should override the class option" do
-      Person.search_for_ids("search", :class => Friendship)
-      
-      ThinkingSphinx::Search.should have_received(:search_for_ids).with(
-        "search", :class => Person
-      )
-    end
-  end
-  
-  describe "search method" do
-    before :each do
-      ThinkingSphinx::Search.stub_method(:search => true)
-    end
-    
-    it "should call ThinkingSphinx::Search#search with the class option set" do
-      Person.search("search")
-      
-      ThinkingSphinx::Search.should have_received(:search).with(
-        "search", :class => Person
-      )
-    end
-    
-    it "should override the class option" do
-      Person.search("search", :class => Friendship)
-      
-      ThinkingSphinx::Search.should have_received(:search).with(
-        "search", :class => Person
-      )
-    end
-  end
-  
-  describe "search_for_id method" do
-    before :each do
-      ThinkingSphinx::Search.stub_method(:search_for_id => true)
-    end
-    
-    it "should call ThinkingSphinx::Search#search with the class option set" do
-      Person.search_for_id(10)
-      
-      ThinkingSphinx::Search.should have_received(:search_for_id).with(
-        10, :class => Person
-      )
-    end
-    
-    it "should override the class option" do
-      Person.search_for_id(10, :class => Friendship)
-      
-      ThinkingSphinx::Search.should have_received(:search_for_id).with(
-        10, :class => Person
-      )
-    end
-  end
-
-  describe "search_count method" do
-    before :each do
-      ThinkingSphinx::Search.stub_method(:count => true)
-    end
-
-    it "should call ThinkingSphinx::Search#search with the class option set" do
-      Person.search_count("search")
-
-      ThinkingSphinx::Search.should have_received(:count).with(
-        "search", :class => Person
-      )
-    end
-
-    it "should override the class option" do
-      Person.search_count("search", :class => Friendship)
-
-      ThinkingSphinx::Search.should have_received(:count).with(
-        "search", :class => Person
-      )
-    end
-  end
-end

data/spec/unit/thinking_sphinx/collection_spec.rb

@@ -1,15 +0,0 @@
-require 'spec/spec_helper'
-
-describe ThinkingSphinx::Collection do
-  it "should behave like WillPaginate::Collection" do
-    instance_methods = ThinkingSphinx::Collection.instance_methods.collect { |m| m.to_s }
-    instance_methods.should include("previous_page")
-    instance_methods.should include("next_page")
-    instance_methods.should include("current_page")
-    instance_methods.should include("total_pages")
-    instance_methods.should include("total_entries")
-    instance_methods.should include("offset")
-    
-    ThinkingSphinx::Collection.ancestors.should include(Array)
-  end
-end

data/spec/unit/thinking_sphinx/facet_collection_spec.rb

@@ -1,64 +0,0 @@
-require 'spec/spec_helper'
-
-describe ThinkingSphinx::FacetCollection do
-  before do
-    @facet_collection = ThinkingSphinx::FacetCollection.new([])
-  end
-
-  # TODO fix nasty hack when we have internet!
-  def mock_results
-    return @results if defined? @results
-    @result = Person.find(:first)
-    @results = [@result]
-    @results.stub!(:each_with_groupby_and_count).and_yield(@result, @result.city.to_crc32, 1)
-    @results
-  end
-
-  describe "#add_from_results" do
-    describe "with empty result set" do
-      before do
-        @facet_collection.add_from_results('attribute_facet', [])
-      end
-
-      it "should add key as attribute" do
-        @facet_collection.should have_key(:attribute)
-      end
-
-      it "should return an empty hash for the facet results" do
-        @facet_collection[:attribute].should be_empty
-      end
-    end
-
-    describe "with non-empty result set" do
-      before do
-        @facet_collection.add_from_results('city_facet', mock_results)
-      end
-
-      it "should return a hash" do
-        @facet_collection.should be_a_kind_of(Hash)
-      end
-
-      it "should add key as attribute" do
-        @facet_collection.keys.should include(:city)
-      end
-
-      it "should return a hash" do
-        @facet_collection[:city].should == {@result.city => 1}
-      end
-    end
-  end
-
-  describe "#for" do
-    before do
-      @facet_collection.add_from_results('city_facet', mock_results)
-    end
-
-    it "should return the search results for the attribute and key pair" do
-      ThinkingSphinx::Search.should_receive(:search) do |options|
-        options[:with].should have_key('city_facet')
-        options[:with]['city_facet'].should == @result.city.to_crc32
-      end
-      @facet_collection.for(:city => @result.city)
-    end
-  end
-end

data/spec/unit/thinking_sphinx/search_spec.rb

@@ -1,228 +0,0 @@
-require 'spec/spec_helper'
-require 'will_paginate/collection'
-
-describe ThinkingSphinx::Search do
-  describe "search method" do
-    describe ":star option" do
-      before :each do
-        @client = Riddle::Client.new
-        @client.stub!(
-          :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 not apply by default" do
-        @client.should_receive(:query).with("foo bar",'*','')
-        
-        ThinkingSphinx::Search.search "foo bar"
-      end
-
-      it "should apply when passed, and handle full extended syntax" do
-        input    = %{a b* c (d | e) 123 5&6 (f_f g) !h "i j" "k l"~10 "m n"/3 @o p -(q|r)}
-        expected = %{*a* b* *c* (*d* | *e*) *123* *5*&*6* (*f_f* *g*) !*h* "i j" "k l"~10 "m n"/3 @o *p* -(*q*|*r*)}
-        @client.should_receive(:query).with(expected,'*','')
-        
-        ThinkingSphinx::Search.search input, :star => true
-      end
-
-      it "should default to /\w+/ as token" do
-        @client.should_receive(:query).with("*foo*@*bar*.*com*",'*','')
-        
-        ThinkingSphinx::Search.search "foo@bar.com", :star => true
-      end
-
-      it "should honour custom token" do
-        @client.should_receive(:query).with("*foo@bar.com* -*foo-bar*",'*','')
-        
-        ThinkingSphinx::Search.search "foo@bar.com -foo-bar", :star => /[\w@.-]+/u
-      end
-    end
-    
-    describe "sort modes" do
-      before :each do
-        @client = Riddle::Client.new
-        @client.stub_method(:query => {:matches => []})
-        Riddle::Client.stub_method(:new => @client)
-      end
-      
-      it "should use :relevance as a default" do
-        ThinkingSphinx::Search.search "foo"
-        @client.sort_mode.should == :relevance
-      end
-      
-      it "should use :attr_asc if a symbol is supplied to :order" do
-        ThinkingSphinx::Search.search "foo", :order => :created_at
-        @client.sort_mode.should == :attr_asc
-      end
-      
-      it "should use :attr_desc if a symbol is supplied and :desc is the mode" do
-        ThinkingSphinx::Search.search "foo", :order => :created_at, :sort_mode => :desc
-        @client.sort_mode.should == :attr_desc
-      end
-      
-      it "should use :extended if a string is supplied to :order" do
-        ThinkingSphinx::Search.search "foo", :order => "created_at ASC"
-        @client.sort_mode.should == :extended
-      end
-      
-      it "should use :expr if explicitly requested with a string supplied to :order" do
-        ThinkingSphinx::Search.search "foo", :order => "created_at ASC", :sort_mode => :expr
-        @client.sort_mode.should == :expr
-      end
-      
-      it "should use :attr_desc if explicitly requested with a string supplied to :order" do
-        ThinkingSphinx::Search.search "foo", :order => "created_at", :sort_mode => :desc
-        @client.sort_mode.should == :attr_desc
-      end
-    end
-    
-    describe "grouping" do
-      before :each do
-        @client = Riddle::Client.new
-        @client.stub_method(:query => {:matches => []})
-        Riddle::Client.stub_method(:new => @client)
-      end
-
-      it "should convert group into group_by and group_function" do
-        ThinkingSphinx::Search.search "foo", :group => :edition
-        @client.group_function.should == :attr
-        @client.group_by.should == "edition"
-      end
-    end
-    
-    describe ":comment option" do
-      before :each do
-        @client = Riddle::Client.new
-        @client.stub_method(:query => {:matches => []})
-        Riddle::Client.stub_method(:new => @client)
-      end
-      
-      it "should add comment to log" do
-        ThinkingSphinx::Search.search 'foo bar', :comment => 'custom log'
-        @client.should have_received(:query).with('foo bar', '*', 'custom log')
-      end
-      
-      it "should use a blank string when no comment is specified" do
-        ThinkingSphinx::Search.search 'foo bar'
-        @client.should have_received(:query).with('foo bar', '*', '')
-      end
-    end
-  end
-  
-  describe "facets method" do
-    before :each do
-      @person = Person.find(:first)
-      
-      @city_results = [@person]
-      @city_results.stub!(:each_with_groupby_and_count).
-        and_yield(@person, @person.city.to_crc32, 1)
-      
-      @birthday_results = [@person]
-      @birthday_results.stub!(:each_with_groupby_and_count).
-        and_yield(@person, @person.birthday.to_i, 1)
-      
-      @config = ThinkingSphinx::Configuration.instance
-      @config.configuration.searchd.max_matches = 10_000
-    end
-    
-    it "should use the system-set max_matches for limit on facet calls" do
-      ThinkingSphinx::Search.stub!(:search).and_return(@city_results, @birthday_results)
-      
-      ThinkingSphinx::Search.should_receive(:search) do |options|
-        options[:max_matches].should  == 10_000
-        options[:limit].should        == 10_000
-      end
-      
-      ThinkingSphinx::Search.facets :all_attributes => true
-    end
-    
-    it "should use the default max-matches if there is no explicit setting" do
-      ThinkingSphinx::Search.stub!(:search).and_return(@city_results, @birthday_results)
-      
-      @config.configuration.searchd.max_matches = nil
-      ThinkingSphinx::Search.should_receive(:search) do |options|
-        options[:max_matches].should  == 1000
-        options[:limit].should        == 1000
-      end
-      
-      ThinkingSphinx::Search.facets :all_attributes => true
-    end
-    
-    it "should ignore user-provided max_matches and limit on facet calls" do
-      ThinkingSphinx::Search.stub!(:search).and_return(@city_results, @birthday_results)
-      
-      ThinkingSphinx::Search.should_receive(:search) do |options|
-        options[:max_matches].should  == 10_000
-        options[:limit].should        == 10_000
-      end
-      
-      ThinkingSphinx::Search.facets(
-        :all_attributes => true,
-        :max_matches    => 500,
-        :limit          => 200
-      )
-    end
-    
-    it "should use explicit facet list if one is provided" do
-      ThinkingSphinx::Search.should_receive(:search).once.and_return(@city_results)
-      
-      ThinkingSphinx::Search.facets(
-        :facets => ['city'],
-        :class  => Person
-      )
-    end
-    
-    it "should not use an explicit :page" do
-      ThinkingSphinx::Search.stub!(:search).and_return(@city_results, @birthday_results)
-      ThinkingSphinx::Search.should_receive(:search) do |options|
-        options[:page].should == 1
-      end
-      
-      ThinkingSphinx::Search.facets(
-        :all_attributes => true,
-        :page           => 3
-      )
-    end
-    
-    describe "conflicting facets" do
-      before :each do
-        @index = ThinkingSphinx::Index::Builder.generate(Alpha) do
-          indexes :name
-          has :value, :as => :city, :facet => true
-        end
-      end
-      
-      after :each do
-        Alpha.sphinx_facets.delete_at(-1)
-        Alpha.sphinx_indexes.delete_at(-1)
-      end
-      
-      it "should raise an error if searching with facets of same name but different type" do
-        lambda {
-          ThinkingSphinx::Search.facets :all_attributes => true
-        }.should raise_error
-      end
-    end
-  end
-end
-
-describe ThinkingSphinx::Search, "playing nice with Search model" do
-  it "should not conflict with models called Search" do
-    lambda { Search.find(:all) }.should_not raise_error
-  end
-end