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
Fix Storybook tests by waiting for URL instead of a port #1137
Conversation
Size Change: -2.62 kB (0%) Total Size: 843 kB
ℹ️ View Unchanged
|
Full-stack documentation: https://docs.openverse.org/_preview/1137 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. |
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.
Wow, that was an strange problem and an interesting fix!
* Update TW line-heights * Update snapshots * Update storybook snapshots * Use the named styles * Update frontend/src/styles/tailwind.css Co-authored-by: Zack Krida <zackkrida@pm.me> * Fix storybook tests running before server is ready --------- Co-authored-by: Zack Krida <zackkrida@pm.me>
Co-authored-by: Dhruv Bhanushali <hi@dhruvkb.dev>
Fixes
Fixes #1135 by @obulat
Description
Playwright accepts either
port
orurl
configuration for the webserver, and checks if one of them is ready before running tests. It seems that theport
is "ready" before Storybook is fully built, which makes the first couple of tests fail.This PR replaces the
port
with the first test's URL to make sure that the Storybook is fully built and ready to run tests.Note: the fix has been merge as part of #946, but this PR still should be merged as it uses a better url than the one in main now.
Testing Instructions
The CI should pass. These changes were added to #1132 to make it pass the CI tests.
Checklist
Update index.md
).main
) ora parent feature branch.
errors.
Developer Certificate of Origin
Developer Certificate of Origin