summaryrefslogtreecommitdiff
path: root/spec/models/customer_relations/contact_spec.rb
diff options
context:
space:
mode:
Diffstat (limited to 'spec/models/customer_relations/contact_spec.rb')
-rw-r--r--spec/models/customer_relations/contact_spec.rb45
1 files changed, 42 insertions, 3 deletions
diff --git a/spec/models/customer_relations/contact_spec.rb b/spec/models/customer_relations/contact_spec.rb
index 7e26d324ac2..1225f9d089b 100644
--- a/spec/models/customer_relations/contact_spec.rb
+++ b/spec/models/customer_relations/contact_spec.rb
@@ -26,6 +26,38 @@ RSpec.describe CustomerRelations::Contact, type: :model do
it_behaves_like 'an object with RFC3696 compliant email-formatted attributes', :email
end
+ describe '#unique_email_for_group_hierarchy' do
+ let_it_be(:parent) { create(:group) }
+ let_it_be(:group) { create(:group, parent: parent) }
+ let_it_be(:subgroup) { create(:group, parent: group) }
+
+ let_it_be(:existing_contact) { create(:contact, group: group) }
+
+ context 'with unique email for group hierarchy' do
+ subject { build(:contact, group: group) }
+
+ it { is_expected.to be_valid }
+ end
+
+ context 'with duplicate email in group' do
+ subject { build(:contact, email: existing_contact.email, group: group) }
+
+ it { is_expected.to be_invalid }
+ end
+
+ context 'with duplicate email in parent group' do
+ subject { build(:contact, email: existing_contact.email, group: subgroup) }
+
+ it { is_expected.to be_invalid }
+ end
+
+ context 'with duplicate email in subgroup' do
+ subject { build(:contact, email: existing_contact.email, group: parent) }
+
+ it { is_expected.to be_invalid }
+ end
+ end
+
describe '#before_validation' do
it 'strips leading and trailing whitespace' do
contact = described_class.new(first_name: ' First ', last_name: ' Last ', phone: ' 123456 ')
@@ -43,20 +75,27 @@ RSpec.describe CustomerRelations::Contact, type: :model do
let_it_be(:other_contacts) { create_list(:contact, 2) }
it 'returns ids of contacts from group' do
- contact_ids = described_class.find_ids_by_emails(group.id, group_contacts.pluck(:email))
+ contact_ids = described_class.find_ids_by_emails(group, group_contacts.pluck(:email))
+
+ expect(contact_ids).to match_array(group_contacts.pluck(:id))
+ end
+
+ it 'returns ids of contacts from parent group' do
+ subgroup = create(:group, parent: group)
+ contact_ids = described_class.find_ids_by_emails(subgroup, group_contacts.pluck(:email))
expect(contact_ids).to match_array(group_contacts.pluck(:id))
end
it 'does not return ids of contacts from other groups' do
- contact_ids = described_class.find_ids_by_emails(group.id, other_contacts.pluck(:email))
+ contact_ids = described_class.find_ids_by_emails(group, other_contacts.pluck(:email))
expect(contact_ids).to be_empty
end
it 'raises ArgumentError when called with too many emails' do
too_many_emails = described_class::MAX_PLUCK + 1
- expect { described_class.find_ids_by_emails(group.id, Array(0..too_many_emails)) }.to raise_error(ArgumentError)
+ expect { described_class.find_ids_by_emails(group, Array(0..too_many_emails)) }.to raise_error(ArgumentError)
end
end
end