passionworkeer commented on issue #63431: URL: https://github.com/apache/airflow/issues/63431#issuecomment-4053006973
One practical pattern here is to push the waiting logic into the page object and expose semantic helpers like wait providersPage.waitUntilLoaded() or wait providersPage.expectProviderVisible(name). Then the spec can replace waitForTimeout() / etworkidle with locator assertions such as wait expect(locator).toBeVisible() and oHaveCount(). If there are places using page.evaluate() just to inspect DOM state, it’s usually a sign the page object should expose a user-facing locator instead. That keeps the test web-first and reduces flakiness without changing coverage. -- 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]
