Skip to content

Improve Playwright test patterns in pools.spec.ts #63430#64328

Merged
choo121600 merged 2 commits into
apache:mainfrom
haseebmalik18:improve-playwright-pools-test-patterns
Mar 31, 2026
Merged

Improve Playwright test patterns in pools.spec.ts #63430#64328
choo121600 merged 2 commits into
apache:mainfrom
haseebmalik18:improve-playwright-pools-test-patterns

Conversation

@haseebmalik18

Copy link
Copy Markdown
Contributor

Improve Playwright test patterns in PoolsPage.ts to align with best practices.

  • Replace waitForLoadState("networkidle") with UI-state assertions
  • Remove waitForTimeout() in favor of web-first assertions
  • Replace CSS selectors (input[name="name"], input[type="number"], textarea) with getByLabel()
  • Replace page.locator('[data-testid="card-list"]') with page.getByTestId("card-list")

closes: #63430

@boring-cyborg boring-cyborg Bot added the area:UI Related to UI/UX. For Frontend Developers. label Mar 27, 2026
Comment thread airflow-core/src/airflow/ui/tests/e2e/pages/PoolsPage.ts Outdated
@haseebmalik18 haseebmalik18 force-pushed the improve-playwright-pools-test-patterns branch 2 times, most recently from 08345e6 to 939db7c Compare March 30, 2026 05:34
@haseebmalik18 haseebmalik18 force-pushed the improve-playwright-pools-test-patterns branch from 939db7c to 3faa85c Compare March 30, 2026 21:42

@choo121600 choo121600 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, Thanks :)

@choo121600 choo121600 merged commit c7fc3b8 into apache:main Mar 31, 2026
83 checks passed
Subham-KRLX pushed a commit to Subham-KRLX/airflow that referenced this pull request Apr 3, 2026
…e#64328)

Co-authored-by: Yeonguk Choo <choo121600@gmail.com>
Suraj-kumar00 pushed a commit to Suraj-kumar00/airflow that referenced this pull request Apr 7, 2026
…e#64328)

Co-authored-by: Yeonguk Choo <choo121600@gmail.com>
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.

E2E: Improve Playwright test patterns in pools.spec.ts

2 participants