Move DomainDeleteConfirm job to interactor

app/interactions/domain_delete_confirm_interaction/send_request.rb

@@ -0,0 +1,22 @@
+module DomainDeleteConfirmInteraction
+  class SendRequest < ActiveInteraction::Base
+    object :domain,
+           class: Domain,
+           description: 'Domain to send delete confirmation'
+
+    def execute
+      log
+      DomainDeleteMailer.confirmation_request(domain: domain,
+                                              registrar: domain.registrar,
+                                              registrant: domain.registrant).deliver_later
+    end
+
+    private
+
+    def log
+      message = "Send DomainDeleteMailer#confirm email for domain #{domain.name} (##{domain.id})" \
+    " to #{domain.registrant.email}"
+      Rails.logger.info(message)
+    end
+  end
+end

app/models/domain.rb

@@ -418,7 +418,7 @@ def pending_delete!
     pending_delete_confirmation!
     save(validate: false) # should check if this did succeed
 
-    DomainDeleteConfirmEmailJob.enqueue(id)
+    DomainDeleteConfirmInteraction::SendRequest.run(domain: self)
   end
 
   def cancel_pending_delete

test/integration/epp/domain/delete/base_test.rb

@@ -35,7 +35,6 @@ def test_bypasses_domain_and_registrant_and_contacts_validation
     XML
 
     post epp_delete_path, params: { frame: request_xml }, headers: { 'HTTP_COOKIE' => 'session=api_bestnames' }
-    # binding.pry
     assert_includes Domain.find_by(name: 'invalid.test').statuses, DomainStatus::PENDING_DELETE_CONFIRMATION
     assert_epp_response :completed_successfully_action_pending
   end
@@ -90,7 +89,9 @@ def test_requests_registrant_confirmation_when_required
       </epp>
     XML
 
-    post epp_delete_path, params: { frame: request_xml }, headers: { 'HTTP_COOKIE' => 'session=api_bestnames' }
+    perform_enqueued_jobs do
+      post epp_delete_path, params: { frame: request_xml }, headers: { 'HTTP_COOKIE' => 'session=api_bestnames' }
+    end
     @domain.reload
 
     assert @domain.registrant_verification_asked?
@@ -121,7 +122,9 @@ def test_skips_registrant_confirmation_when_not_required
       </epp>
     XML
 
-    post epp_delete_path, params: { frame: request_xml }, headers: { 'HTTP_COOKIE' => 'session=api_bestnames' }
+    perform_enqueued_jobs do
+      post epp_delete_path, params: { frame: request_xml }, headers: { 'HTTP_COOKIE' => 'session=api_bestnames' }
+    end
     @domain.reload
 
     assert_not @domain.registrant_verification_asked?
@@ -152,7 +155,9 @@ def test_skips_registrant_confirmation_when_required_but_already_verified_by_reg
       </epp>
     XML
 
-    post epp_delete_path, params: { frame: request_xml }, headers: { 'HTTP_COOKIE' => 'session=api_bestnames' }
+    perform_enqueued_jobs do
+      post epp_delete_path, params: { frame: request_xml }, headers: { 'HTTP_COOKIE' => 'session=api_bestnames' }
+    end
     @domain.reload
 
     assert_not @domain.registrant_verification_asked?

test/models/whois/record_test.rb

@@ -70,6 +70,6 @@ def test_updates_whois_record_from_auction_when_payment_received
   end
 
   def registration_deadline
-    Time.zone.now + 10.days
+    @registration_deadline ||= Time.zone.now + 10.days
   end
 end