* feat: add message update processing in IncomingMessageBaileysService * feat: implement message update handling in IncomingMessageBaileysService * feat: add MessageNotFoundError for handling invalid update messages in IncomingMessageBaileysService * chore: specs for message.update events and bug fixes * chore: refactor message update handling in IncomingMessageBaileysService * chore: nit remove redundant comment * chore: enhance logging for unsupported message update statuses in IncomingMessageBaileysService and disabled metrics * chore: message status update logic with transition checks * chore: update status mapping for PENDING to sent in status_mapper * chore: update status_mapper comments and fix case statement for status codes * fix: logging for unsupported message updates in update_message_content method * test: add specs for unsupported status transitions in messages.update event * refactor: ensure message status is reloaded before assertion in messages.update event spec * refactor: status variable in status_mapper method * refactor: rename status_transition_allowed method to status_transition_allowed? * refactor: streamline message creation in specs by using let! for setup * feat: process webhook whatsapp await response (#21) * feat: enhance WhatsApp webhook processing and error responses handling * chore: correct spelling of 'WhatsApp' in webhook controller specs * refactor: rename webhook processing method and improve error handling * chore: update error handling in WhatsApp controller specs for specific exceptions * refactor: remove handling for StandardError in WhatsApp controller specs * refactor: simplify perform_whatsapp_events_job method * chore: update response status from not_found to bad_request for invalid message * refactor: update expectations for job processing in WhatsApp controller specs
94 lines
3.8 KiB
Ruby
94 lines
3.8 KiB
Ruby
require 'rails_helper'
|
|
|
|
RSpec.describe 'Webhooks::WhatsappController', type: :request do
|
|
let(:channel) { create(:channel_whatsapp, provider: 'whatsapp_cloud', sync_templates: false, validate_provider_config: false) }
|
|
|
|
describe 'GET /webhooks/verify' do
|
|
it 'returns 401 when valid params are not present' do
|
|
get "/webhooks/whatsapp/#{channel.phone_number}"
|
|
expect(response).to have_http_status(:unauthorized)
|
|
end
|
|
|
|
it 'returns 401 when invalid params' do
|
|
get "/webhooks/whatsapp/#{channel.phone_number}",
|
|
params: { 'hub.challenge' => '123456', 'hub.mode' => 'subscribe', 'hub.verify_token' => 'invalid' }
|
|
expect(response).to have_http_status(:unauthorized)
|
|
end
|
|
|
|
it 'returns challenge when valid params' do
|
|
get "/webhooks/whatsapp/#{channel.phone_number}",
|
|
params: { 'hub.challenge' => '123456', 'hub.mode' => 'subscribe', 'hub.verify_token' => channel.provider_config['webhook_verify_token'] }
|
|
expect(response.body).to include '123456'
|
|
end
|
|
end
|
|
|
|
describe 'POST /webhooks/whatsapp/{:phone_number}' do
|
|
it 'calls the whatsapp events job asynchronously with perform_later when awaitResponse is not present' do
|
|
allow(Webhooks::WhatsappEventsJob).to receive(:perform_later)
|
|
|
|
post '/webhooks/whatsapp/123221321', params: { content: 'hello' }
|
|
|
|
expect(Webhooks::WhatsappEventsJob).to have_received(:perform_later)
|
|
expect(response).to have_http_status(:ok)
|
|
end
|
|
|
|
context 'when phone number is in inactive list' do
|
|
before do
|
|
allow(GlobalConfig).to receive(:get_value).with('INACTIVE_WHATSAPP_NUMBERS').and_return('+1234567890,+9876543210')
|
|
end
|
|
|
|
it 'returns service unavailable for inactive phone number in URL params' do
|
|
allow(Rails.logger).to receive(:warn)
|
|
|
|
post '/webhooks/whatsapp/+1234567890', params: { content: 'hello' }
|
|
|
|
expect(Rails.logger).to have_received(:warn).with('Rejected webhook for inactive WhatsApp number: +1234567890')
|
|
expect(response).to have_http_status(:unprocessable_entity)
|
|
expect(response.parsed_body['error']).to eq('Inactive WhatsApp number')
|
|
end
|
|
end
|
|
|
|
context 'when INACTIVE_WHATSAPP_NUMBERS config is not set' do
|
|
before do
|
|
allow(GlobalConfig).to receive(:get_value).with('INACTIVE_WHATSAPP_NUMBERS').and_return(nil)
|
|
end
|
|
|
|
it 'processes the webhook normally' do
|
|
allow(Webhooks::WhatsappEventsJob).to receive(:perform_later)
|
|
|
|
post '/webhooks/whatsapp/+1234567890', params: { content: 'hello' }
|
|
|
|
expect(Webhooks::WhatsappEventsJob).to have_received(:perform_later)
|
|
expect(response).to have_http_status(:ok)
|
|
end
|
|
end
|
|
|
|
context 'when awaitResponse param is present' do
|
|
it 'calls the whatsapp events job synchronously' do
|
|
allow(Webhooks::WhatsappEventsJob).to receive(:perform_now)
|
|
|
|
post '/webhooks/whatsapp/123221321', params: { content: 'hello', awaitResponse: true }
|
|
|
|
expect(Webhooks::WhatsappEventsJob).to have_received(:perform_now)
|
|
expect(response).to have_http_status(:ok)
|
|
end
|
|
|
|
it 'returns 401 when InvalidWebhookVerifyToken is raised' do
|
|
allow(Webhooks::WhatsappEventsJob).to receive(:perform_now).and_raise(Whatsapp::IncomingMessageBaileysService::InvalidWebhookVerifyToken)
|
|
|
|
post '/webhooks/whatsapp/123221321', params: { content: 'hello', awaitResponse: true }
|
|
|
|
expect(response).to have_http_status(:unauthorized)
|
|
end
|
|
|
|
it 'returns 400 when MessageNotFoundError is raised' do
|
|
allow(Webhooks::WhatsappEventsJob).to receive(:perform_now).and_raise(Whatsapp::IncomingMessageBaileysService::MessageNotFoundError)
|
|
|
|
post '/webhooks/whatsapp/123221321', params: { content: 'hello', awaitResponse: true }
|
|
|
|
expect(response).to have_http_status(:bad_request)
|
|
end
|
|
end
|
|
end
|
|
end
|