test(browser): refactor browser tests for reliability and maintainability#8510

Merged

DrJKL merged 65 commits intomainfrom

drjkl/playwwrong

Feb 3, 2026

Commits on Jan 31, 2026

test: replace waitForTimeout with retrying assertions

DrJKL
and
ampcode-com
committed
test: replace expect() with waitFor() in fixture files

DrJKL
and
ampcode-com
committed
test: migrate brittle :nth-child() selectors to getByRole('radio')

DrJKL
and
ampcode-com
committed
refactor: extract DefaultGraphPositions constants from ComfyPage.ts

DrJKL
and
ampcode-com
committed
refactor: extract DebugHelper from ComfyPage.ts

DrJKL
and
ampcode-com
committed
refactor: extract SubgraphHelper from ComfyPage.ts

DrJKL
and
ampcode-com
committed
refactor: extract CanvasHelper from ComfyPage.ts

DrJKL
and
ampcode-com
committed
style: remove semicolons from litegraphUtils.ts

DrJKL
and
ampcode-com
committed
refactor: extract NodeOperationsHelper from ComfyPage.ts

DrJKL
and
ampcode-com
committed
refactor(browser_tests): remove deprecated proxy methods from ComfyPage

DrJKL
and
ampcode-com
committed
fix: remove setTimeout anti-pattern in toggleTheme

DrJKL
and
ampcode-com
committed
fix: remove setTimeout anti-patterns in test fixtures

DrJKL
and
ampcode-com
committed
feat: add centralized test selectors constants file

DrJKL
and
ampcode-com
committed
feat: add data-testid to sidebar components for stable test selectors

DrJKL
and
ampcode-com
committed
feat: add data-testid to dialog components for stable test selectors

DrJKL
and
ampcode-com
committed
refactor: replace fragile CSS selectors with data-testid in ComfyPage.ts

DrJKL
and
ampcode-com
committed
refactor: replace fragile CSS selectors in SidebarTab.ts and SettingDialog.ts

DrJKL
and
ampcode-com
committed
test: replace hardcoded positions with computed NodeReference positions

DrJKL
and
ampcode-com
committed
test: extract SettingsHelper from ComfyPage (Phase 4.1)

DrJKL
and
ampcode-com
committed
test: add deprecation notices to legacy node click methods (Phase 4.5)

DrJKL
and
ampcode-com
committed
test(browser): add BaseDialog class (Phase 5.2)

DrJKL
and
ampcode-com
committed
test(browser): add ContextMenu page object (Phase 5.1)

DrJKL
and
ampcode-com
committed
fix(browser_tests): use getByRole for interactive elements

DrJKL
and
ampcode-com
committed
fix(browser_tests): replace .nth() with semantic selectors

DrJKL
and
ampcode-com
committed
Bump container to 0.0.11 (Updated playwright)
DrJKL
committed
test: remove unused select2Nodes and fix flaky timeout

DrJKL
and
ampcode-com
committed
refactor: inject KeyboardHelper into ClipboardHelper

DrJKL
and
ampcode-com
committed
Use named imports.
DrJKL
committed
refactor: remove ComfyPage wrapper methods, use helpers directly

DrJKL
and
ampcode-com
committed
Maybe newer comfy core?
DrJKL
committed
refactor(tests): use DefaultGraphPositions constants for raw coordinates

DrJKL
and
ampcode-com
committed
fix: forward data-testid attrs through TreeExplorer to fix E2E tests

DrJKL
and
ampcode-com
committed
[automated] Apply ESLint and Oxfmt fixes
actions-user
committed
refactor: create and extend browser test helpers for ComfyPage extraction

DrJKL
and
ampcode-com
committed
refactor: wire helpers into ComfyPage and update test usages

DrJKL
and
ampcode-com
committed
Update agent docs to use local version for playwright tests.
DrJKL
committed
chore: enforce LF line endings for all text files

DrJKL
and
ampcode-com
committed

Commits on Feb 1, 2026