Skip to content

Feature/safo6 nrl 1841 performance tests public mode#1115

Merged
anjalitrace2-nhs merged 22 commits intodevelopfrom
feature/SAFO6-NRL-1841-performance-tests-public-mode
Jan 26, 2026
Merged

Feature/safo6 nrl 1841 performance tests public mode#1115
anjalitrace2-nhs merged 22 commits intodevelopfrom
feature/SAFO6-NRL-1841-performance-tests-public-mode

Conversation

@sandyforresternhs
Copy link
Contributor

@sandyforresternhs sandyforresternhs commented Jan 6, 2026

  • The internal tests haven’t changed - the name of the commands have just been updated in the makefile to make it clearer they’re internal
  • This pr covers the changes required for the consumer performance tests to run in public mode - the producer changes will be addressed in a subsequent pr
  • Includes changes for both consumer & producer tests to be run in public mode
  • Updated readme with new make commands for running these public tests, now including configuring the base URL and a simple local http server to serve up refreshed bearer auth tokens (each token is valid for 5 mins and the tests run for 36 mins, so needs refreshing mid-test run). See ticket for more info on why this approach was selected.
  • Tested these new commands for both producer & consumer in public & internal modes ✅

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

🚀 PR environment successfully deployed.
Commit Hash: 0bf7aff14df6ac96f80e3550dc1d362130e74361
URL: https://nrl1841-01ba47.api.record-locator.dev.national.nhs.uk/

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

🚀 PR environment successfully deployed.
Commit Hash: 48e7f01c0b519e31bb73d4fd66bce3e1676e652f
URL: https://nrl1841-01ba47.api.record-locator.dev.national.nhs.uk/

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

🚀 PR environment successfully deployed.
Commit Hash: 9a9b2fb88653efe6919b854bc324b1cd8aff02a7
URL: https://nrl1841-01ba47.api.record-locator.dev.national.nhs.uk/

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 2f052be3b70d3662e06050b22588939c80997645
URL: https://nrl1841-01ba47.api.record-locator.dev.national.nhs.uk/

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 15f102ea3edded40efff8a967d56cc82ff11296a
URL: https://nrl1841-01ba47.api.record-locator.dev.national.nhs.uk/

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 5f9f5c5030127f9e939babe5c7838caa6446a91b
URL: https://nrl1841-01ba47.api.record-locator.dev.national.nhs.uk/

…eshing bearer token and have public tests make requests to grab the latest token. This prevents tests from failing at the 10 min mark when the initially fetched token expires
@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 87d93121887227f67f7fb80370d1169772085fd8
URL: https://nrl1841-01ba47.api.record-locator.dev.national.nhs.uk/

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 5e514d1ff5bfed6857ff5f8c8cd73b0c1384f8c3
URL: https://nrl1841-01ba47.api.record-locator.dev.national.nhs.uk/

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: f6696cf8155ca9b9ae708500de823f39349527be
URL: https://nrl1841-01ba47.api.record-locator.dev.national.nhs.uk/

@sonarqubecloud
Copy link

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: fff45f56580e3e3ba38982fb2f488ecd505b382c
URL: https://nrl1841-01ba47.api.record-locator.dev.national.nhs.uk/

1 similar comment
@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: fff45f56580e3e3ba38982fb2f488ecd505b382c
URL: https://nrl1841-01ba47.api.record-locator.dev.national.nhs.uk/

Copy link
Contributor

@mattdean3-nhs mattdean3-nhs left a comment

Choose a reason for hiding this comment

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

Love this - nice work on the Python token handler/refresher 💪

Haven't tested it myself yet, but will give it a go later in the day.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants