From d62db9937060434d8165db82d4404b2e8888019a Mon Sep 17 00:00:00 2001 From: Phil Lee Date: Tue, 7 Feb 2023 16:39:29 +0000 Subject: [PATCH] bulk upload send upload again email --- app/services/bulk_upload/processor.rb | 5 +++++ spec/services/bulk_upload/processor_spec.rb | 11 +++++++++++ 2 files changed, 16 insertions(+) diff --git a/app/services/bulk_upload/processor.rb b/app/services/bulk_upload/processor.rb index d69f76243..bbf7e6cd1 100644 --- a/app/services/bulk_upload/processor.rb +++ b/app/services/bulk_upload/processor.rb @@ -13,6 +13,7 @@ class BulkUpload::Processor validator.call create_logs if validator.create_logs? + send_correct_and_upload_again_mail unless validator.create_logs? send_fix_errors_mail if created_logs_but_incompleted? send_success_mail if created_logs_and_all_completed? @@ -25,6 +26,10 @@ class BulkUpload::Processor private + def send_correct_and_upload_again_mail + BulkUploadMailer.send_correct_and_upload_again_mail(bulk_upload:).deliver_later + end + def send_fix_errors_mail BulkUploadMailer.send_bulk_upload_with_errors_mail(bulk_upload:).deliver_later end diff --git a/spec/services/bulk_upload/processor_spec.rb b/spec/services/bulk_upload/processor_spec.rb index b751e7dd7..d56b2ff4e 100644 --- a/spec/services/bulk_upload/processor_spec.rb +++ b/spec/services/bulk_upload/processor_spec.rb @@ -171,6 +171,17 @@ RSpec.describe BulkUpload::Processor do expect(mock_downloader).to have_received(:delete_local_file!) end + it "sends correct and upload again mail" do + mail_double = instance_double("ActionMailer::MessageDelivery", deliver_later: nil) + + allow(BulkUploadMailer).to receive(:send_correct_and_upload_again_mail).and_return(mail_double) + + processor.call + + expect(BulkUploadMailer).to have_received(:send_correct_and_upload_again_mail) + expect(mail_double).to have_received(:deliver_later) + end + it "does not send fix errors email" do allow(BulkUploadMailer).to receive(:send_bulk_upload_with_errors_mail).and_call_original