C2M assigns actual_version to AuditResults once known

sul-dlss · Jan 31, 2018 · 939451b · 939451b
1 parent b6fd28a
commit 939451b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/lib/audit/catalog_to_moab.rb b/lib/audit/catalog_to_moab.rb
@@ -72,6 +72,7 @@ def check_catalog_version
     end
 
     moab_version = moab.current_version_id
+    results.actual_version = moab_version
     catalog_version = preserved_copy.version
     if catalog_version == moab_version
       results.add_result(AuditResults::VERSION_MATCHES, preserved_copy.class.name)

diff --git a/spec/lib/audit/catalog_to_moab_instance_spec.rb b/spec/lib/audit/catalog_to_moab_instance_spec.rb
@@ -57,7 +57,7 @@
     end
 
     it 'calls POHandlerResults.report_results' do
-      pohandler_results = instance_double(AuditResults, add_result: nil)
+      pohandler_results = instance_double(AuditResults, add_result: nil, :actual_version= => nil)
       allow(AuditResults).to receive(:new).and_return(pohandler_results)
       expect(pohandler_results).to receive(:report_results)
       c2m.check_catalog_version
@@ -103,7 +103,7 @@
 
     context 'catalog version == moab version (happy path)' do
       it 'adds a VERSION_MATCHES result' do
-        pohandler_results = instance_double(AuditResults, report_results: nil)
+        pohandler_results = instance_double(AuditResults, report_results: nil, :actual_version= => nil)
         allow(AuditResults).to receive(:new).and_return(pohandler_results)
         expect(pohandler_results).to receive(:add_result).with(
           AuditResults::VERSION_MATCHES, 'PreservedCopy'
@@ -120,7 +120,7 @@
       end
 
       it 'adds an UNEXPECTED_VERSION result' do
-        pohandler_results = instance_double(AuditResults, report_results: nil)
+        pohandler_results = instance_double(AuditResults, report_results: nil, :actual_version= => nil)
         expect(pohandler_results).to receive(:add_result).with(
           AuditResults::UNEXPECTED_VERSION, 'PreservedCopy'
         )
@@ -144,7 +144,7 @@
       end
 
       it 'adds an UNEXPECTED_VERSION result' do
-        pohandler_results = instance_double(AuditResults, report_results: nil)
+        pohandler_results = instance_double(AuditResults, report_results: nil, :actual_version= => nil)
         expect(pohandler_results).to receive(:add_result).with(
           AuditResults::UNEXPECTED_VERSION, 'PreservedCopy'
         )
@@ -180,7 +180,7 @@
           expect(new_status).to eq PreservedCopy::INVALID_MOAB_STATUS
         end
         it 'adds an INVALID_MOAB result' do
-          pohandler_results = instance_double(AuditResults, report_results: nil)
+          pohandler_results = instance_double(AuditResults, report_results: nil, :actual_version= => nil)
           expect(pohandler_results).to receive(:add_result).with(
             AuditResults::INVALID_MOAB, anything
           )
@@ -190,7 +190,7 @@
         end
       end
       it 'adds a PC_STATUS_CHANGED result' do
-        pohandler_results = instance_double(AuditResults, report_results: nil)
+        pohandler_results = instance_double(AuditResults, report_results: nil, :actual_version= => nil)
         expect(pohandler_results).to receive(:add_result).with(
           AuditResults::PC_STATUS_CHANGED, a_hash_including(:old_status, :new_status)
         )