Skip to content

Conversation

@vbottu
Copy link
Contributor

@vbottu vbottu commented Dec 21, 2025

What this PR does

Adds E2E tests for the dashboard/home page to verify metrics display correctly.

Changes

New files:

  • tests/e2e/pages/HomePage.ts - Page object for dashboard elements
  • tests/e2e/specs/home-dashboard.spec.ts - E2E test specs

Tests added

  1. should display dashboard stats section with DAG metrics - Verifies active, running, and failed DAG counts
  2. should display health status badges - Verifies metadatabase, scheduler, and triggerer health
  3. should navigate to filtered DAGs list when clicking stats cards - Verifies card click navigation
  4. should display welcome heading on dashboard - Verifies basic dashboard load
  5. should refresh and display updated metrics on page reload - Verifies metrics persist after reload
  6. should display historical metrics section with recent runs - Verifies DAG runs and task instances
  7. should handle DAG import errors display when errors exist - Verifies import errors behavior

Testing

All 7 tests pass locally on Chromium.

closes: #59306


@boring-cyborg boring-cyborg bot added the area:UI Related to UI/UX. For Frontend Developers. label Dec 21, 2025
@vatsrahul1001
Copy link
Contributor

@vbottu can you check CI failures? Also, you need to rebase to get changes in PR #59692

@vbottu
Copy link
Contributor Author

vbottu commented Dec 23, 2025

@vbottu can you check CI failures? Also, you need to rebase to get changes in PR #59692

Sure Let me check the failures

@vbottu vbottu force-pushed the add-e2e-home-dashboard-metrics-59306 branch from bd00bea to 559158e Compare December 28, 2025 04:49
@vbottu
Copy link
Contributor Author

vbottu commented Dec 28, 2025

@vatsrahul1001 , could you review when you get a chance

@vatsrahul1001
Copy link
Contributor

@vbottu Thanks for implementing review comments. PR looks in good shape, We can merge after fixing this review comment

@vbottu
Copy link
Contributor Author

vbottu commented Dec 29, 2025

@vatsrahul1001 Addressed your comments, thank you for reviewing the changes

@vatsrahul1001 vatsrahul1001 merged commit f6d2948 into apache:main Dec 30, 2025
75 checks passed
@vatsrahul1001
Copy link
Contributor

@vatsrahul1001 Addressed your comments, thank you for reviewing the changes

Thanks @vbottu. LGTM

Subham-KRLX pushed a commit to Subham-KRLX/airflow that referenced this pull request Jan 2, 2026
Add E2E tests for dashboard metrics display
@vbottu vbottu deleted the add-e2e-home-dashboard-metrics-59306 branch January 6, 2026 03:41
stegololz pushed a commit to stegololz/airflow that referenced this pull request Jan 9, 2026
Add E2E tests for dashboard metrics display
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.

UI E2E Test || HOME-001: Verify dashboard metrics display

2 participants