pduey-sunspot 1.2.1.1

data/spec/api/indexer/removal_spec.rb

@@ -0,0 +1,53 @@
+require File.expand_path('spec_helper', File.dirname(__FILE__))
+
+describe 'document removal', :type => :indexer do
+  it 'removes an object from the index' do
+    session.remove(post)
+    connection.should have_delete("Post #{post.id}")
+  end
+
+  it 'removes an object by type and id' do
+    session.remove_by_id(Post, 1)
+    connection.should have_delete('Post 1')
+  end
+
+  it 'removes an object by type and id and immediately commits' do
+    connection.should_receive(:delete_by_id).with(['Post 1']).ordered
+    connection.should_receive(:commit).ordered
+    session.remove_by_id!(Post, 1)
+  end
+
+  it 'removes an object from the index and immediately commits' do
+    connection.should_receive(:delete_by_id).ordered
+    connection.should_receive(:commit).ordered
+    session.remove!(post)
+  end
+
+  it 'removes everything from the index' do
+    session.remove_all
+    connection.should have_delete_by_query("*:*")
+  end
+
+  it 'removes everything from the index and immediately commits' do
+    connection.should_receive(:delete_by_query).ordered
+    connection.should_receive(:commit).ordered
+    session.remove_all!
+  end
+
+  it 'removes everything of a given class from the index' do
+    session.remove_all(Post)
+    connection.should have_delete_by_query("type:Post")
+  end
+
+  it 'correctly escapes namespaced classes when removing everything from the index' do
+    connection.should_receive(:delete_by_query).with('type:Namespaced\:\:Comment')
+    session.remove_all(Namespaced::Comment)
+  end
+
+  it 'should remove by query' do
+    session.remove(Post) do
+      with(:title, 'monkeys')
+    end
+    connection.should have_delete_by_query("(type:Post AND title_ss:monkeys)")
+  end
+end

data/spec/api/indexer/spec_helper.rb

@@ -0,0 +1 @@
+require File.expand_path('spec_helper', File.join(File.dirname(__FILE__), '..'))

data/spec/api/indexer_spec.rb

@@ -0,0 +1,14 @@
+require File.expand_path('spec_helper', File.dirname(__FILE__))
+
+describe 'indexer', :type => :indexer do
+  it 'should completely wipe setup if class redefined (reloaded)' do
+    Object::ReloadableClass = Class.new(MockRecord)
+    Sunspot.setup(ReloadableClass) { string(:title) }
+    Object.class_eval { remove_const(:ReloadableClass) }
+    Object::ReloadableClass = Class.new(MockRecord)
+    Sunspot.setup(ReloadableClass) {}
+    lambda do
+      Sunspot.search(ReloadableClass) { with(:title, 'title') }
+    end.should raise_error(Sunspot::UnrecognizedFieldError)
+  end
+end

data/spec/api/query/advanced_manipulation_examples.rb

@@ -0,0 +1,35 @@
+require File.expand_path('spec_helper', File.dirname(__FILE__))
+
+shared_examples_for "query with advanced manipulation" do
+  describe 'adjust_solr_params' do
+    before :each do
+      search do
+        adjust_solr_params do |params|
+          params[:rows] = 40
+          params[:qt] = 'complicated'
+        end
+      end
+    end
+
+    it "modifies existing param" do
+      connection.should have_last_search_with(:rows  => 40)
+    end
+
+    it "adds new param" do
+      connection.should have_last_search_with(:qt => 'complicated')
+    end
+  end
+
+  describe 'request_handler' do
+    before :each do
+      connection.expected_handler = :myRequestHandler
+      search do
+        request_handler :myRequestHandler
+      end
+    end
+
+    it 'should use specified request handler' do
+      connection.should have_last_search_with({})
+    end
+  end
+end

data/spec/api/query/connectives_examples.rb

@@ -0,0 +1,189 @@
+shared_examples_for "query with connective scope" do
+  it 'creates a disjunction between two restrictions' do
+    search do
+      any_of do
+        with :category_ids, 1
+        with :blog_id, 2
+      end
+    end
+    connection.should have_last_search_including(
+      :fq, '(category_ids_im:1 OR blog_id_i:2)'
+    )
+  end
+
+  it 'creates a conjunction inside of a disjunction' do
+    search do
+      any_of do
+        with :blog_id, 2
+        all_of do
+          with :category_ids, 1
+          with(:average_rating).greater_than(3.0)
+        end
+      end
+    end
+    connection.should have_last_search_including(
+      :fq,
+      '(blog_id_i:2 OR (category_ids_im:1 AND average_rating_ft:[3\.0 TO *]))'
+    )
+  end
+
+  it 'creates a disjunction with nested conjunction with negated restrictions' do
+    search do
+      any_of do
+        with :category_ids, 1
+        all_of do
+          without(:average_rating).greater_than(3.0)
+          with(:blog_id, 1)
+        end
+      end
+    end
+    connection.should have_last_search_including(
+      :fq, '(category_ids_im:1 OR (-average_rating_ft:[3\.0 TO *] AND blog_id_i:1))'
+    )
+  end
+
+  it 'does nothing special if #all_of called from the top level' do
+    search do
+      all_of do
+        with :blog_id, 2
+        with :category_ids, 1
+      end
+    end
+    connection.should have_last_search_including(
+      :fq, 'blog_id_i:2', 'category_ids_im:1'
+    )
+  end
+
+  it 'creates a disjunction with negated restrictions' do
+    search do
+      any_of do
+        with :category_ids, 1
+        without(:average_rating).greater_than(3.0)
+      end
+    end
+    connection.should have_last_search_including(
+      :fq, '-(-category_ids_im:1 AND average_rating_ft:[3\.0 TO *])'
+    )
+  end
+
+  it 'creates a disjunction with a negated restriction and a nested disjunction in a conjunction with a negated restriction' do
+    search do
+      any_of do
+        without(:title, 'Yes')
+        all_of do
+          with(:blog_id, 1)
+          any_of do
+            with(:category_ids, 4)
+            without(:average_rating, 2.0)
+          end
+        end
+      end
+    end
+    connection.should have_last_search_including(
+      :fq, '-(title_ss:Yes AND -(blog_id_i:1 AND -(-category_ids_im:4 AND average_rating_ft:2\.0)))'
+    )
+  end
+  it 'creates a disjunction with nested conjunction with nested disjunction with negated restriction' do
+    search do
+      any_of do
+        with(:title, 'Yes')
+        all_of do
+          with(:blog_id, 1)
+          any_of do
+            with(:category_ids, 4)
+            without(:average_rating, 2.0)
+          end
+        end
+      end
+    end
+    connection.should have_last_search_including(
+      :fq, '(title_ss:Yes OR (blog_id_i:1 AND -(-category_ids_im:4 AND average_rating_ft:2\.0)))'
+    )
+  end
+
+  #
+  # This is important because if a disjunction could be nested in another
+  # disjunction, then the inner disjunction could denormalize (and thus
+  # become negated) after the outer disjunction denormalized (checking to
+  # see if the inner one is negated). Since conjunctions never need to
+  # denormalize, if a disjunction can only contain conjunctions or restrictions,
+  # we can guarantee that the negation state of a disjunction's components will
+  # not change when #to_params is called on them.
+  #
+  # Since disjunction is associative, this behavior has no effect on the actual
+  # logical semantics of the disjunction.
+  #
+  it 'creates a single disjunction when disjunctions nested' do
+    search do
+      any_of do
+        with(:title, 'Yes')
+        any_of do
+          with(:blog_id, 1)
+          with(:category_ids, 4)
+        end
+      end
+    end
+    connection.should have_last_search_including(
+      :fq, '(title_ss:Yes OR blog_id_i:1 OR category_ids_im:4)'
+    )
+  end
+
+  it 'creates a disjunction with instance exclusion' do
+    post = Post.new
+    search do
+      any_of do
+        without(post)
+        with(:category_ids, 1)
+      end
+    end
+    connection.should have_last_search_including(
+      :fq, "-(id:(Post\\ #{post.id}) AND -category_ids_im:1)"
+    )
+  end
+
+  it 'creates a disjunction with empty restriction' do
+    search do
+      any_of do
+        with(:average_rating, nil)
+        with(:average_rating).greater_than(3.0)
+      end
+    end
+    connection.should have_last_search_including(
+      :fq, '-(average_rating_ft:[* TO *] AND -average_rating_ft:[3\.0 TO *])'
+    )
+  end
+
+  it 'creates a disjunction with instance inclusion' do
+    post = Post.new
+    search do
+      any_of do
+        with(post)
+        with(:average_rating).greater_than(3.0)
+      end
+    end
+    connection.should have_last_search_including(
+      :fq, "(id:(Post\\ #{post.id}) OR average_rating_ft:[3\\.0 TO *])"
+    )
+  end
+
+  it 'creates a disjunction with some text field components' do
+    search do
+      any_of do
+        text_fields do
+          with(:title).starting_with('test')
+        end
+        with(:blog_id, 1)
+      end
+    end
+    connection.should have_last_search_including(
+      :fq, '(title_text:test* OR blog_id_i:1)'
+    )
+  end
+
+  it 'should ignore empty connectives' do
+    search do
+      any_of {}
+    end
+    connection.should_not have_last_search_including(:fq, '')
+  end
+end

data/spec/api/query/dsl_spec.rb

@@ -0,0 +1,18 @@
+require File.expand_path('spec_helper', File.dirname(__FILE__))
+
+describe 'query DSL', :type => :query do
+  it 'should allow building search using block argument rather than instance_eval' do
+    @blog_id = 1
+    session.search Post do |query|
+      query.with(:blog_id, @blog_id)
+    end
+    connection.should have_last_search_including(:fq, 'blog_id_i:1')
+  end
+
+  it 'should accept a block in the #new_search method' do
+    search = session.new_search(Post) { with(:blog_id, 1) }
+    search.execute
+    connection.should have_last_search_including(:fq, 'blog_id_i:1')
+  end
+end
+

data/spec/api/query/dynamic_fields_examples.rb

@@ -0,0 +1,165 @@
+shared_examples_for "query with dynamic field support" do
+  it 'restricts by dynamic string field with equality restriction' do
+    search do
+      dynamic :custom_string do
+        with :test, 'string'
+      end
+    end
+    connection.should have_last_search_including(:fq, 'custom_string\:test_ss:string')
+  end
+
+  it 'restricts by dynamic integer field with less than restriction' do
+    search do
+      dynamic :custom_integer do
+        with(:test).less_than(1)
+      end
+    end
+    connection.should have_last_search_including(:fq, 'custom_integer\:test_i:[* TO 1]')
+  end
+
+  it 'restricts by dynamic float field with between restriction' do
+    search do
+      dynamic :custom_float do
+        with(:test).between(2.2..3.3)
+      end
+    end
+    connection.should have_last_search_including(:fq, 'custom_float\:test_fm:[2\.2 TO 3\.3]')
+  end
+
+  it 'restricts by dynamic time field with any of restriction' do
+    search do
+      dynamic :custom_time do
+        with(:test).any_of([Time.parse('2009-02-10 14:00:00 UTC'),
+                            Time.parse('2009-02-13 18:00:00 UTC')])
+      end
+    end
+    connection.should have_last_search_including(:fq, 'custom_time\:test_d:(2009\-02\-10T14\:00\:00Z OR 2009\-02\-13T18\:00\:00Z)')
+  end
+
+  it 'restricts by dynamic boolean field with equality restriction' do
+    search do
+      dynamic :custom_boolean do
+        with :test, false
+      end
+    end
+    connection.should have_last_search_including(:fq, 'custom_boolean\:test_b:false')
+  end
+
+  it 'negates a dynamic field restriction' do
+    search do
+      dynamic :custom_string do
+        without :test, 'foo'
+      end
+    end
+    connection.should have_last_search_including(:fq, '-custom_string\:test_ss:foo')
+  end
+
+  it 'scopes by a dynamic field inside a disjunction' do
+    search do
+      any_of do
+        dynamic :custom_string do
+          with :test, 'foo'
+        end
+        with :title, 'bar'
+      end
+    end
+    connection.should have_last_search_including(
+      :fq, '(custom_string\:test_ss:foo OR title_ss:bar)'
+    )
+  end
+
+  it 'orders by a dynamic field' do
+    search do
+      dynamic :custom_integer do
+        order_by :test, :desc
+      end
+    end
+    connection.should have_last_search_with(:sort => 'custom_integer:test_i desc')
+  end
+
+  it 'orders by a dynamic field and static field, with given precedence' do
+    search do
+      dynamic :custom_integer do
+        order_by :test, :desc
+      end
+      order_by :sort_title, :asc
+    end
+    connection.should have_last_search_with(:sort => 'custom_integer:test_i desc, sort_title_s asc')
+  end
+
+  it 'raises an UnrecognizedFieldError if an unknown dynamic field is searched by' do
+    lambda do
+      search do
+        dynamic(:bogus) { with :some, 'value' }
+      end
+    end.should raise_error(Sunspot::UnrecognizedFieldError)
+  end
+
+  it 'raises a NoMethodError if pagination is attempted in a dynamic query' do
+    lambda do
+      search do
+        dynamic :custom_string do
+          paginate :page => 3, :per_page => 10
+        end
+      end
+    end.should raise_error(NoMethodError)
+  end
+
+  it 'requests field facet on dynamic field' do
+    search do
+      dynamic :custom_string do
+        facet(:test)
+      end
+    end
+    connection.should have_last_search_including(:"facet.field", 'custom_string:test_ss')
+  end
+
+  it 'requests named field facet on dynamic field' do
+    search do
+      dynamic :custom_string do
+        facet(:test, :name => :bogus)
+      end
+    end
+    connection.should have_last_search_including(:"facet.field", '{!key=bogus}custom_string:test_ss')
+  end
+
+  it 'requests query facet with internal dynamic field' do
+    search do
+      facet :test do
+        row 'foo' do
+          dynamic :custom_string do
+            with :test, 'foo'
+          end
+        end
+      end
+    end
+    connection.should have_last_search_with(
+      :"facet.query" => 'custom_string\:test_ss:foo'
+    )
+  end
+
+  it 'requests query facet with external dynamic field' do
+    search do
+      dynamic :custom_string do
+        facet :test do
+          row 'foo' do
+            with :test, 'foo'
+          end
+        end
+      end
+    end
+    connection.should have_last_search_including(
+      :"facet.query",
+      'custom_string\:test_ss:foo'
+    )
+  end
+
+  it 'allows scoping on dynamic fields common to all types' do
+    search Post, Namespaced::Comment do
+      dynamic :custom_float do
+        with(:test, 1.23)
+      end
+    end
+    connection.should have_last_search_including(:fq, 'custom_float\\:test_fm:1\\.23')
+  end
+end