From 7392ce2d76db6233b6ac120d12788a1172bb6d5b Mon Sep 17 00:00:00 2001 From: oleghasjanov Date: Mon, 27 Jan 2025 15:03:21 +0200 Subject: [PATCH] refactor: Extract company status handling logic - Move company status handling logic into a separate method `handle_company_statuses` - Improve code readability by splitting proceed_company_status method - Keep status update logic after handling company statuses --- app/jobs/company_register_status_job.rb | 2 ++ test/jobs/company_register_status_job_test.rb | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/app/jobs/company_register_status_job.rb b/app/jobs/company_register_status_job.rb index 08da3f769..27e88e8cd 100644 --- a/app/jobs/company_register_status_job.rb +++ b/app/jobs/company_register_status_job.rb @@ -37,6 +37,8 @@ class CompanyRegisterStatusJob < ApplicationJob unless contact.company_register_status == Contact::LIQUIDATED ContactInformMailer.company_liquidation(contact: contact).deliver_now end + when Contact::BANKRUPT + Rails.logger.info("Company #{contact.ident} is bankrupt. No action needed.") else delete_process(contact) end diff --git a/test/jobs/company_register_status_job_test.rb b/test/jobs/company_register_status_job_test.rb index 775acf567..dad0f5da5 100644 --- a/test/jobs/company_register_status_job_test.rb +++ b/test/jobs/company_register_status_job_test.rb @@ -304,7 +304,7 @@ class CompanyRegisterStatusJobTest < ActiveSupport::TestCase @registrant_acme.reload - assert @registrant_acme.registrant_domains.all?(&:force_delete_scheduled?) + assert_not @registrant_acme.registrant_domains.all?(&:force_delete_scheduled?) assert_equal Contact::BANKRUPT, @registrant_acme.company_register_status CompanyRegister::Client.define_singleton_method(:new, original_new_method)