45e43b0b89
- Add endpoint which lists inboxes through which a contact can be contacted - Conversation creation API auto-creates contact_inbox for specific channels [ Twilio, email, api] - Ability to send the initial message payload along with the conversation creation - Fixes #1678 ( issue saving additional attributes for conversation )
231 lines
8.9 KiB
Ruby
231 lines
8.9 KiB
Ruby
require 'rails_helper'
|
|
|
|
describe ::ContactInboxBuilder do
|
|
let(:account) { create(:account) }
|
|
let(:contact) { create(:contact, account: account) }
|
|
|
|
describe '#perform' do
|
|
describe 'twilio sms inbox' do
|
|
let!(:twilio_sms) { create(:channel_twilio_sms, account: account) }
|
|
let!(:twilio_inbox) { create(:inbox, channel: twilio_sms, account: account) }
|
|
|
|
it 'does not create contact inbox when contact inbox already exists with the source id provided' do
|
|
existing_contact_inbox = create(:contact_inbox, contact: contact, inbox: twilio_inbox, source_id: contact.phone_number)
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: twilio_inbox.id,
|
|
source_id: contact.phone_number
|
|
).perform
|
|
|
|
expect(contact_inbox.id).to be(existing_contact_inbox.id)
|
|
end
|
|
|
|
it 'does not create contact inbox when contact inbox already exists with phone number and source id is not provided' do
|
|
existing_contact_inbox = create(:contact_inbox, contact: contact, inbox: twilio_inbox, source_id: contact.phone_number)
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: twilio_inbox.id
|
|
).perform
|
|
|
|
expect(contact_inbox.id).to be(existing_contact_inbox.id)
|
|
end
|
|
|
|
it 'creates a new contact inbox when different source id is provided' do
|
|
existing_contact_inbox = create(:contact_inbox, contact: contact, inbox: twilio_inbox, source_id: contact.phone_number)
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: twilio_inbox.id,
|
|
source_id: '+224213223422'
|
|
).perform
|
|
|
|
expect(contact_inbox.id).not_to be(existing_contact_inbox.id)
|
|
expect(contact_inbox.source_id).not_to be('+224213223422')
|
|
end
|
|
|
|
it 'creates a contact inbox with contact phone number when source id not provided and no contact inbox exists' do
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: twilio_inbox.id
|
|
).perform
|
|
|
|
expect(contact_inbox.source_id).not_to be(contact.phone_number)
|
|
end
|
|
end
|
|
|
|
describe 'twilio whatsapp inbox' do
|
|
let!(:twilio_whatsapp) { create(:channel_twilio_sms, medium: :whatsapp, account: account) }
|
|
let!(:twilio_inbox) { create(:inbox, channel: twilio_whatsapp, account: account) }
|
|
|
|
it 'does not create contact inbox when contact inbox already exists with the source id provided' do
|
|
existing_contact_inbox = create(:contact_inbox, contact: contact, inbox: twilio_inbox, source_id: "whatsapp:#{contact.phone_number}")
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: twilio_inbox.id,
|
|
source_id: "whatsapp:#{contact.phone_number}"
|
|
).perform
|
|
|
|
expect(contact_inbox.id).to be(existing_contact_inbox.id)
|
|
end
|
|
|
|
it 'does not create contact inbox when contact inbox already exists with phone number and source id is not provided' do
|
|
existing_contact_inbox = create(:contact_inbox, contact: contact, inbox: twilio_inbox, source_id: "whatsapp:#{contact.phone_number}")
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: twilio_inbox.id
|
|
).perform
|
|
|
|
expect(contact_inbox.id).to be(existing_contact_inbox.id)
|
|
end
|
|
|
|
it 'creates a new contact inbox when different source id is provided' do
|
|
existing_contact_inbox = create(:contact_inbox, contact: contact, inbox: twilio_inbox, source_id: "whatsapp:#{contact.phone_number}")
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: twilio_inbox.id,
|
|
source_id: 'whatsapp:+555555'
|
|
).perform
|
|
|
|
expect(contact_inbox.id).not_to be(existing_contact_inbox.id)
|
|
expect(contact_inbox.source_id).not_to be('whatsapp:+55555')
|
|
end
|
|
|
|
it 'creates a contact inbox with contact phone number when source id not provided and no contact inbox exists' do
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: twilio_inbox.id
|
|
).perform
|
|
|
|
expect(contact_inbox.source_id).not_to be("whatsapp:#{contact.phone_number}")
|
|
end
|
|
end
|
|
|
|
describe 'email inbox' do
|
|
let!(:email_channel) { create(:channel_email, account: account) }
|
|
let!(:email_inbox) { create(:inbox, channel: email_channel, account: account) }
|
|
|
|
it 'does not create contact inbox when contact inbox already exists with the source id provided' do
|
|
existing_contact_inbox = create(:contact_inbox, contact: contact, inbox: email_inbox, source_id: contact.email)
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: email_inbox.id,
|
|
source_id: contact.email
|
|
).perform
|
|
|
|
expect(contact_inbox.id).to be(existing_contact_inbox.id)
|
|
end
|
|
|
|
it 'does not create contact inbox when contact inbox already exists with email and source id is not provided' do
|
|
existing_contact_inbox = create(:contact_inbox, contact: contact, inbox: email_inbox, source_id: contact.email)
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: email_inbox.id
|
|
).perform
|
|
|
|
expect(contact_inbox.id).to be(existing_contact_inbox.id)
|
|
end
|
|
|
|
it 'creates a new contact inbox when different source id is provided' do
|
|
existing_contact_inbox = create(:contact_inbox, contact: contact, inbox: email_inbox, source_id: contact.email)
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: email_inbox.id,
|
|
source_id: 'xyc@xyc.com'
|
|
).perform
|
|
|
|
expect(contact_inbox.id).not_to be(existing_contact_inbox.id)
|
|
expect(contact_inbox.source_id).not_to be('xyc@xyc.com')
|
|
end
|
|
|
|
it 'creates a contact inbox with contact email when source id not provided and no contact inbox exists' do
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: email_inbox.id
|
|
).perform
|
|
|
|
expect(contact_inbox.source_id).not_to be(contact.email)
|
|
end
|
|
end
|
|
|
|
describe 'api inbox' do
|
|
let!(:api_channel) { create(:channel_api, account: account) }
|
|
let!(:api_inbox) { create(:inbox, channel: api_channel, account: account) }
|
|
|
|
it 'does not create contact inbox when contact inbox already exists with the source id provided' do
|
|
existing_contact_inbox = create(:contact_inbox, contact: contact, inbox: api_inbox, source_id: 'test')
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: api_inbox.id,
|
|
source_id: 'test'
|
|
).perform
|
|
|
|
expect(contact_inbox.id).to be(existing_contact_inbox.id)
|
|
end
|
|
|
|
it 'creates a new contact inbox when different source id is provided' do
|
|
existing_contact_inbox = create(:contact_inbox, contact: contact, inbox: api_inbox, source_id: SecureRandom.uuid)
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: api_inbox.id,
|
|
source_id: 'test'
|
|
).perform
|
|
|
|
expect(contact_inbox.id).not_to be(existing_contact_inbox.id)
|
|
expect(contact_inbox.source_id).not_to be('test')
|
|
end
|
|
|
|
it 'creates a contact inbox with SecureRandom.uuid when source id not provided and no contact inbox exists' do
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: api_inbox.id
|
|
).perform
|
|
|
|
expect(contact_inbox.source_id).not_to be(nil)
|
|
end
|
|
end
|
|
|
|
describe 'web widget' do
|
|
let!(:website_channel) { create(:channel_widget, account: account) }
|
|
let!(:website_inbox) { create(:inbox, channel: website_channel, account: account) }
|
|
|
|
it 'does not create contact inbox' do
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: website_inbox.id,
|
|
source_id: 'test'
|
|
).perform
|
|
|
|
expect(contact_inbox).to be(nil)
|
|
end
|
|
end
|
|
|
|
describe 'facebook inbox' do
|
|
let!(:facebook_channel) { create(:channel_facebook_page, account: account) }
|
|
let!(:facebook_inbox) { create(:inbox, channel: facebook_channel, account: account) }
|
|
|
|
it 'does not create contact inbox' do
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: facebook_inbox.id,
|
|
source_id: 'test'
|
|
).perform
|
|
|
|
expect(contact_inbox).to be(nil)
|
|
end
|
|
end
|
|
|
|
describe 'twitter inbox' do
|
|
let!(:twitter_channel) { create(:channel_twitter_profile, account: account) }
|
|
let!(:twitter_inbox) { create(:inbox, channel: twitter_channel, account: account) }
|
|
|
|
it 'does not create contact inbox' do
|
|
contact_inbox = described_class.new(
|
|
contact_id: contact.id,
|
|
inbox_id: twitter_inbox.id,
|
|
source_id: 'test'
|
|
).perform
|
|
|
|
expect(contact_inbox).to be(nil)
|
|
end
|
|
end
|
|
end
|
|
end
|