passionworkeer commented on issue #63430: URL: https://github.com/apache/airflow/issues/63430#issuecomment-4045895211
A clean way to tackle this is to push the waiting logic into PoolsPage and expose only user-facing locators/actions from the spec. Suggested refactor: - replace waitForFunction / waitForTimeout with wait expect(locator).toBeVisible() or oHaveText() on the table / empty state / toast that actually gates the next step - avoid etworkidle; wait for the specific pool row count or a known row name instead - if the spec currently uses :has-text() or DOM queries, promote those into stable getByRole(...) / getByText(...) locators in the page object so the test reads intent, not implementation That usually removes most 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]
