Skip to content

APPT-2029 Playwright test v1 cannot cancel page#1468

Open
pchoi-al wants to merge 9 commits intomainfrom
APPT-2029/playwright-test-v1-cannot-cancel-page
Open

APPT-2029 Playwright test v1 cannot cancel page#1468
pchoi-al wants to merge 9 commits intomainfrom
APPT-2029/playwright-test-v1-cannot-cancel-page

Conversation

@pchoi-al
Copy link
Contributor

@pchoi-al pchoi-al commented Mar 2, 2026

Description

Playwright test for cannot cancel page when CancelADateRangeWithBookings is disabled and bookings are detected. You only have options to navigate to View Availability or Pick a date range

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


test.beforeAll(async () => {
await overrideFeatureFlag('CancelADateRange', true);
await overrideFeatureFlag('CancelADateRangeWithBookings', false);
Copy link
Contributor

Choose a reason for hiding this comment

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

If the flag is false, no need to set and revert it, as false is the default behaviour.

`/manage-your-appointments/site/${site.id}/change-availability`,
);

// await expect(
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like we dont need this code, could we delete it?

@pchoi-al pchoi-al force-pushed the APPT-2029/playwright-test-v1-cannot-cancel-page branch 2 times, most recently from 54ebba1 to ffac583 Compare March 4, 2026 09:28
@pchoi-al pchoi-al force-pushed the APPT-2029/playwright-test-v1-cannot-cancel-page branch from e7bb841 to b76d30c Compare March 4, 2026 18:01
@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.

3 participants