Merge pull request #1544 from pod4lib/1541-handling-compacted

Do not try to extract records from compacted uploads

Author: cbeer

app/jobs/generate_full_dump_job.rb

@@ -39,7 +39,7 @@ def perform(stream, effective_date: Time.zone.now, publish: true)
 
     GenerateDeltaDumpJob.perform_now(stream, publish: publish) if stream.current_full_dump
 
-    uploads.where.not(status: 'processed').find_each do |upload|
+    uploads.where.not(status: %w[processed compacted]).find_each do |upload|
       ExtractMarcRecordMetadataJob.perform_now(upload)
     end
 

spec/jobs/generate_full_dump_job_spec.rb

@@ -77,6 +77,15 @@
   end
   # rubocop:enable RSpec/ExampleLength
 
+  it 'does not attempt to extract records from compacted uploads' do
+    described_class.perform_now(organization.default_stream)
+    CompactUploadsJob.perform_now(organization.default_stream, age: 0.seconds, min_uploads: 0)
+
+    expect do
+      described_class.perform_now(organization.default_stream)
+    end.not_to change(MarcRecord, :count)
+  end
+
   it 'has a content type of application/gzip for compressed marcxml' do
     described_class.perform_now(organization.default_stream)