I3eka commented on PR #43278: URL: https://github.com/apache/superset/pull/43278#issuecomment-5351944231
I traced the failing SQL Lab Playwright job before retrying it: all 10 tests stopped before the initial `/tabstateview/` POST because `syncQueryEditor` read `databases[dbId]` while the database state was not loaded yet, then the broad catch suppressed the exception. Fixed in bb094297a2 by validating a non-null database ID through the API without depending on the partial Redux state. The existing migration regression now covers the unloaded-state case; the focused Jest suite passes (79 passed, 1 skipped). -- 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]
