Browse Source

bulk upload send upload again email

pull/1274/head
Phil Lee 3 years ago
parent
commit
d62db99370
  1. 5
      app/services/bulk_upload/processor.rb
  2. 11
      spec/services/bulk_upload/processor_spec.rb

5
app/services/bulk_upload/processor.rb

@ -13,6 +13,7 @@ class BulkUpload::Processor
validator.call validator.call
create_logs if validator.create_logs? 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_fix_errors_mail if created_logs_but_incompleted?
send_success_mail if created_logs_and_all_completed? send_success_mail if created_logs_and_all_completed?
@ -25,6 +26,10 @@ class BulkUpload::Processor
private 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 def send_fix_errors_mail
BulkUploadMailer.send_bulk_upload_with_errors_mail(bulk_upload:).deliver_later BulkUploadMailer.send_bulk_upload_with_errors_mail(bulk_upload:).deliver_later
end end

11
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!) expect(mock_downloader).to have_received(:delete_local_file!)
end 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 it "does not send fix errors email" do
allow(BulkUploadMailer).to receive(:send_bulk_upload_with_errors_mail).and_call_original allow(BulkUploadMailer).to receive(:send_bulk_upload_with_errors_mail).and_call_original

Loading…
Cancel
Save