Extract CancelAeonAppointmentJob to handle the requests cleanup.

Author: cbeer

app/controllers/aeon_appointments_controller.rb

@@ -81,15 +81,10 @@ def update # rubocop:disable Metrics/AbcSize
     redirect_to aeon_appointments_path, notice: 'Appointment created successfully'
   end
 
-  def destroy # rubocop:disable Metrics/AbcSize
+  def destroy
     authorize! :delete, @appointment
 
-    @appointment.requests.each do |request|
-      aeon_client.update_request(request.transaction_number, AeonClient::DeleteAppointmentRequestData.new)
-      aeon_client.update_request_route(transaction_number: request.transaction_number,
-                                       status: Settings.aeon.queue_names.draft.transaction.first)
-    end
-    aeon_client.cancel_appointment(params[:id])
+    CancelAeonAppointmentJob.perform_now(@appointment)
 
     redirect_to aeon_appointments_path, notice: 'Appointment cancelled successfully'
   end

app/jobs/cancel_aeon_appointment_job.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+##
+# Rails Job to submit a hold request to Folio for processing
+class CancelAeonAppointmentJob < ApplicationJob
+  queue_as :default
+
+  def perform(appointment)
+    appointment.requests.each do |request|
+      move_request_to_draft(request)
+    end
+    aeon_client.cancel_appointment(appointment.id)
+  end
+
+  # When an appointment is cancelled, Aeon cancels the requests by default. We want to move them to draft
+  # and dis-associated the appointment instead.
+  def move_request_to_draft(request)
+    aeon_client.update_request(request.transaction_number, AeonClient::DeleteAppointmentRequestData.new)
+    aeon_client.update_request_route(transaction_number: request.transaction_number,
+                                     status: Settings.aeon.queue_names.draft.transaction.first)
+  end
+
+  def aeon_client
+    @aeon_client ||= AeonClient.new
+  end
+end