Merge 76a7c11 into 07c30ac

app/models/complete_moab.rb

@@ -17,6 +17,7 @@ class CompleteMoab < ApplicationRecord
 
   after_create :create_zipped_moab_versions!
   after_update :create_zipped_moab_versions!, if: :saved_change_to_version? # an ActiveRecord dynamic method
+  after_save :validate_checksums!, if: :status_change?
 
   belongs_to :preserved_object, inverse_of: :complete_moabs
   belongs_to :moab_storage_root, inverse_of: :complete_moabs
@@ -79,6 +80,10 @@ def validate_checksums!
     ChecksumValidationJob.perform_later(self)
   end
 
+  def status_change?
+    saved_change_to_status? && validity_unknown?
+  end
+
   def druid_version_zip
     @druid_version_zip ||= DruidVersionZip.new(preserved_object.druid, version)
   end

spec/models/complete_moab_spec.rb

@@ -380,4 +380,19 @@
       cm.save!
     end
   end
+
+  describe '.after_save callback' do
+    it 'does not call validate_checksums when status is unchanged' do
+      cm.size = 234
+      expect(cm).not_to receive(:validate_checksums!)
+      cm.save!
+    end
+
+    it 'does calls validate_checksums when status is validity_unknown' do
+      cm.ok! # object starts out with validity_unknown status
+      cm.status = 'validity_unknown'
+      expect(cm).to receive(:validate_checksums!)
+      cm.save!
+    end
+  end
 end