diff --git a/app/models/contact.rb b/app/models/contact.rb index 33cc3ff0f..f07c0c114 100644 --- a/app/models/contact.rb +++ b/app/models/contact.rb @@ -504,7 +504,8 @@ class Contact < ApplicationRecord def update_related_whois_records # not doing anything if no real changes - return if changes.slice(*(self.class.column_names - ["updated_at", "created_at", "statuses", "status_notes"])).empty? + ignored_columns = %w[updated_at created_at statuses status_notes] + return if saved_changes.slice(*(self.class.column_names - ignored_columns)).empty? names = related_domain_descriptions.keys UpdateWhoisRecordJob.enqueue(names, 'domain') if names.present? diff --git a/config/environments/test.rb b/config/environments/test.rb index 784b48f42..c55e59e31 100644 --- a/config/environments/test.rb +++ b/config/environments/test.rb @@ -1,6 +1,6 @@ Rails.application.configure do - $VERBOSE = nil + # $VERBOSE = nil # Settings specified here will take precedence over those in config/application.rb. # The test environment is used exclusively to run your application's diff --git a/test/models/contact_test.rb b/test/models/contact_test.rb index 923a8d44e..4e500e98d 100644 --- a/test/models/contact_test.rb +++ b/test/models/contact_test.rb @@ -248,6 +248,15 @@ class ContactTest < ActiveSupport::TestCase assert_equal %w[ok], contact.statuses end + def test_whois_gets_updated_after_contact_save + @contact.name = 'SomeReallyWeirdRandomTestName' + domain = @contact.registrant_domains.first + + @contact.save! + + assert_equal domain.whois_record.try(:json).try(:[], 'registrant'), @contact.name + end + private def make_contact_free_of_domains_where_it_acts_as_a_registrant(contact)