fix(cypress.config): patch Cypress Authentik test configuration& update contact E2E tests#313
Conversation
Preview DeploymentPreview URL: https://preview-fix-bdms-839-patch-ci-tests-auejgdbofq-uc.a.run.app Note: This preview uses the staging API endpoints. |
Preview DeploymentPreview URL: https://preview-fix-bdms-839-patch-ci-tests-auejgdbofq-uc.a.run.app Note: This preview uses the staging API endpoints. |
There was a problem hiding this comment.
Contact Create E2E coverage was temporarily removed because the existing test no longer reflected the current UI.
Preview DeploymentPreview URL: https://preview-fix-bdms-839-patch-ci-tests-auejgdbofq-uc.a.run.app Note: This preview uses the staging API endpoints. |
chasetmartin
left a comment
There was a problem hiding this comment.
Nice work, great to see the entire test suite passing!
Doesn't relate to the PR merging but worth a note: location_create.cy.ts and location_edit.cy.ts tests in the Cypress suite and their associated app routes have not been worked on for months given V1 launch and the pivot to Jeremy's initial editing capability work. The location/create and edit routes are only available via direct url to admin users I believe. Same can be said about many of the other currently untested and/or stale routes documented by Jeremy in his OcotilloUI: URL Inventory doc (Shared on Slack). Wanted to mention this for @lizalino and any code health / technical debt backlog tickets.
Why
This PR addresses the following problem/context:
How
Implementation summary - the following was changed/added/removed:
VITE_TEST_AUTHto the Cypress GitHub Actions workflow so the frontend uses the test authentication flow during E2E testing.Notes
Any special considerations, workarounds, or follow-up work to note?