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

Fix flakiness of saving entities in the site editor #51728

Merged
merged 2 commits into from
Jun 26, 2023

Conversation

kevin940726
Copy link
Member

What?

Save flakiness of saving entities in the site editor e2e tests.

Why?

I've been noticing some flakiness when running the template-part e2e tests in the site editor (especially can detach blocks from a template part).

How?

Instead of waiting for a CSS class, which is an implementation detail, we wait for a notice snackbar instead.

Testing Instructions

CI should pass.

@kevin940726 kevin940726 added the [Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests. label Jun 21, 2023
@github-actions
Copy link

github-actions bot commented Jun 21, 2023

Flaky tests detected in 6e5e1ce.
Some tests passed with failed attempts. The failures may not be related to this commit but are still reported for visibility. See the documentation for more information.

🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/5373278307
📝 Reported issues:

@kevin940726 kevin940726 requested a review from Mamaduka June 21, 2023 09:22
@kevin940726 kevin940726 marked this pull request as ready for review June 21, 2023 09:22
Copy link
Contributor

@apeatling apeatling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kevin940726 kevin940726 force-pushed the fix/saving-site-editor-flakiness branch from 6b5ed01 to 6e5e1ce Compare June 26, 2023 02:50
@kevin940726 kevin940726 enabled auto-merge (squash) June 26, 2023 06:04
@kevin940726 kevin940726 merged commit 74f3ce3 into trunk Jun 26, 2023
@kevin940726 kevin940726 deleted the fix/saving-site-editor-flakiness branch June 26, 2023 06:42
@github-actions github-actions bot added this to the Gutenberg 16.2 milestone Jun 26, 2023
sethrubenstein pushed a commit to pewresearch/gutenberg that referenced this pull request Jul 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants