luizotavio32 commented on PR #42005: URL: https://github.com/apache/superset/pull/42005#issuecomment-4961003109
### CI note: playwright failures are unrelated to this change The `playwright-tests (chromium)` failures are not caused by this PR (which only touches the event-log `source` classification regex in `loggerMiddleware.ts` and its unit test): - **`clear-all-filters.spec.ts:38`** — flaky: timed out once, then **passed on retry #1**. - **`delete-display-control.spec.ts:49`** — fails with `locator.hover: Test timeout of 90000ms exceeded`. This is a dashboard display-control hover flow with no relationship to event logging. The spec was added very recently in #41999 and sits on top of the Ant Design v5→v6 upgrade (#41636), which is the base commit of this branch — a hover timeout on a display-control element is consistent with that upgrade / a not-yet-stabilized new test rather than anything here. `playwright-tests-experimental` passed, and the change is covered by the unit tests in `logger.test.ts`. Happy to rebase once master stabilizes, or a maintainer can re-run the required job. -- 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]
