potiuk opened a new pull request, #67879:
URL: https://github.com/apache/airflow/pull/67879

   The Dag Calendar tab e2e tests 
(`airflow-core/src/airflow/ui/tests/e2e/specs/dag-calendar-tab.spec.ts`) read 
run states from the hover tooltip (`BasicTooltip`), which opens on a real 
`mouseenter` after a 500ms delay.
   
   In **Firefox**, the previous `cell.hover({ force: true })` teleports the 
pointer and does not fire `mouseenter`, and re-hovering an already-hovered cell 
never re-fires it — so the existing retry loop could never recover and the 
three state-reading tests timed out. Chromium was unaffected. Seen in [run 
26766429736](https://github.com/apache/airflow/actions/runs/26766429736/job/78940004431)
 (Firefox UI e2e tests, PROD image): 3 failed / 117 passed, all three in 
`dag-calendar-tab.spec.ts`.
   
   ## Fix
   
   In `DagCalendarTab.getManualRunStates()`, reset the pointer to `(0, 0)` and 
then move onto the cell centre **in steps** (`page.mouse.move(x, y, { steps: 5 
})`) so a genuine `mouseenter` is dispatched on each attempt, instead of the 
teleporting `hover({ force: true })`.
   
   Test-only change (e2e page object) — no production code touched. The UI 
compile/lint prek hook passes locally; the Firefox behaviour itself is verified 
by the e2e job in CI.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Opus 4.8)
   
   Generated-by: Claude Code (Opus 4.8) 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]

Reply via email to