chore: Add subscription delete API (#3703)
This commit is contained in:
parent
a61dae0bb2
commit
e1fe4f09bc
3 changed files with 35 additions and 1 deletions
|
@ -7,6 +7,12 @@ class Api::V1::NotificationSubscriptionsController < Api::BaseController
|
||||||
render json: notification_subscription
|
render json: notification_subscription
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def destroy
|
||||||
|
notification_subscription = NotificationSubscription.where(["subscription_attributes->>'push_token' = ?", params[:push_token]]).first
|
||||||
|
notification_subscription.destroy
|
||||||
|
head :ok
|
||||||
|
end
|
||||||
|
|
||||||
private
|
private
|
||||||
|
|
||||||
def set_user
|
def set_user
|
||||||
|
|
|
@ -172,7 +172,7 @@ Rails.application.routes.draw do
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
resource :notification_subscriptions, only: [:create]
|
resource :notification_subscriptions, only: [:create, :destroy]
|
||||||
|
|
||||||
namespace :widget do
|
namespace :widget do
|
||||||
resource :config, only: [:create]
|
resource :config, only: [:create]
|
||||||
|
|
|
@ -80,4 +80,32 @@ RSpec.describe 'Notifications Subscriptions API', type: :request do
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
describe 'DELETE /api/v1/notification_subscriptions' do
|
||||||
|
context 'when it is an unauthenticated user' do
|
||||||
|
it 'returns unauthorized' do
|
||||||
|
delete '/api/v1/notification_subscriptions'
|
||||||
|
|
||||||
|
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 'delete existing notification subscription if subscription exists' do
|
||||||
|
subscription = create(:notification_subscription, subscription_type: 'fcm', subscription_attributes: { push_token: 'bUvZo8AYGGmCMr' },
|
||||||
|
user: agent)
|
||||||
|
delete '/api/v1/notification_subscriptions',
|
||||||
|
params: {
|
||||||
|
push_token: subscription.subscription_attributes['push_token']
|
||||||
|
},
|
||||||
|
headers: agent.create_new_auth_token,
|
||||||
|
as: :json
|
||||||
|
|
||||||
|
expect(response).to have_http_status(:success)
|
||||||
|
expect { subscription.reload }.to raise_exception(ActiveRecord::RecordNotFound)
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in a new issue