From 813cdd8aeced7ebc44c4dbf88754ae5b425d5b4c Mon Sep 17 00:00:00 2001 From: Tim Lange Date: Thu, 20 Feb 2020 07:42:19 +0100 Subject: [PATCH] Chore: Added tests for facebook_indicators_controller (#526) --- .../api/v1/contacts_controller_spec.rb | 10 +- .../v1/facebook_indicators_controller_spec.rb | 143 ++++++++++++++++++ 2 files changed, 145 insertions(+), 8 deletions(-) create mode 100644 spec/controllers/api/v1/facebook_indicators_controller_spec.rb diff --git a/spec/controllers/api/v1/contacts_controller_spec.rb b/spec/controllers/api/v1/contacts_controller_spec.rb index 93146423b..f8aa54589 100644 --- a/spec/controllers/api/v1/contacts_controller_spec.rb +++ b/spec/controllers/api/v1/contacts_controller_spec.rb @@ -57,11 +57,9 @@ RSpec.describe 'Contacts API', type: :request do context 'when it is an unauthenticated user' do it 'creates the contact' do - post '/api/v1/contacts', - params: valid_params + expect { post '/api/v1/contacts', params: valid_params }.to change(Contact, :count).by(1) expect(response).to have_http_status(:success) - expect(Contact.count).to eq(1) end end @@ -69,13 +67,9 @@ RSpec.describe 'Contacts API', type: :request do let(:admin) { create(:user, account: account, role: :administrator) } it 'creates the contact' do - post '/api/v1/contacts', - headers: admin.create_new_auth_token, - params: valid_params, - as: :json + expect { post '/api/v1/contacts', headers: admin.create_new_auth_token, params: valid_params }.to change(Contact, :count).by(1) expect(response).to have_http_status(:success) - expect(Contact.count).to eq(1) end end end diff --git a/spec/controllers/api/v1/facebook_indicators_controller_spec.rb b/spec/controllers/api/v1/facebook_indicators_controller_spec.rb new file mode 100644 index 000000000..5154ca4dd --- /dev/null +++ b/spec/controllers/api/v1/facebook_indicators_controller_spec.rb @@ -0,0 +1,143 @@ +require 'rails_helper' + +RSpec.describe 'Facebook Indicators API', type: :request do + let(:account) { create(:account) } + let(:facebook_channel) { create(:channel_facebook_page, account: account) } + let(:inbox) { create(:inbox, account: account, channel: facebook_channel) } + let(:contact) { create(:contact, account: account) } + let(:valid_params) { { contact_id: contact.id, inbox_id: inbox.id } } + + before do + allow(Facebook::Messenger::Bot).to receive(:deliver).and_return(true) + allow(Facebook::Messenger::Subscriptions).to receive(:subscribe).and_return(true) + end + + describe 'POST /api/v1/facebook_indicators/mark_seen' do + context 'when it is an unauthenticated user' do + it 'returns unauthorized' do + post '/api/v1/facebook_indicators/mark_seen' + + expect(response).to have_http_status(:unauthorized) + end + end + + context 'when it is an authenticated user' do + let(:agent) { create(:user, account: account, role: :agent) } + + it 'marks a payload as seen' do + contact_inbox = create(:contact_inbox, contact: contact, inbox: inbox) + + expect(Facebook::Messenger::Bot).to receive(:deliver).with( + { recipient: { id: contact_inbox.source_id }, sender_action: 'mark_seen' }, + access_token: inbox.channel.page_access_token + ) + + post '/api/v1/facebook_indicators/mark_seen', + headers: agent.create_new_auth_token, + params: valid_params, + as: :json + + expect(response).to have_http_status(:success) + end + + it 'rescues an error' do + create(:contact_inbox, contact: contact, inbox: inbox) + + allow(Facebook::Messenger::Bot).to receive(:deliver).and_raise(Facebook::Messenger::Error) + + post '/api/v1/facebook_indicators/mark_seen', + headers: agent.create_new_auth_token, + params: valid_params, + as: :json + + expect(response).to have_http_status(:success) + end + end + end + + describe 'POST /api/v1/facebook_indicators/typing_on' do + context 'when it is an unauthenticated user' do + it 'returns unauthorized' do + post '/api/v1/facebook_indicators/typing_on' + + expect(response).to have_http_status(:unauthorized) + end + end + + context 'when it is an authenticated user' do + let(:agent) { create(:user, account: account, role: :agent) } + + it 'marks a payload as typing_on' do + contact_inbox = create(:contact_inbox, contact: contact, inbox: inbox) + + expect(Facebook::Messenger::Bot).to receive(:deliver).with( + { recipient: { id: contact_inbox.source_id }, sender_action: 'typing_on' }, + access_token: inbox.channel.page_access_token + ) + + post '/api/v1/facebook_indicators/typing_on', + headers: agent.create_new_auth_token, + params: valid_params, + as: :json + + expect(response).to have_http_status(:success) + end + + it 'rescues an error' do + create(:contact_inbox, contact: contact, inbox: inbox) + + allow(Facebook::Messenger::Bot).to receive(:deliver).and_raise(Facebook::Messenger::Error) + + post '/api/v1/facebook_indicators/typing_on', + headers: agent.create_new_auth_token, + params: valid_params, + as: :json + + expect(response).to have_http_status(:success) + end + end + end + + describe 'POST /api/v1/facebook_indicators/typing_off' do + context 'when it is an unauthenticated user' do + it 'returns unauthorized' do + post '/api/v1/facebook_indicators/typing_off' + + expect(response).to have_http_status(:unauthorized) + end + end + + context 'when it is an authenticated user' do + let(:agent) { create(:user, account: account, role: :agent) } + + it 'marks a payload as typing_off' do + contact_inbox = create(:contact_inbox, contact: contact, inbox: inbox) + + expect(Facebook::Messenger::Bot).to receive(:deliver).with( + { recipient: { id: contact_inbox.source_id }, sender_action: 'typing_off' }, + access_token: inbox.channel.page_access_token + ) + + post '/api/v1/facebook_indicators/typing_off', + headers: agent.create_new_auth_token, + params: valid_params, + as: :json + + expect(response).to have_http_status(:success) + end + + it 'rescues an error' do + create(:contact_inbox, contact: contact, inbox: inbox) + + allow(Facebook::Messenger::Bot).to receive(:deliver).and_raise(Facebook::Messenger::Error) + + post '/api/v1/facebook_indicators/typing_off', + headers: agent.create_new_auth_token, + params: valid_params, + as: :json + + expect(response).to have_http_status(:success) + end + end + end +end