Skip to content

Appt 2014 - Cancel date range integration tests#1478

Open
ste-banks wants to merge 4 commits intomainfrom
appt-2014/cancel-date-range-integration-tests
Open

Appt 2014 - Cancel date range integration tests#1478
ste-banks wants to merge 4 commits intomainfrom
appt-2014/cancel-date-range-integration-tests

Conversation

@ste-banks
Copy link
Contributor

Description

Integration tests for the cancel a date range endpoint with the two feature flags in various states and utilising the new MultipleFeatureToggledSteps class. There are some repeated tests in the feature files and this is to ensure correct behaviour when the flags are in different states.

Fixes # (issue)

Checklist:

  • My work is behind a feature toggle (if appropriate)
  • If my work is behind a feature toggle, I've added a full suite of tests for both the ON and OFF state
  • The ticket number is in the Pull Request title, with format "APPT-XXX: My Title Here"
  • I have ran npm tsc / lint (in the future these will be ran automatically)
  • My code generates no new .NET warnings (in the future these will be treated as errors)
  • If I've added a new Function, it is disabled in all but one of the terraform groups (e.g. http_functions)
  • If I've added a new Function, it has both unit and integration tests. Any request body validators have unit tests also
  • If I've made UI changes, I've added appropriate Playwright and Jest tests
  • If I've added/updated an end-point, I've added the appropriate annotations and tested the Swagger documentation reflects the change

@ste-banks ste-banks force-pushed the appt-2014/cancel-date-range-integration-tests branch from 65c71b8 to 5250cec Compare March 4, 2026 16:11
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 4, 2026

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.

1 participant