diff --git a/app/models/concerns/domain/registry_lockable.rb b/app/models/concerns/domain/registry_lockable.rb index ccb0f305e..2325e4b60 100644 --- a/app/models/concerns/domain/registry_lockable.rb +++ b/app/models/concerns/domain/registry_lockable.rb @@ -12,7 +12,7 @@ module Concerns statuses << DomainStatus::SERVER_DELETE_PROHIBITED statuses << DomainStatus::SERVER_TRANSFER_PROHIBITED self.locked_by_registrant_at = Time.zone.now - alert_registrar_lock_changes! + alert_registrar_lock_changes!(lock: true) save! end @@ -43,14 +43,14 @@ module Concerns statuses.delete(DomainStatus::SERVER_DELETE_PROHIBITED) statuses.delete(DomainStatus::SERVER_TRANSFER_PROHIBITED) self.locked_by_registrant_at = nil - alert_registrar_lock_changes! + alert_registrar_lock_changes!(lock: false) save! end end - def alert_registrar_lock_changes! - translation = locked_by_registrant? ? 'locked' : 'unlocked' + def alert_registrar_lock_changes!(lock: true) + translation = lock ? 'locked' : 'unlocked' registrar.notifications.create!( text: I18n.t("notifications.texts.registrar_#{translation}", domain_name: name), diff --git a/test/jobs/domain_update_confirm_job_test.rb b/test/jobs/domain_update_confirm_job_test.rb index dc6e14bf9..d2d3a3252 100644 --- a/test/jobs/domain_update_confirm_job_test.rb +++ b/test/jobs/domain_update_confirm_job_test.rb @@ -22,7 +22,7 @@ class DomainUpdateConfirmJobTest < ActiveSupport::TestCase refute @domain.locked_by_registrant? @domain.apply_registry_lock assert @domain.locked_by_registrant? - assert_equal(@domain.registrar.notifications.last.text, "Domain #{@domain.name} has been unlocked by registrant") + assert_equal(@domain.registrar.notifications.last.text, "Domain #{@domain.name} has been locked by registrant") end def test_registrant_unlocked_domain