Tooltip: Fix flaky unit test#77751
Conversation
|
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 If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
|
Size Change: 0 B Total Size: 7.82 MB ℹ️ View Unchanged
|
|
Flaky tests detected in f170830. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/25061653523
|
What?
Updates the
Tooltipcustom delay unit test to use a larger additional delay buffer.Why?
The test has been failing intermittently on trunk CI. Recent Unit Tests workflow failures showed the same assertion failing repeatedly:
Tooltip › delay › should respect custom delay prop when showing tooltipThe original test only left a 100ms gap between the default delay and the custom delay, which can be consumed by CI overhead before the "not visible yet" assertion runs.
How?
This keeps the behavior being tested the same, but increases the arbitrary additional delay from
100msto500ms. The test still verifies that the tooltip does not appear after the default delay, then appears after the custom delay.Testing Instructions
Run the focused Tooltip test:
Stress test the previously flaky test:
Expected result: all runs pass.