bito-code-review[bot] commented on PR #43512:
URL: https://github.com/apache/superset/pull/43512#issuecomment-5457015959

   <!-- Bito Reply -->
   The migration to v14 of `@testing-library/user-event` requires `await` for 
all interactions, as they are now asynchronous. While this PR has updated many 
`userEvent` calls across the codebase, it is possible that some instances, such 
as those involving `openComparator` or other complex interactions, were missed 
or require additional `await` statements to ensure tests remain stable and 
avoid flakiness.
   
   If you identify specific unawaited interactions in 
`AdhocFilterEditPopoverSimpleTabContent.test.tsx` or elsewhere, adding `await` 
to those `userEvent` calls is the correct approach to align with the v14 
migration requirements.


-- 
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]

Reply via email to