diff --git a/app/models/domain.rb b/app/models/domain.rb index dff0ecddc..c9c301bd2 100644 --- a/app/models/domain.rb +++ b/app/models/domain.rb @@ -737,7 +737,7 @@ class Domain < ActiveRecord::Base end def primary_contact_emails - admin_contact_emails << registrant_email + (admin_contact_emails << registrant_email).uniq end def new_registrant_email diff --git a/spec/models/domain_spec.rb b/spec/models/domain_spec.rb index ef70f9649..78ada6d51 100644 --- a/spec/models/domain_spec.rb +++ b/spec/models/domain_spec.rb @@ -794,13 +794,13 @@ RSpec.describe Domain, db: false do before :example do expect(domain).to receive(:registrant_email).and_return('registrant@test.com') - expect(domain).to receive(:admin_contact_emails).and_return(['admin.contact.email@test.com']) + expect(domain).to receive(:admin_contact_emails).and_return(%w(admin.contact@test.com admin.contact@test.com)) end - it 'returns registrant and administrative contact emails' do + it 'returns unique list of registrant and administrative contact emails' do expect(domain.primary_contact_emails).to match_array(%w( registrant@test.com - admin.contact.email@test.com + admin.contact@test.com )) end end