6a6a37a67b
fixes: #3853 - Introduced DISABLE_GRAVATAR Global Config, which will stop chatwoot from making API requests to gravatar - Cleaned up avatar-related logic and centralized it into the avatarable concern - Added specs for the missing cases - Added migration for existing installations to move the avatar to attachment, rather than making the API that results in 404.
78 lines
2.4 KiB
Ruby
78 lines
2.4 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require 'rails_helper'
|
|
|
|
require Rails.root.join 'spec/models/concerns/avatarable_shared.rb'
|
|
|
|
RSpec.describe Contact do
|
|
context 'validations' do
|
|
it { is_expected.to validate_presence_of(:account_id) }
|
|
end
|
|
|
|
context 'associations' do
|
|
it { is_expected.to belong_to(:account) }
|
|
it { is_expected.to have_many(:conversations).dependent(:destroy_async) }
|
|
end
|
|
|
|
describe 'concerns' do
|
|
it_behaves_like 'avatarable'
|
|
end
|
|
|
|
context 'prepare contact attributes before validation' do
|
|
it 'sets email to lowercase' do
|
|
contact = create(:contact, email: 'Test@test.com')
|
|
expect(contact.email).to eq('test@test.com')
|
|
end
|
|
|
|
it 'sets email to nil when empty string' do
|
|
contact = create(:contact, email: '')
|
|
expect(contact.email).to be_nil
|
|
end
|
|
|
|
it 'sets custom_attributes to {} when nil' do
|
|
contact = create(:contact, custom_attributes: nil)
|
|
expect(contact.custom_attributes).to eq({})
|
|
end
|
|
|
|
it 'sets custom_attributes to {} when empty string' do
|
|
contact = create(:contact, custom_attributes: '')
|
|
expect(contact.custom_attributes).to eq({})
|
|
end
|
|
|
|
it 'sets additional_attributes to {} when nil' do
|
|
contact = create(:contact, additional_attributes: nil)
|
|
expect(contact.additional_attributes).to eq({})
|
|
end
|
|
|
|
it 'sets additional_attributes to {} when empty string' do
|
|
contact = create(:contact, additional_attributes: '')
|
|
expect(contact.additional_attributes).to eq({})
|
|
end
|
|
end
|
|
|
|
context 'when phone number format' do
|
|
it 'will throw error for existing invalid phone number' do
|
|
contact = create(:contact)
|
|
expect { contact.update!(phone_number: '123456789') }.to raise_error(ActiveRecord::RecordInvalid)
|
|
end
|
|
|
|
it 'updates phone number when adding valid phone number' do
|
|
contact = create(:contact)
|
|
expect(contact.update!(phone_number: '+12312312321')).to be true
|
|
expect(contact.phone_number).to eq '+12312312321'
|
|
end
|
|
end
|
|
|
|
context 'when email format' do
|
|
it 'will throw error for existing invalid email' do
|
|
contact = create(:contact)
|
|
expect { contact.update!(email: '<2324234234') }.to raise_error(ActiveRecord::RecordInvalid)
|
|
end
|
|
|
|
it 'updates email when adding valid email' do
|
|
contact = create(:contact)
|
|
expect(contact.update!(email: 'test@test.com')).to be true
|
|
expect(contact.email).to eq 'test@test.com'
|
|
end
|
|
end
|
|
end
|