Skip to content

Improve Playwright test patterns in backfill.spec.ts#63112

Merged
choo121600 merged 3 commits intoapache:mainfrom
stats-dev:e2e/63041-improve-backfill-patterns
Mar 9, 2026
Merged

Improve Playwright test patterns in backfill.spec.ts#63112
choo121600 merged 3 commits intoapache:mainfrom
stats-dev:e2e/63041-improve-backfill-patterns

Conversation

@stats-dev
Copy link
Contributor

Refactor backfill.spec.ts and BackfillPage.ts to align with Playwright best practices.

  • Replace page.waitForTimeout() with expect.poll()
  • Replace manual assertions with web-first toHaveCount() assertions
  • Replace CSS :has-text() with getByText() user-facing locator
  • Remove unused getTableColumnCount() method

closes: #63041

Was generative AI tooling used to co-author this PR?
  • Yes (Antigravity)

Generated-by: Antigravity following the guidelines

@boring-cyborg boring-cyborg bot added the area:UI Related to UI/UX. For Frontend Developers. label Mar 8, 2026
@stats-dev stats-dev marked this pull request as draft March 8, 2026 12:32
@stats-dev stats-dev marked this pull request as ready for review March 8, 2026 12:33
@stats-dev stats-dev force-pushed the e2e/63041-improve-backfill-patterns branch from 0d65510 to 2363bf7 Compare March 8, 2026 13:36
Copy link
Member

@choo121600 choo121600 left a comment

Choose a reason for hiding this comment

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

Cool, LGTM :)

@choo121600 choo121600 merged commit 3151443 into apache:main Mar 9, 2026
78 checks passed
@stats-dev stats-dev deleted the e2e/63041-improve-backfill-patterns branch March 9, 2026 02:46
@stats-dev
Copy link
Contributor Author

Cool, LGTM :)

Thanks for your guidance and helpful feedback! 👍

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 backfill.spec.ts

2 participants