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
Improve Playwright navigation utilities #1869
Conversation
Size Change: 0 B Total Size: 857 kB ℹ️ View Unchanged
|
Full-stack documentation: https://docs.openverse.org/_preview/1869 Please note that GitHub pages takes a little time to deploy newly pushed code, if the links above don't work or you see old versions, wait 5 minutes and try again. You can check the GitHub pages deployment action list to see the current status of the deployments. |
Based on the high urgency of this PR, the following reviewers are being gently reminded to review this PR: @zackkrida Excluding weekend1 days, this PR was updated 2 day(s) ago. PRs labelled with high urgency are expected to be reviewed within 2 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 |
e01523d
to
63e916b
Compare
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. Lots of big changes but the improvement in the tests is obvious 🚀
} | ||
} | ||
|
||
export const filters = { |
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.
Nice! I like this object-grouped model. In the future maybe the cookies could be handled similarly.
63e916b
to
df0b7c3
Compare
df0b7c3
to
d3bebd0
Compare
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.
Fixes
Fixes #1848 by @obulat
Description
This PR refactors Playwright navigation utilities:
waitForNavigation
withwaitForURL
to prevent race conditions (as mentioned in Replace deprecatedwaitForNavigation
method in Playwright tests #1848 description)setBreakpointCookie
,dismissBannersUsingCookies
andcloseFiltersUsingCookies
The CI failed on the first run. The failures however should be fixed by #1871 and #1808.
Testing Instructions
The Playwright tests should pass.
Checklist
Update index.md
).main
) or a parent feature branch.Developer Certificate of Origin
Developer Certificate of Origin