Repository navigation
feat: add e2e tests - #23
Merged
Merged
Conversation
adds e2e tests for `/` and `/issues`
feat: adds landing page fixture feat: updates landing page E2E tests to use fixture fix: adds ARIA landmark and label to app sidebar component to make it easier to find via 'getByRole'
adds ARIA snapshot for homepage main content
rtmcardle
approved these changes
Sep 24, 2026
rtmcardle
left a comment
rtmcardle
left a comment
Collaborator
There was a problem hiding this comment.
I'm not well versed in Playwright, but I like tests and things look good!
dheshanm
approved these changes
Sep 29, 2026
dheshanm
left a comment
dheshanm
left a comment
Member
There was a problem hiding this comment.
I haven't used playwright before. The tests definitely seem like a good addition!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds Playwright based E2E tests for:
//issues/issues/multiPart/interviews/journalsCurrently tests are non-destructive i.e. simply validate the page is available and certain structural elements are present.
To run simply use
npx playwright test. Note the use of abaseURLinplaywright.config.ts.Many tests use known good ARIA snapshots, stored under
tests/__snapshots__/manual editing is not recommended. Use the--update-snapshotsflag with the test command if necessary.