Skip to content

Fix flaky DagRunsTab E2E test race condition in markRunAs#63890

Merged
vatsrahul1001 merged 1 commit intoapache:mainfrom
choo121600:fix/flaky-dagrunstab-race-markrunas
Mar 19, 2026
Merged

Fix flaky DagRunsTab E2E test race condition in markRunAs#63890
vatsrahul1001 merged 1 commit intoapache:mainfrom
choo121600:fix/flaky-dagrunstab-race-markrunas

Conversation

@choo121600
Copy link
Member

Handle the case where the menu option becomes disabled while transitioning the dag run state between reading the state badge and opening the “mark-as” menu.
If the option is already disabled, close the menu with Escape and return early to prevent flaky tests.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@boring-cyborg boring-cyborg bot added the area:UI Related to UI/UX. For Frontend Developers. label Mar 18, 2026
@choo121600 choo121600 changed the title Fix flaky DagRuns E2E test race condition in markRunAs Fix flaky DagRunsTab E2E test race condition in markRunAs Mar 18, 2026
@choo121600
Copy link
Member Author

not work😢
I'll check it tomorrow

@choo121600
Copy link
Member Author

choo121600 commented Mar 19, 2026

This is the flaky behavior in requiredAction.spec.ts and isn’t related to my changes.
Noted in #63036.

@vatsrahul1001 vatsrahul1001 merged commit 0b18161 into apache:main Mar 19, 2026
149 of 151 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:UI Related to UI/UX. For Frontend Developers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants