kokhlo opened a new pull request, #44324: URL: https://github.com/apache/superset/pull/44324
### Summary `useExploreAdditionalActionsMenu.test.tsx` still drove the nested export submenu with 12 un-awaited `userEvent` calls after #43512 bumped `@testing-library/user-event` to v14. In v14 every call returns a promise and dispatches its pointer events across `delay: 0` yields inside RTL's `asyncWrapper`, so each un-awaited hover leaves a race that the following `findByText` has to absorb — on a loaded runner the 1000 ms budget expires before the second-level submenu (`Export All Data`) renders, producing the intermittent `Unable to find an element with the text: Export All Data` in `sharded-jest-tests`. Fixes #44322 ### Change Mechanical `await` on all 12 calls (hover ×8 / click ×4) across the four affected tests, lines 228–230 / 244–246 / 270–272 / 301–303. No other changes — same shape as the fix that already proved itself in `ExploreChartHeader.test.tsx` via #43512 (per the issue's natural-experiment analysis) and follows precedent #35918 / #35993 / #44315. ### Verification - Full test file run locally: 18/18 passing, both idle and under artificial CPU load (the failure mode only manifests on a loaded runner). - Diff is exactly 12 lines: `userEvent.` → `await userEvent.`. ### Additional context CI flake seen on master: run 34818352387 / job 103895789828 (2026-09-14). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
