Skip to content

Test(UI): Add E2E test for Calendar tab functionality #60436

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
vighneshtule wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Test(UI): Add E2E test for Calendar tab functionality #60436

vighneshtule wants to merge 5 commits into from
+89 −15

Conversation

@vighneshtule
Copy link
Contributor

@vighneshtule vighneshtule commented Jan 12, 2026 •
edited
Loading

This PR adds End-to-End (E2E) tests for the Calendar tab on the DAG details page.

Changes:

  • Modified DagsPage.ts to include locators for the Calendar tab and grid, and added a verifyCalendarTab method.
  • Added dag-calendar.spec.ts to verify:
    • Navigation to the Calendar tab.
    • Visibility of the month grid.
    • Presence of run markers (filled cells) after triggering a DAG run.
    • Interactivity of run markers (clicking).

related: #59544

Was generative AI tooling used to co-author this PR?

  • Yes

@boring-cyborg boring-cyborg bot added the area:UI Related to UI/UX. For Frontend Developers. label Jan 12, 2026
@vighneshtule
Copy link
Contributor Author

vighneshtule commented Jan 13, 2026

Hey maintainers,
I would really appreciate it if you could review this PR and share any suggestions on how I can improve it further. Thank you!

choo121600
choo121600 reviewed Jan 13, 2026
View reviewed changes
airflow-core/src/airflow/ui/tests/e2e/pages/DagsPage.ts
Copy link
Member

@choo121600 choo121600 Jan 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like the static checks and e2e tests are failing. Could you please check on this?

Copy link
Contributor Author

@vighneshtule vighneshtule Jan 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hey its getting hard for me to resolve this checks would anyone like to help me out

Copy link
Contributor

@vatsrahul1001 vatsrahul1001 Jan 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vighneshtule could you elaborate more on what issues you are facing fixes the failures?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@choo121600 choo121600 choo121600 left review comments

@vatsrahul1001 vatsrahul1001 Awaiting requested review from vatsrahul1001 vatsrahul1001 is a code owner

@bbovenzi bbovenzi Awaiting requested review from bbovenzi bbovenzi is a code owner

@pierrejeambrun pierrejeambrun Awaiting requested review from pierrejeambrun pierrejeambrun is a code owner

@ryanahamilton ryanahamilton Awaiting requested review from ryanahamilton ryanahamilton is a code owner

@shubhamraj-git shubhamraj-git Awaiting requested review from shubhamraj-git shubhamraj-git is a code owner

@guan404ming guan404ming Awaiting requested review from guan404ming guan404ming is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@vighneshtule @vatsrahul1001 @choo121600