Hello,
+ +Your Facebook Inbox Access has expired.
+Please reconnect Facebook Page to continue receiving messages in Chatwoot
+ ++Click here to re-connect. +
diff --git a/spec/builders/messages/facebook/message_builder_spec.rb b/spec/builders/messages/facebook/message_builder_spec.rb index 52edd0413..dbfdb0c3d 100644 --- a/spec/builders/messages/facebook/message_builder_spec.rb +++ b/spec/builders/messages/facebook/message_builder_spec.rb @@ -8,20 +8,17 @@ describe ::Messages::Facebook::MessageBuilder do let!(:incoming_fb_text_message) { Integrations::Facebook::MessageParser.new(message_object) } let(:fb_object) { double } - before do - allow(Koala::Facebook::API).to receive(:new).and_return(fb_object) - allow(fb_object).to receive(:get_object).and_return( - { - first_name: 'Jane', - last_name: 'Dae', - account_id: facebook_channel.inbox.account_id, - profile_pic: 'https://via.placeholder.com/250x250.png' - }.with_indifferent_access - ) - end - describe '#perform' do it 'creates contact and message for the facebook inbox' do + allow(Koala::Facebook::API).to receive(:new).and_return(fb_object) + allow(fb_object).to receive(:get_object).and_return( + { + first_name: 'Jane', + last_name: 'Dae', + account_id: facebook_channel.inbox.account_id, + profile_pic: 'https://via.placeholder.com/250x250.png' + }.with_indifferent_access + ) message_builder contact = facebook_channel.inbox.contacts.first @@ -30,5 +27,13 @@ describe ::Messages::Facebook::MessageBuilder do expect(contact.name).to eq('Jane Dae') expect(message.content).to eq('facebook message') end + + it 'increments channel authorization_error_count when error is thrown' do + allow(Koala::Facebook::API).to receive(:new).and_return(fb_object) + allow(fb_object).to receive(:get_object).and_raise(Koala::Facebook::AuthenticationError.new(500, 'Error validating access token')) + message_builder + + expect(facebook_channel.authorization_error_count).to eq(1) + end end end diff --git a/spec/mailers/administrator_notifications/channel_notifications_mailer_spec.rb b/spec/mailers/administrator_notifications/channel_notifications_mailer_spec.rb new file mode 100644 index 000000000..b75a899c9 --- /dev/null +++ b/spec/mailers/administrator_notifications/channel_notifications_mailer_spec.rb @@ -0,0 +1,40 @@ +# frozen_string_literal: true + +require 'rails_helper' + +RSpec.describe AdministratorNotifications::ChannelNotificationsMailer, type: :mailer do + let(:class_instance) { described_class.new } + let!(:account) { create(:account) } + let!(:administrator) { create(:user, :administrator, email: 'agent1@example.com', account: account) } + + before do + allow(described_class).to receive(:new).and_return(class_instance) + allow(class_instance).to receive(:smtp_config_set_or_development?).and_return(true) + end + + describe 'slack_disconnect' do + let(:mail) { described_class.with(account: account).slack_disconnect.deliver_now } + + it 'renders the subject' do + expect(mail.subject).to eq('Your Slack integration has expired') + end + + it 'renders the receiver email' do + expect(mail.to).to eq([administrator.email]) + end + end + + describe 'facebook_disconnect' do + let!(:facebook_channel) { create(:channel_facebook_page, account: account) } + let!(:facebook_inbox) { create(:inbox, channel: facebook_channel, account: account) } + let(:mail) { described_class.with(account: account).facebook_disconnect(facebook_inbox).deliver_now } + + it 'renders the subject' do + expect(mail.subject).to eq('Your Facebook page connection has expired') + end + + it 'renders the receiver email' do + expect(mail.to).to eq([administrator.email]) + end + end +end diff --git a/spec/mailers/agent_notifications/conversation_notifications_mailer_spec.rb b/spec/mailers/agent_notifications/conversation_notifications_mailer_spec.rb index 3f909fb97..54dd9dae3 100644 --- a/spec/mailers/agent_notifications/conversation_notifications_mailer_spec.rb +++ b/spec/mailers/agent_notifications/conversation_notifications_mailer_spec.rb @@ -14,7 +14,7 @@ RSpec.describe AgentNotifications::ConversationNotificationsMailer, type: :maile end describe 'conversation_creation' do - let(:mail) { described_class.conversation_creation(conversation, agent).deliver_now } + let(:mail) { described_class.with(account: account).conversation_creation(conversation, agent).deliver_now } it 'renders the subject' do expect(mail.subject).to eq("#{agent.available_name}, A new conversation [ID - #{conversation @@ -27,7 +27,7 @@ RSpec.describe AgentNotifications::ConversationNotificationsMailer, type: :maile end describe 'conversation_assignment' do - let(:mail) { described_class.conversation_assignment(conversation, agent).deliver_now } + let(:mail) { described_class.with(account: account).conversation_assignment(conversation, agent).deliver_now } it 'renders the subject' do expect(mail.subject).to eq("#{agent.available_name}, A new conversation [ID - #{conversation.display_id}] has been assigned to you.") @@ -40,7 +40,7 @@ RSpec.describe AgentNotifications::ConversationNotificationsMailer, type: :maile describe 'conversation_mention' do let(:message) { create(:message, conversation: conversation, account: account) } - let(:mail) { described_class.conversation_mention(message, agent).deliver_now } + let(:mail) { described_class.with(account: account).conversation_mention(message, agent).deliver_now } it 'renders the subject' do expect(mail.subject).to eq("#{agent.available_name}, You have been mentioned in conversation [ID - #{conversation.display_id}]") @@ -53,7 +53,7 @@ RSpec.describe AgentNotifications::ConversationNotificationsMailer, type: :maile describe 'assigned_conversation_new_message' do let(:message) { create(:message, conversation: conversation, account: account) } - let(:mail) { described_class.assigned_conversation_new_message(message, agent).deliver_now } + let(:mail) { described_class.with(account: account).assigned_conversation_new_message(message, agent).deliver_now } it 'renders the subject' do expect(mail.subject).to eq("#{agent.available_name}, New message in your assigned conversation [ID - #{message.conversation.display_id}].") diff --git a/spec/models/channel/facebook_page_spec.rb b/spec/models/channel/facebook_page_spec.rb index bcd468556..343b0de09 100644 --- a/spec/models/channel/facebook_page_spec.rb +++ b/spec/models/channel/facebook_page_spec.rb @@ -13,6 +13,17 @@ RSpec.describe Channel::FacebookPage do describe 'concerns' do it_behaves_like 'reauthorizable' + + context 'when prompt_reauthorization!' do + it 'calls channel notifier mail for facebook' do + admin_mailer = double + mailer_double = double + expect(AdministratorNotifications::ChannelNotificationsMailer).to receive(:with).and_return(admin_mailer) + expect(admin_mailer).to receive(:facebook_disconnect).with(channel.inbox).and_return(mailer_double) + expect(mailer_double).to receive(:deliver_later) + channel.prompt_reauthorization! + end + end end it 'has a valid name' do