UtkershBasnet opened a new pull request, #63451:
URL: https://github.com/apache/airflow/pull/63451
All refactorings applied. Here's a summary:
DagCalendarTab.ts — 6 changes:
4× .waitFor({ state: "visible" }) → await expect(...).toBeVisible()
(web-first assertions)
1× Manual isVisible() + conditional waitFor("hidden") → await
expect(...).toBeHidden() ,this also simplifies the logic since toBeHidden()
passes immediately if the overlay was never present
1× waitForFunction() with document.querySelectorAll() → await
expect(this.calendarCells.first()).toBeVisible() (locator-based waiting)
dag-calendar-tab.spec.ts — 1 change:
const count = await ...; expect(count).toBeGreaterThan(0) → await
expect(calendar.activeCells).not.toHaveCount(0) (web-first assertion)
* closes: #63409
---
##### Was generative AI tooling used to co-author this PR?
- [x] Yes (please specify the tool below)
Generated-by: Claude Code following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
---
--
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]
