rusackas opened a new pull request, #36723: URL: https://github.com/apache/superset/pull/36723
## Summary Optimizes the DatasourceEditorCurrency tests that were timing out on CI runners. ### Changes: - **Use `fastRender` instead of `asyncRender`** - Avoids unnecessary `waitFor` wrapper around render, reducing test time by ~30% - **Extract `setupCurrencySection` helper** - Reduces code duplication across tests - **Remove redundant `cleanupAsyncOperations` calls** - Only needed in `afterEach`, not mid-test - **Increase timeout to 60s** - Safety net for slower CI runners ### Performance: - Local run time: ~21s (down from ~29s) - More reliable on CI due to faster execution staying within timeout ## Test plan - [x] Tests pass locally - [x] Pre-commit hooks pass 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
