chen0427ok commented on code in PR #59374:
URL: https://github.com/apache/airflow/pull/59374#discussion_r2616832154
##########
airflow-core/src/airflow/ui/tests/e2e/pages/LoginPage.ts:
##########
@@ -84,6 +84,9 @@ export class LoginPage extends BasePage {
await this.maximizeBrowser();
await this.navigateTo(LoginPage.loginUrl);
+
+ // Wait for login form to be fully loaded (especially important for first
test run)
+ await this.usernameInput.waitFor({ state: "visible", timeout: 120_000 });
Review Comment:
@choo121600 Thanks for the suggestion! I've run the tests locally with
`breeze testing ui-e2e-tests --test-pattern "dags-list.spec.ts"` and all tests
are now passing:
**Test Results:**
- Chromium: 3/3 passed
- Firefox: 3/3 passed ✨ (previously failed, now working!)
- WebKit: 3/3 passed
**Total: 9 passed in 54.8s**
The increased timeouts in `LoginPage.ts` (60s for navigation, 120s for
form load) successfully resolved the Firefox issues. The tests are ready for
review!
--
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]
