Skip to content

Draft: Hdpi 6527 api automation tests#1921

Closed
kanthnew wants to merge 50 commits into
masterfrom
HDPI-6527_api_automation_tests
Closed

Draft: Hdpi 6527 api automation tests#1921
kanthnew wants to merge 50 commits into
masterfrom
HDPI-6527_api_automation_tests

Conversation

@kanthnew

@kanthnew kanthnew commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

https://tools.hmcts.net/jira/browse/HDPI-6527

Change description

To add automated API tests for the payment-uodate api service callback
The aim is to verify that the integration layer behaves as expected, so that the callbacks request is validated before processing payment update.
Payment update process is successful
Callbacks in scope:
Payment Update Callback: PUT /payment-update
Testing Support Endpoint: GET /testing-support/case-payment-references (or similar path) to retrieve reference details for test context.
Test type:
Automated API / Functional tests: Covering the REST controller and Service layer.
Validation Testing: Specifically targeting mandatory field rules (negative testing).
Verification points:
Data Validation: Ensuring the service rejects requests with missing mandatory fields as per the data model.
Persistence: Verifying that invalid requests do not result in a record being saved to the database.
Integration: Verifying the REST controller successfully hands off valid payloads to the service layer.

Testing done

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

@kanthnew kanthnew requested review from a team, andrewdami11, benouaer and toby-plunkett June 2, 2026 15:09
@kanthnew kanthnew changed the title Hdpi 6527 api automation tests Draft: Hdpi 6527 api automation tests Jun 2, 2026
@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown

CCD diff summary

👉 Full report: https://github.com/hmcts/pcs-api/actions/runs/27603654722?check_suite_focus=true

No change

arun-hmcts and others added 15 commits June 23, 2026 15:40
)

Signed-off-by: sstewart <202802468+scottstewart-sl@users.noreply.github.com>
Co-authored-by: tvr-solirius <trevor.hinson@solirius.com>
Co-authored-by: srinijg <srinivasan.joghee@hmcts.net>
Co-authored-by: hmcts-jenkins-j-to-z <61242337+hmcts-jenkins-j-to-z[bot]@users.noreply.github.com>
Co-authored-by: PadmaDeenadayalan <Padma.Deenadayalan@hmcts.net>
Co-authored-by: marianadpereira <71711509+marianadpereira@users.noreply.github.com>
Co-authored-by: padmakumari <padma.kumari@hmcts.net>
Co-authored-by: jessieharrigan <144362161+jessieharrigan@users.noreply.github.com>
Co-authored-by: marianadpereira <mariana.pereira@HMCTS.NET>
Co-authored-by: guygrewal77 <guygrewal7@gmail.com>
Co-authored-by: arun <arun.kumar@hmcts.net>
Co-authored-by: marianadpereira <71711509+marianadpereira@users.noreply.github.com>
Co-authored-by: Alex McAusland <a.m@user.com>
Co-authored-by: arun <arun.kumar@hmcts.net>
Co-authored-by: Marianne-Azzopardi <147049624+mazzopardi2@users.noreply.github.com>
Co-authored-by: marianadpereira <71711509+marianadpereira@users.noreply.github.com>
Co-authored-by: Lovelin <lovelin.marcus@hmcts.net>
Co-authored-by: Marianne-Azzopardi <marianne.azzopardi@solirius.com>
Co-authored-by: guygrewal77 <guygrewal7@gmail.com>
Co-authored-by: guygrewal77 <163517291+guygrewal77@users.noreply.github.com>
Co-authored-by: Rupali Lowalekar <Rupali.Lowalekar@HMCTS.NET>
Co-authored-by: Praveen Adusumilli <47391951+adusumillipraveen@users.noreply.github.com>
#1987)

Co-authored-by: adusumillipraveen <praveen.adusumilli@hmcts.net>
Co-authored-by: Praveen Adusumilli <47391951+adusumillipraveen@users.noreply.github.com>
Co-authored-by: arun <arun.kumar@hmcts.net>
Co-authored-by: marianadpereira <71711509+marianadpereira@users.noreply.github.com>
Co-authored-by: arun <arun.kumar@hmcts.net>
Co-authored-by: arun-hmcts <73122505+arun-hmcts@users.noreply.github.com>
Co-authored-by: madhavi1203 <Madhavi.Atmakuri1@HMCTS.NET>
Co-authored-by: Alex McAusland <a.m@user.com>
Co-authored-by: srinijg <srinivasan.joghee@hmcts.net>
@hmcts-jenkins-j-to-z

Copy link
Copy Markdown
Contributor

Plan Result (aat)

⚠️ Resource Deletion will happen

This plan contains resource delete operation. Please check the plan result very carefully!

Plan: 0 to add, 0 to change, 1 to destroy.
  • Delete
    • module.postgresql.azurerm_postgresql_flexible_server_configuration.pgsql_server_config["pg_qs.query_capture_mode"]
Change Result (Click me)
  # module.postgresql.azurerm_postgresql_flexible_server_configuration.pgsql_server_config["pg_qs.query_capture_mode"] will be destroyed
  # (because key ["pg_qs.query_capture_mode"] is not in for_each map)
  - resource "azurerm_postgresql_flexible_server_configuration" "pgsql_server_config" {
      - id        = "/subscriptions/1c4f0704-a29e-403d-b719-b90c34ef14c9/resourceGroups/pcs-data-aat/providers/Microsoft.DBforPostgreSQL/flexibleServers/pcs-aat/configurations/pg_qs.query_capture_mode" -> null
      - name      = "pg_qs.query_capture_mode" -> null
      - server_id = "/subscriptions/1c4f0704-a29e-403d-b719-b90c34ef14c9/resourceGroups/pcs-data-aat/providers/Microsoft.DBforPostgreSQL/flexibleServers/pcs-aat" -> null
      - value     = "top" -> null
    }

Plan: 0 to add, 0 to change, 1 to destroy.

@kanthnew

Copy link
Copy Markdown
Contributor Author

Openned a new PR with latest from main branch as rebase got complicated

@kanthnew kanthnew closed this Jun 24, 2026
@kanthnew

Copy link
Copy Markdown
Contributor Author

new PR at #2019

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.