Failing test: Jest Tests.x-pack/platform/plugins/shared/lens/public/app_plugin/shared/edit_on_the_fly - Flyout wrapper inline mode should call onCancel when the header close button is clicked

[kibanamachine]

A test failed on a tracked branch

Test Details:

Field	Value
Report name	Jest Tests
Location	x-pack/platform/plugins/shared/lens/public/app_plugin/shared/edit_on_the_fly/flyout_wrapper.test.tsx
Duration	6.01s
Config path	x-pack/platform/plugins/shared/lens/jest.config.js
Code Owners	elastic/kibana-visualizations

Error: thrown: "Exceeded timeout of 5000 ms for a test.
Add a timeout value to this test to increase the timeout, if this is a long-running test. See https://jestjs.io/docs/api#testname-fn-timeout."
    at it (x-pack/platform/plugins/shared/lens/public/app_plugin/shared/edit_on_the_fly/flyout_wrapper.test.tsx:89:5)
    at _dispatchDescribe (node_modules/jest-circus/build/index.js:91:26)
    at describe (node_modules/jest-circus/build/index.js:55:5)
    at describe (x-pack/platform/plugins/shared/lens/public/app_plugin/shared/edit_on_the_fly/flyout_wrapper.test.tsx:57:3)
    at _dispatchDescribe (node_modules/jest-circus/build/index.js:91:26)
    at describe (node_modules/jest-circus/build/index.js:55:5)
    at Object.describe (x-pack/platform/plugins/shared/lens/public/app_plugin/shared/edit_on_the_fly/flyout_wrapper.test.tsx:56:1)
    at Runtime._execModule (node_modules/jest-runtime/build/index.js:1439:24)
    at Runtime._loadModule (node_modules/jest-runtime/build/index.js:1022:12)
    at Runtime.requireModule (node_modules/jest-runtime/build/index.js:882:12)
    at jestAdapter (node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:77:13)
    at runTestInternal (node_modules/jest-runner/build/runTest.js:367:16)
    at runTest (node_modules/jest-runner/build/runTest.js:444:34)

First failure: kibana-on-merge - main

[infra-vault-gh-plugin-prod]

Pinging @elastic/kibana-visualizations (Team:Visualizations)

[github-actions]

Test needs an update — userEvent.click overhead tips a heavy render past the 5s budget

The only case using await userEvent.click (flyout_wrapper.test.tsx#L89-L95) hit the 5000 ms timeout, while the three synchronous cases that render the same component passed — the async interaction machinery, not the assertion, ran out of budget.

Details

Proposed fix

Remove the async step — the click triggers a synchronous callback, so there is nothing to await.

• flyout_wrapper.test.tsx:92 — replace await userEvent.click(screen.getByTestId('euiFlyoutCloseButton')) with fireEvent.click(screen.getByTestId('euiFlyoutCloseButton')) (add fireEvent to the @testing-library/react import). RTL wraps fireEvent in act, so the existing expect(onCancel).toHaveBeenCalledTimes(1) still reads synchronously on the next line.

If the failure recurs after this, the render itself is the dominant cost across the whole file and the fallback is a jest.setTimeout/testTimeout bump — but the userEvent swap should be enough on its own.

Root cause & evidence

• The close handler is synchronous: onClick={onCancel} — no promise or timer sits between the click and the callback, so nothing genuinely needs awaiting.
• This is the only case that uses await userEvent.click; the other three render the same component synchronously and pass, so the render alone fits the 5s budget. userEvent's pointer/actionability machinery — the button is wrapped in an EuiToolTip that arms show/hide timers on hover — adds enough overhead under CI parallel load to tip past 5000 ms.
• Classic Jest/RTL cost-not-waiting timeout: the test ran out of time, not an assertion failure or a product race.

Additional context

• Ruled out: not a product bug (the callback path is a plain synchronous onClick) and not an assertion bug (the assertion never ran — the timeout fired during the click).
• Verification: node scripts/jest x-pack/platform/plugins/shared/lens/public/app_plugin/shared/edit_on_the_fly/flyout_wrapper.test.tsx

Tip

Marked "AI-fixable". Add ai:fix-flaky to request a fix now; otherwise it will be requested automatically if the test fails again.

Generated by Failed Test Investigator for #290124 · claude · opus · 141.8 AIC · ⌖ 59.8 AIC · ⊞ 13.2K · ◷

[kibanamachine]

New failure: kibana-on-merge - main

Error message matches a failure already reported on this issue.

[github-actions]

✅ Flaky Test Fixer completed successfully!

Generated by Flaky Test Fixer for #290124

[github-actions]

➡️ A fix PR is ready for review: #290263

Swaps await userEvent.click for fireEvent.click on the flyout close button, removing the async-interaction overhead that tipped this synchronous-callback test past the 5s Jest budget.

Generated by Flaky Test Fixer for #290124 · claude · opus · 270.2 AIC · ⌖ 64.9 AIC · ⊞ 14.5K · ◷