about summary refs log tree commit diff
path: root/spec/controllers/api/v1/push/subscriptions_controller_spec.rb
diff options
context:
space:
mode:
authorClaire <claire.github-309c@sitedethib.com>2023-01-23 13:05:30 +0100
committerGitHub <noreply@github.com>2023-01-23 13:05:30 +0100
commit448be26b34c965fcae65eb4c5227b35d97120537 (patch)
tree6aa1ea8bbfffcfcd42aea10c9bc473d1d0e40c79 /spec/controllers/api/v1/push/subscriptions_controller_spec.rb
parent9b795a25cd7a014d8c08cc213031b55dff83bb69 (diff)
Add missing `policy` attribute to `WebPushSubscriptionSerializer` (#23210)
* Add missing `policy` attribute to `WebPushSubscriptionSerializer`

Fixes #23145

* Add tests
Diffstat (limited to 'spec/controllers/api/v1/push/subscriptions_controller_spec.rb')
-rw-r--r--spec/controllers/api/v1/push/subscriptions_controller_spec.rb8
1 files changed, 8 insertions, 0 deletions
diff --git a/spec/controllers/api/v1/push/subscriptions_controller_spec.rb b/spec/controllers/api/v1/push/subscriptions_controller_spec.rb
index 534d02879..9487251e1 100644
--- a/spec/controllers/api/v1/push/subscriptions_controller_spec.rb
+++ b/spec/controllers/api/v1/push/subscriptions_controller_spec.rb
@@ -61,6 +61,10 @@ describe Api::V1::Push::SubscriptionsController do
       post :create, params: create_payload
       expect(Web::PushSubscription.where(endpoint: create_payload[:subscription][:endpoint]).count).to eq 1
     end
+
+    it 'returns the expected JSON' do
+      expect(body_as_json.with_indifferent_access).to include({ endpoint: create_payload[:subscription][:endpoint], alerts: {}, policy: 'all' })
+    end
   end
 
   describe 'PUT #update' do
@@ -78,6 +82,10 @@ describe Api::V1::Push::SubscriptionsController do
         expect(push_subscription.data['alerts'][type]).to eq(alerts_payload[:data][:alerts][type.to_sym].to_s)
       end
     end
+
+    it 'returns the expected JSON' do
+      expect(body_as_json.with_indifferent_access).to include({ endpoint: create_payload[:subscription][:endpoint], alerts: alerts_payload[:data][:alerts], policy: alerts_payload[:data][:policy] })
+    end
   end
 
   describe 'DELETE #destroy' do