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

Try: Fix e2e tests 'visitSiteEditor' helper #68534

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

Mamaduka
Copy link
Member

@Mamaduka Mamaduka commented Jan 7, 2025

What?

Trying to fix flaky tests like #61884.

I see a welcome modal in report files, which might be causing test failures.

Why?

The welcome modals should be hidden before asserting any other locators on a page.

Testing Instructions

CI checks should be green.

Testing Instructions for Keyboard

Same.

Screenshots or screencast

test-failed-1

@Mamaduka Mamaduka added [Type] Bug An existing feature does not function as intended [Tool] E2E Test Utils /packages/e2e-test-utils labels Jan 7, 2025
@Mamaduka Mamaduka self-assigned this Jan 7, 2025
@Mamaduka Mamaduka requested a review from kevin940726 as a code owner January 7, 2025 15:16
Copy link

github-actions bot commented Jan 7, 2025

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: Mamaduka <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

Copy link

github-actions bot commented Jan 7, 2025

Flaky tests detected in beb2a28.
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/12654027603
📝 Reported issues:

@Mamaduka
Copy link
Member Author

Mamaduka commented Jan 8, 2025

Let's give this a try.

@Mamaduka Mamaduka merged commit ad4da87 into trunk Jan 8, 2025
69 checks passed
@Mamaduka Mamaduka deleted the try/fix-e2e-helper-visit-site-editor branch January 8, 2025 05:19
@github-actions github-actions bot added this to the Gutenberg 20.1 milestone Jan 8, 2025
@Mamaduka
Copy link
Member Author

Mamaduka commented Jan 8, 2025

It seems that the e2e test in question now fails most of the time on the trunk. I'll observe and revert this accordingly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Tool] E2E Test Utils /packages/e2e-test-utils [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant