sunspot_submodel_index 0.0.2 → 0.0.3

data/README.rdoc

@@ -45,7 +45,7 @@ Examples
 
 === If Option
 
-There is also the option to call a Proc before parent is indexed.  The Proc is passed the instance of the object.  If the Proc returns false the parent is not indexed.  
+There is also the option to call a Proc before parent is indexed.  The Proc is passed the instance of the object.  If the Proc returns false the parent is not indexed. Note, that since the Proc is called on destroy of the object you should write to code to ensure it will not throw and exception in that case.  
 
 Example
 

data/lib/sunspot_submodel_index/submodel_index.rb

@@ -97,6 +97,7 @@ module Sunspot
       
       #always call reindex on destroy
       def parent_solr_index_on_destroy
+        return unless check_parent_solr_if_statement
         call_parent_solr_index
       end
 

data/lib/sunspot_submodel_index/version.rb

@@ -1,3 +1,3 @@
 module SunspotSubmodelIndex
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 end

data/spec/submodel_index_spec.rb

@@ -2,6 +2,7 @@ require File.expand_path('../spec_helper', __FILE__)
 
 def stub_object_no_index(new_record = false)
   obj = SunspotSubmodelTestModel.new
+  obj.class._sunspot_submodel_options[:if] = nil
   obj.class._sunspot_submodel_options[:included_attributes] = false
   obj.class._sunspot_submodel_options[:ignored_attributes] = false
   obj.stubs(:new_record?).returns(new_record)
@@ -28,6 +29,7 @@ end
 
 def stub_object_will_index(new_record = false)
   obj = SunspotSubmodelTestModel.new
+  obj.class._sunspot_submodel_options[:if] = nil
   obj.class._sunspot_submodel_options[:included_attributes] = false
   obj.class._sunspot_submodel_options[:ignored_attributes] = false
   obj.stubs(:new_record?).returns(new_record)
@@ -151,9 +153,18 @@ describe Sunspot::SubmodelIndex do
        obj = stub_object_will_index
        obj.destroy
      end
+     it "should not call parents index if the if proc fails" do
+       obj = stub_object_no_index
+       obj.expects(:check_parent_solr_if_statement).returns false
+       obj.destroy
+     end
+     it "should call parents index if the if proc is ok" do
+       obj = stub_object_will_index
+       obj.expects(:check_parent_solr_if_statement).returns true
+       obj.destroy
+     end
    end
    
-   
    context "force association reload" do
      it "should call parent with true with option is set" do
        obj = SunspotSubmodelTestModel.new

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: sunspot_submodel_index
 version: !ruby/object:Gem::Version 
-  hash: 27
+  hash: 25
   prerelease: 
   segments: 
   - 0
   - 0
-  - 2
-  version: 0.0.2
+  - 3
+  version: 0.0.3
 platform: ruby
 authors: 
 - Scott Diedrick
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-08-16 00:00:00 -05:00
+date: 2011-09-19 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency