Skip to content

CME-513: Fix pact for consumer: xui_webApp and providers: acc_manageCaseAssignment, acc_manageCaseAssignment_Noc - #5320

Open
anthonyb-cgi wants to merge 18 commits into
masterfrom
CME-513-fix-pact
Open

CME-513: Fix pact for consumer: xui_webApp and providers: acc_manageCaseAssignment, acc_manageCaseAssignment_Noc#5320
anthonyb-cgi wants to merge 18 commits into
masterfrom
CME-513-fix-pact

Conversation

@anthonyb-cgi

@anthonyb-cgi anthonyb-cgi commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Jira link

See CME-513

Change description

EXUI Consumer Pact tests were updated to support AAC Case Assignment provider verification by:

  • Aligning expectations with provider behavior for shared_with idam_id fields to ensure contract compliance.
  • Migrating interactions to the Pact V3 fluent DSL and matchers.
  • Exercising the real consumer API client, including request/response transformation.
  • Aligning provider names, state setup, case IDs, headers, paths, and query parameters with provider contracts.
  • Relaxing matchers and removing response fields not used by the consumer.
  • Adding case_title/caseTitle coverage for case-assignment responses.
  • Updating NoC contracts to match the provider’s actual request and response payloads.

…ectation of provider acc_manageCaseAssignment.
headers: {
'Content-Type': 'application/json',
},
body: mockResponse,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why body is not validated here?

@anthonyb-cgi anthonyb-cgi Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good question - as far as I can tell the real EXUI consumer API client doesn't use the provider response body field status_message when successfully calling it. Therefore, I have omitted validation of it from the pact (there used to be pact validation of the body previously when the consumer pact tests weren't testing against the real consumer API client).

However, if the response is rejected the real consumer API client does read the the API response so will check that.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here is the new commit I made to cover consumer API consumption of the provider error response body:
d71d6ac

@kiran-yenigala-hmcts kiran-yenigala-hmcts left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One comment added

@kiran-yenigala-hmcts kiran-yenigala-hmcts left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants