Add #autosave to AssociationMatcher

lib/shoulda/matchers/active_record/association_matcher.rb

@@ -18,6 +18,8 @@ def belong_to(name)
       # * <tt>dependent</tt> - tests that the association makes use of the
       #   dependent option.
       # * <tt>:class_name</tt> - tests that the association makes use of the class_name option.
+      # * <tt>:autosave</tt> - tests that the association makes use of the
+      #   autosave option.
       #
       # Example:
       #   it { should have_many(:friends) }
@@ -36,6 +38,8 @@ def have_many(name)
       # * <tt>:dependent</tt> - tests that the association makes use of the
       #   dependent option.
       # * <tt>:class_name</tt> - tests that the association makes use of the class_name option.
+      # * <tt>:autosave</tt> - tests that the association makes use of the
+      #   autosave option.
       #
       # Example:
       #   it { should have_one(:god) } # unless hindu
@@ -80,6 +84,11 @@ def conditions(conditions)
           self
         end
 
+        def autosave(autosave)
+          @options[:autosave] = autosave
+          self
+        end
+
         def class_name(class_name)
           @options[:class_name] = class_name
           self
@@ -93,6 +102,7 @@ def matches?(subject)
             through_association_valid? &&
             dependent_correct? &&
             class_name_correct? &&
+            autosave_correct? &&
             order_correct? &&
             conditions_correct? &&
             join_table_exists?
@@ -112,6 +122,7 @@ def description
           description += " dependent => #{@options[:dependent]}"   if @options.key?(:dependent)
           description += " class_name => #{@options[:class_name]}" if @options.key?(:class_name)
           description += " order => #{@options[:order]}"           if @options.key?(:order)
+          description += " autosave => #{@options[:autosave]}"     if @options.key?(:autosave)
           description
         end
 
@@ -194,6 +205,19 @@ def class_name_correct?
           end
         end
 
+        def autosave_correct?
+          if @options.key?(:autosave)
+            if @options[:autosave].to_s == reflection.options[:autosave].to_s
+              true
+            else
+              @missing = "#{@name} should have #{@options[:autosave]} as autosave"
+              false
+            end
+          else
+            true
+          end
+        end
+
         def order_correct?
           if @options.key?(:order)
             if @options[:order].to_s == reflection.options[:order].to_s

spec/shoulda/active_record/association_matcher_spec.rb

@@ -98,6 +98,23 @@
       end
       Child.new.should_not @matcher.class_name('TreeChild')
     end
+
+    it "should accept an association with a valid :autosave option" do
+      define_model :parent, :adopter => :boolean
+      define_model :child, :parent_id => :integer do
+        belongs_to :parent, :autosave => true
+      end
+      Child.new.should @matcher.autosave(true)
+    end
+
+    it "should reject an association with a valid :autosave option" do
+      define_model :parent, :adopter => :boolean
+      define_model :child, :parent_id => :integer do
+        belongs_to :parent, :autosave => false
+      end
+      Child.new.should_not @matcher.autosave(true)
+    end
+
   end
 
   context "have_many" do
@@ -240,6 +257,22 @@
       Parent.new.should_not @matcher.class_name('Node')
     end
 
+    it "should accept an association with a valid :autosave option" do
+      define_model :child, :parent_id => :integer
+      define_model :parent do
+        has_many :children, :autosave => true
+      end
+      Parent.new.should @matcher.autosave(true)
+    end
+
+    it "should reject an association with a valid :autosave option" do
+      define_model :child, :parent_id => :integer
+      define_model :parent do
+        has_many :children, :autosave => false
+      end
+      Parent.new.should_not @matcher.autosave(true)
+    end
+
     it "should accept an association with a nonstandard reverse foreign key, using :inverse_of" do
       define_model :child, :ancestor_id => :integer do
         belongs_to :ancestor, :inverse_of => :children, :class_name => :Parent
@@ -364,6 +397,22 @@
       Person.new.should_not @matcher.class_name('PersonDetail')
     end
 
+    it "should accept an association with a valid :autosave option" do
+      define_model :detail, :person_id => :integer, :disabled => :boolean
+      define_model :person do
+        has_one :detail, :autosave => true
+      end
+      Person.new.should @matcher.autosave(true)
+    end
+
+    it "should reject an association with a valid :autosave option" do
+      define_model :detail, :person_id => :integer, :disabled => :boolean
+      define_model :person do
+        has_one :detail, :autosave => false
+      end
+      Person.new.should_not @matcher.autosave(true)
+    end
+
   end
 
   context "have_and_belong_to_many" do
@@ -445,5 +494,24 @@
       Person.new.should_not @matcher.class_name('PersonRelatives')
     end
 
+    it "should accept an association with a valid :autosave option" do
+      define_model :relatives, :adopted => :boolean
+      define_model :person do
+        has_and_belongs_to_many :relatives, :autosave => true
+      end
+      define_model :people_relative, :person_id   => :integer,
+                                     :relative_id => :integer
+      Person.new.should @matcher.autosave(true)
+    end
+
+    it "should reject an association with a bad :autosave option" do
+      define_model :relatives, :adopted => :boolean
+      define_model :person do
+        has_and_belongs_to_many :relatives
+      end
+      define_model :people_relative, :person_id   => :integer,
+                                     :relative_id => :integer
+      Person.new.should_not @matcher.autosave(true)
+    end
   end
 end