-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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 flaky 'Push to Global Styles' e2e test #51636
Conversation
requestUtils.deleteAllTemplates( 'wp_template' ), | ||
requestUtils.deleteAllTemplates( 'wp_template_part' ), | ||
] ); | ||
await requestUtils.activateTheme( 'emptytheme' ); |
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.
A small clean-up. The test doesn't create any templates or template parts, so there is no need to delete them.
postId: 'emptytheme//index', | ||
postType: 'wp_template', |
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.
Usually, it's better to specify a template or template part in test setups.
Size Change: 0 B Total Size: 1.4 MB ℹ️ 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.
Makes perfect sense to me 👍. It's a lot easier to read now too 💯 .
Flaky tests detected in 89b5d58. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/5309255597
|
* Fix flaky 'Push to Global Styles' e2e test * Simplify beforeAll
What?
Fixes #49942.
PR fixes flaky 'Push to Global Styles' e2e test by better scoping
Styles
andSettings
button locators.Why?
The button with the label "Styles" is no longer unique in the Site Editor.
Error log
Testing Instructions