Implement test to ensure except option jibes with comment_required op…

…tion

- collectiveidea#418

spec/audited/auditor_spec.rb

@@ -800,6 +800,7 @@ def stub_global_max_audits(max_audits)
       let( :user ) { Models::ActiveRecord::CommentRequiredUser.create!( audit_comment: 'Create' ) }
       let( :on_create_user ) { Models::ActiveRecord::OnDestroyCommentRequiredUser.create }
       let( :on_destroy_user ) { Models::ActiveRecord::OnDestroyCommentRequiredUser.create }
+      let( :on_update_except_name_user ) { Models::ActiveRecord::OnUpdateCommentRequiredExceptNameUser.create }
 
       it "should not validate when audit_comment is not supplied" do
         expect(user.update_attributes(name: 'Test')).to eq(false)
@@ -810,6 +811,10 @@ def stub_global_max_audits(max_audits)
         expect(on_destroy_user.update_attributes(name: 'Test')).to eq(true)
       end
 
+      it "should validate when audit_comment is not supplied, and only excepted attributes have been updated" do
+        expect(on_update_except_name_user.update_attributes(name: 'Test')).to eq(true)
+      end
+
       it "should validate when audit_comment is supplied" do
         expect(user.update_attributes(name: 'Test', audit_comment: 'Update')).to eq(true)
       end

spec/support/active_record/models.rb

@@ -44,6 +44,11 @@ class OnDestroyCommentRequiredUser < ::ActiveRecord::Base
       audited comment_required: true, on: :destroy
     end
 
+    class OnUpdateCommentRequiredExceptNameUser < ::ActiveRecord::Base
+      self.table_name = :users
+      audited comment_required: true, except: :name, on: :update
+    end
+
     class AccessibleAfterDeclarationUser < ::ActiveRecord::Base
       self.table_name = :users
       audited