From 0d4a05476ba6642c613e36cfd43d4bdb27ca8241 Mon Sep 17 00:00:00 2001 From: Mateusz Grotek Date: Fri, 12 Apr 2024 14:36:36 +0100 Subject: [PATCH] Remove specialist topics from email alert api pact This commit removes specialist topics related code from tests. It matches the changes introduced by the #2147 PR. --- CHANGELOG.md | 4 ++++ test/pacts/email_alert_api_pact_test.rb | 22 +++++++++++----------- 2 files changed, 15 insertions(+), 11 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 8cc65169..0279c977 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,3 +1,7 @@ +# Unreleased + +* Update Pact specs to match the email-alert-api [PR](https://github.com/alphagov/email-alert-api/pull/2136) + # 96.0.0 * BREAKING: Rename imminence endpoints to places_manager [PR](https://github.com/alphagov/gds-api-adapters/pull/1253) diff --git a/test/pacts/email_alert_api_pact_test.rb b/test/pacts/email_alert_api_pact_test.rb index 9bab52ea..fa30d7e0 100644 --- a/test/pacts/email_alert_api_pact_test.rb +++ b/test/pacts/email_alert_api_pact_test.rb @@ -23,7 +23,7 @@ { id: Pact.like(1), links: {}, - tags: { topics: { any: ["motoring/road_rage"] } }, + tags: { tribunal_decision_categories: { any: %w[agency-workers] } }, document_type: "", slug: Pact.like("title-1"), title: Pact.like("title 1"), @@ -39,7 +39,7 @@ path: "/subscriber-lists", body: { title: "new-title", - tags: { topics: { any: ["motoring/road_rage"] } }, + tags: { tribunal_decision_categories: { any: %w[agency-workers] } }, }, headers: GdsApi::JsonClient.default_request_with_json_body_headers, ) @@ -53,19 +53,19 @@ }, ) - api_client.find_or_create_subscriber_list(title: "new-title", tags: { topics: { any: ["motoring/road_rage"] } }) + api_client.find_or_create_subscriber_list(title: "new-title", tags: { tribunal_decision_categories: { any: %w[agency-workers] } }) end it "responds with the subscriber list with an updated title" do email_alert_api - .given("a subscriber list with the tag topic: motoring/road_rage exists") + .given("a subscriber list with the tag tribunal decision category: agency-workers exists") .upon_receiving("a request to find or create a subscriber list") .with( method: :post, path: "/subscriber-lists", body: { title: "new-title", - tags: { topics: { any: ["motoring/road_rage"] } }, + tags: { tribunal_decision_categories: { any: %w[agency-workers] } }, }, headers: GdsApi::JsonClient.default_request_with_json_body_headers, ) @@ -79,7 +79,7 @@ }, ) - api_client.find_or_create_subscriber_list(title: "new-title", tags: { topics: { any: ["motoring/road_rage"] } }) + api_client.find_or_create_subscriber_list(title: "new-title", tags: { tribunal_decision_categories: { any: %w[agency-workers] } }) end end @@ -92,7 +92,7 @@ path: "/subscriber-lists", # Query here is odd because the parser doesn't quite handle # the output of Rack::Utils.build_nested_query - query: { "tags[topics][any][]": ["motoring/road_rage"] }, + query: { "tags[tribunal_decision_categories][any][]": %w[agency-workers] }, headers: GdsApi::JsonClient.default_request_headers, ) .will_respond_with( @@ -104,7 +104,7 @@ ) begin - api_client.find_subscriber_list(tags: { topics: { any: ["motoring/road_rage"] } }) + api_client.find_subscriber_list(tags: { tribunal_decision_categories: { any: %w[agency-workers] } }) rescue GdsApi::HTTPNotFound # We expect this to throw an exception end @@ -112,14 +112,14 @@ it "responds with the subscriber list" do email_alert_api - .given("a subscriber list with the tag topic: motoring/road_rage exists") + .given("a subscriber list with the tag tribunal decision category: agency-workers exists") .upon_receiving("a request for the subscriber list") .with( method: :get, path: "/subscriber-lists", # Query here is odd because the parser doesn't quite handle # the output of Rack::Utils.build_nested_query - query: { "tags[topics][any][]": ["motoring/road_rage"] }, + query: { "tags[tribunal_decision_categories][any][]": %w[agency-workers] }, headers: GdsApi::JsonClient.default_request_headers, ) .will_respond_with( @@ -132,7 +132,7 @@ }, ) - api_client.find_subscriber_list(tags: { topics: { any: ["motoring/road_rage"] } }) + api_client.find_subscriber_list(tags: { tribunal_decision_categories: { any: %w[agency-workers] } }) end end