-
Notifications
You must be signed in to change notification settings - Fork 197
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace single image unit tests with Playwright e2e #2583
Conversation
Size Change: 0 B Total Size: 829 kB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Apologies I don't have more feedback, but it looks perfect.
f784492
to
308d183
Compare
Based on the medium urgency of this PR, the following reviewers are being gently reminded to review this PR: @AetherUnbound Excluding weekend1 days, this PR was ready for review 4 day(s) ago. PRs labelled with medium urgency are expected to be reviewed within 4 weekday(s)2. @obulat, if this PR is not ready for a review, please draft it to prevent reviewers from getting further unnecessary pings. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
308d183
to
9a3780e
Compare
9a3780e
to
e8e817c
Compare
Fixes
Related to #2553 by @obulat
Description
While working on the related issue, I realized there are two different unit tests for the single image result pages. These tests were mostly written for analytics events. Now that we have a better way of testing analytics requests in Playwright tests, I moved these tests to Playwright e2e tests.
I also added the
GET_MEDIA
test for audio.Testing Instructions
CI should pass and tests changes should be reasonable.
Checklist
Update index.md
).main
) or a parent feature branch.Developer Certificate of Origin
Developer Certificate of Origin