Skip to content
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

test: Fix flakey Playwright tests due to static assets #13199

Merged

Conversation

billyvg
Copy link
Member

@billyvg billyvg commented Aug 2, 2024

Our playwright tests can be flakey caused by the build process when generating static assets. Even though we check if file exists before symlinking, it can at times fail because file already exists. I have not dug/thought about why this happens -- just catch/ignore and move on.

Closes #11902

Our playwright tests can be flakey caused by the build process when generating static assets. Even though we check if file exists before symlinking, it can at times fail because file already exists. I have not dug/thought about why this happens -- just catch/ignore and move on.

Closes #11902
@billyvg billyvg force-pushed the test-fix-flakey-tests-when-generating-static-assets branch from 27eb3c4 to 80a3ec4 Compare August 2, 2024 16:01
@billyvg billyvg marked this pull request as ready for review August 2, 2024 16:19
@billyvg
Copy link
Member Author

billyvg commented Aug 2, 2024

It would be nice if the build error did not get swallowed up and instead get logged to CI. The only way to see that this was happening was via the playwright trace/debugger.

@billyvg billyvg changed the title fix(tracing): Fix flakey web vitals LCP test test: Fix flakey Playwright tests due to static assets Aug 2, 2024
@billyvg billyvg merged commit a7fbe01 into develop Aug 2, 2024
113 checks passed
@billyvg billyvg deleted the test-fix-flakey-tests-when-generating-static-assets branch August 2, 2024 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Flaky CI]: Html Webpack Plugin: Error: EEXIST: file already exists, symlink <...>
2 participants