Merge pull request #877 from uktrade/develop

Use ActiveJob#perform_later method

app/controllers/admin/enquiries_controller.rb

@@ -32,7 +32,7 @@ def index
       format.csv do
         enquiries = policy_scope(Enquiry).includes(:enquiry_response).where('enquiries.created_at >= ?', @enquiry_form.from).where('enquiries.created_at < ?', @enquiry_form.to).order('enquiries.created_at DESC')
         zip_file_enquiries_cutoff_env_var = Figaro.env.zip_file_enquiries_cutoff ? Figaro.env.zip_file_enquiries_cutoff!.to_i : 6000
-        SendEnquiriesReportToMatchingAdminUser.perform_async(current_editor.email, enquiries.pluck('enquiries.id'), @enquiry_form.from, @enquiry_form.to, zip_file_enquiries_cutoff_env_var) if @enquiry_form.dates?
+        SendEnquiriesReportToMatchingAdminUser.perform_later(current_editor.email, enquiries.pluck('enquiries.id'), @enquiry_form.from, @enquiry_form.to, zip_file_enquiries_cutoff_env_var) if @enquiry_form.dates?
         redirect_to admin_enquiries_path, notice: 'The Enquiries report has been emailed. If you have requested a large amount of data, the report will be sent as sections in separate emails.'
       end
     end

app/controllers/admin/reports_controller.rb

@@ -20,7 +20,7 @@ def index
       authorize :reports
 
       if params[:commit]
-        SendMonthlyReportToMatchingAdminUser.perform_async(current_editor.email, params)
+        SendMonthlyReportToMatchingAdminUser.perform_later(current_editor.email, params)
         redirect_to admin_reports_path, notice: 'The requested Monthly Outcome against Targets by Country report has been emailed to you.'
       end
 

app/services/update_opportunity_status.rb

@@ -64,6 +64,6 @@ def message
 
 class SubscriberNotificationSender
   def call(opportunity)
-    AddOpportunityToMatchingSubscriptionsQueueWorker.perform_async(opportunity.id)
+    AddOpportunityToMatchingSubscriptionsQueueWorker.perform_later(opportunity.id)
   end
 end

spec/workers/add_opportunity_to_matching_subscriptions_queue_worker_spec.rb

@@ -6,7 +6,7 @@
     create_list(:subscription, 2, search_term: 'matching')
 
     expect do
-      AddOpportunityToMatchingSubscriptionsQueueWorker.perform_async(opportunity.id)
+      AddOpportunityToMatchingSubscriptionsQueueWorker.perform_later(opportunity.id)
     end.to change { SubscriptionNotification.count }.by(2)
   end
 
@@ -16,7 +16,7 @@
     create_list(:subscription, 2, user: user, search_term: 'matching')
 
     expect do
-      AddOpportunityToMatchingSubscriptionsQueueWorker.perform_async(opportunity.id)
+      AddOpportunityToMatchingSubscriptionsQueueWorker.perform_later(opportunity.id)
     end.to change { SubscriptionNotification.count }.by(1)
   end
 
@@ -27,7 +27,7 @@
     create(:subscription, user: user, search_term: 'subscription')
 
     expect do
-      AddOpportunityToMatchingSubscriptionsQueueWorker.perform_async(opportunity.id)
+      AddOpportunityToMatchingSubscriptionsQueueWorker.perform_later(opportunity.id)
     end.to change { SubscriptionNotification.count }.by(1)
   end
 
@@ -37,7 +37,7 @@
       subscription = create(:subscription, search_term: 'matching')
 
       expect do
-        AddOpportunityToMatchingSubscriptionsQueueWorker.perform_async(opportunity.id)
+        AddOpportunityToMatchingSubscriptionsQueueWorker.perform_later(opportunity.id)
       end.to change { subscription.notifications.count }.by(1)
 
       expect(subscription.notifications.last.opportunity).to eq opportunity
@@ -53,7 +53,7 @@
       expect_any_instance_of(SubscriptionFinder).to receive(:call).and_return([first_subscription, second_subscription])
 
       expect do
-        AddOpportunityToMatchingSubscriptionsQueueWorker.perform_async(opportunity.id)
+        AddOpportunityToMatchingSubscriptionsQueueWorker.perform_later(opportunity.id)
       end.to change { SubscriptionNotification.count }.by(1)
     end
   end