Dante 19fff29204 test: backfill e2e coverage gaps for toolkit widgets, minimap, mask editor, painter (#11183) ...

## Summary

Backfills missing e2e test coverage identified in the [FixIt
Burndown](https://www.notion.so/comfy-org/FixIt-Burndown-32e6d73d365080609a81cdc9bc884460)
audit. Adds 39 new behavioral tests across 5 spec files with zero
test-code overlap.

## Changes

- **What**: New e2e specs for Image Crop (6 tests) and Curve Widget (6
tests). Deepened coverage for Minimap (+6), Mask Editor (+10), Painter
(+11).
- **New fixtures**: `curve_widget.json`, updated
`image_crop_widget.json`

## Test Inventory

| Spec | New tests | Coverage area |
|---|---|---|
| `imageCrop.spec.ts` | 6 | Empty state, bounding box inputs, ratio
selector/presets, lock toggle, programmatic value update |
| `curveWidget.spec.ts` | 6 | SVG render, click-to-add point,
drag-to-reshape, Ctrl+click remove, interpolation mode switch, min-2
guard |
| `minimap.spec.ts` | +6 | Click-to-pan, drag-to-pan, zoom viewport
shrink, node count changes, workflow reload, pan state reflection |
| `maskEditor.spec.ts` | +10 | Brush drawing, undo/redo, clear, cancel,
invert, Ctrl+Z, tool panel/switching, brush settings, save with mock,
eraser |
| `painter.spec.ts` | +11 | Clear, eraser, control visibility toggle,
brush size slider, stroke width comparison, canvas dimensions,
background color, multi-stroke accumulate, color picker, opacity,
partial erase |

## Review Focus

- Mask editor tests use `.maskEditor_toolPanelContainer` class selectors
— may need test-id hardening later
- Painter slider interaction tests could be flaky if slider layout
changes
- All canvas pixel-count assertions use `expect.poll()` with timeouts
for reliability

## Test plan
- [ ] CI passes all new/modified specs
- [ ] No duplicate coverage with existing tests (verified via grep
before writing)
- [ ] No `waitForTimeout` usage (confirmed)

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-11183-test-backfill-e2e-coverage-gaps-for-toolkit-widgets-minimap-mask-editor-painter-3416d73d3650819ca33edd1f27b9651a)
by [Unito](https://www.unito.io)

---------

Co-authored-by: GitHub Action <action@github.com>

2026-04-16 09:48:13 +00:00

..

colorPalette.spec.ts-snapshots

feat: surface missing models in Error Tab for OSS and remove legacy dialog (#9921)

2026-03-15 22:46:47 +09:00

copyPaste.spec.ts-snapshots

feat: surface missing models in Error Tab for OSS and remove legacy dialog (#9921)

2026-03-15 22:46:47 +09:00

dialogs

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

domWidget.spec.ts-snapshots

Component: The Rest of the PrimeVue buttons (#7649)

2025-12-29 15:03:34 -08:00

execution.spec.ts-snapshots

fix: improve error overlay design and indicator placement (#10564)

2026-03-27 09:11:26 +09:00

graphCanvasMenu.spec.ts-snapshots

feat: surface missing models in Error Tab for OSS and remove legacy dialog (#9921)

2026-03-15 22:46:47 +09:00

groupNode.spec.ts-snapshots

test: audit skipped tests — prune stale, re-enable stable, remove dead code (#10312)

2026-03-28 13:08:52 -07:00

imageCompare.spec.ts-snapshots

test: add E2E tests for ImageCompare widget (#10767)

2026-04-13 00:09:50 -04:00

interaction.spec.ts-snapshots

test: harden 82 Playwright specs for deterministic CI runs (#10967)

2026-04-09 20:50:56 -07:00

load3d

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

loadWorkflowInMedia.spec.ts-snapshots

test: harden 82 Playwright specs for deterministic CI runs (#10967)

2026-04-09 20:50:56 -07:00

lodThreshold.spec.ts-snapshots

feat: surface missing models in Error Tab for OSS and remove legacy dialog (#9921)

2026-03-15 22:46:47 +09:00

maskEditor.spec.ts-snapshots

add basic mask editor tests (#10574)

2026-03-27 16:04:36 -04:00

minimap.spec.ts-snapshots

test: add minimap e2e tests for close button, viewport, and pan (#10596)

2026-03-28 23:19:04 -07:00

mobileBaseline.spec.ts-snapshots

fix: improve error overlay design and indicator placement (#10564)

2026-03-27 09:11:26 +09:00

nodeBadge.spec.ts-snapshots

feat: surface missing models in Error Tab for OSS and remove legacy dialog (#9921)

2026-03-15 22:46:47 +09:00

nodeDisplay.spec.ts-snapshots

Style: Font Consistency (#7220)

2025-12-08 19:48:11 -07:00

nodeSearchBox.spec.ts-snapshots

feat: surface missing models in Error Tab for OSS and remove legacy dialog (#9921)

2026-03-15 22:46:47 +09:00

noteNode.spec.ts-snapshots

Style: Font Consistency (#7220)

2025-12-08 19:48:11 -07:00

painter.spec.ts-snapshots

test: Painter Widget E2E Test Plan (#10846)

2026-04-13 00:13:04 -04:00

primitiveNode.spec.ts-snapshots

Disable control widgets on link to parent (#8112)

2026-01-16 22:11:00 -07:00

propertiesPanel

fix: missing-asset correctness follow-ups from #10856 (#11233)

2026-04-15 10:58:24 +00:00

queue

feat: add eslint-plugin-playwright via oxlint JS plugins (#11136)

2026-04-11 01:25:14 +00:00

recordAudio.spec.ts-snapshots

feat: surface missing models in Error Tab for OSS and remove legacy dialog (#9921)

2026-03-15 22:46:47 +09:00

rerouteNode.spec.ts-snapshots

feat: surface missing models in Error Tab for OSS and remove legacy dialog (#9921)

2026-03-15 22:46:47 +09:00

rightClickMenu.spec.ts-snapshots

[feat] Improve group title layout (#9839)

2026-03-24 12:25:18 -07:00

saveImageAndWebp.spec.ts-snapshots

fix: center ComfyUI logo in sidebar menu button with chevron (#9722)

2026-03-12 01:10:26 -07:00

selectionToolbox.spec.ts-snapshots

test: update E2E to expect missing nodes overlay on tab switch (#10190)

2026-03-17 10:46:19 -07:00

sidebar

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

subgraph

test: Add waitForWorkflowIdle & remove redundant nextFrame (#11264)

2026-04-15 16:52:41 +00:00

templates.spec.ts-snapshots

feat: add Tag component from design system and rename SquareChip (#10650)

2026-03-29 08:27:53 +09:00

topbar

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

viewport.spec.ts-snapshots

fix: improve error overlay design and indicator placement (#10564)

2026-03-27 09:11:26 +09:00

vueNodes

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

widget.spec.ts-snapshots

feat: surface missing models in Error Tab for OSS and remove legacy dialog (#9921)

2026-03-15 22:46:47 +09:00

actionbar.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

appModeArrange.spec.ts

test: App mode - additional app mode coverage (#11194)

2026-04-15 11:42:22 +00:00

appModeDropdownClipping.spec.ts

fix: remove redundant and counterproductive e2e timeout overrides (#11110)

2026-04-10 19:47:20 +00:00

appModePruning.spec.ts

refactor: fix lint errors in tests (#11182)

2026-04-13 15:50:11 +00:00

appModeVueNodeSwitch.spec.ts

test: App mode - additional app mode coverage (#11194)

2026-04-15 11:42:22 +00:00

appModeWelcome.spec.ts

test: App mode - additional app mode coverage (#11194)

2026-04-15 11:42:22 +00:00

appModeWidgetRename.spec.ts

fix: remove redundant and counterproductive e2e timeout overrides (#11110)

2026-04-10 19:47:20 +00:00

appModeWidgetValues.spec.ts

fix: remove redundant and counterproductive e2e timeout overrides (#11110)

2026-04-10 19:47:20 +00:00

assetHelper.spec.ts

test: migrate browser_tests/ to @e2e/ path alias and add lint rule (#10958)

2026-04-08 11:28:59 -07:00

backgroundImageUpload.spec.ts

test: migrate browser_tests/ to @e2e/ path alias and add lint rule (#10958)

2026-04-08 11:28:59 -07:00

bottomPanel.spec.ts

test: add E2E tests for bottom panel core behaviors (#10814)

2026-04-14 23:33:11 +00:00

bottomPanelLogs.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

bottomPanelShortcuts.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

browserTabTitle.spec.ts

test: Remove unnecessary setup, UseNewMenu and waitForNodes calls (#11237)

2026-04-14 17:40:23 +00:00

builderReorder.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

builderSaveFlow.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

canvasModeSelector.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

changeTracker.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

changeTrackerLoadGuard.spec.ts

test: harden 82 Playwright specs for deterministic CI runs (#10967)

2026-04-09 20:50:56 -07:00

cloud-asset-default.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

cloud.spec.ts

refactor: fix lint errors in tests (#11182)

2026-04-13 15:50:11 +00:00

collapsedNodeLinks.spec.ts

test: Auto wait for nodes after loadWorkflow in vue-node tests (#11238)

2026-04-14 17:30:49 +00:00

colorPalette.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

commands.spec.ts

test: harden 82 Playwright specs for deterministic CI runs (#10967)

2026-04-09 20:50:56 -07:00

confirmDialogTextWrap.spec.ts

test: migrate browser_tests/ to @e2e/ path alias and add lint rule (#10958)

2026-04-08 11:28:59 -07:00

containAudit.spec.ts

test: migrate browser_tests/ to @e2e/ path alias and add lint rule (#10958)

2026-04-08 11:28:59 -07:00

copyPaste.spec.ts

refactor(test): use canvasOps.clickEmptySpace in copyPaste spec (#10991)

2026-04-16 09:44:06 +00:00

customIcons.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

defaultKeybindings.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

dialog.spec.ts

Add missing dialog tests (#11133)

2026-04-14 14:31:31 -07:00

domWidget.spec.ts

feat: add eslint-plugin-playwright via oxlint JS plugins (#11136)

2026-04-11 01:25:14 +00:00

errorDialog.spec.ts

Add missing dialog tests (#11133)

2026-04-14 14:31:31 -07:00

errorOverlay.spec.ts

fix: missing-asset correctness follow-ups from #10856 (#11233)

2026-04-15 10:58:24 +00:00

execution.spec.ts

fix: remove redundant and counterproductive e2e timeout overrides (#11110)

2026-04-10 19:47:20 +00:00

extensionAPI.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

featureFlags.spec.ts

feat: add eslint-plugin-playwright via oxlint JS plugins (#11136)

2026-04-11 01:25:14 +00:00

focusMode.spec.ts

fix: include focusMode in splitter refresh key to prevent panel resize (#11295)

2026-04-16 13:43:02 +09:00

graph.spec.ts

feat: add eslint-plugin-playwright via oxlint JS plugins (#11136)

2026-04-11 01:25:14 +00:00

graphCanvasMenu.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

groupCopyPaste.spec.ts

test: harden 82 Playwright specs for deterministic CI runs (#10967)

2026-04-09 20:50:56 -07:00

groupNode.spec.ts

test: Add waitForWorkflowIdle & remove redundant nextFrame (#11264)

2026-04-15 16:52:41 +00:00

groupSelectChildren.spec.ts

test: Add waitForWorkflowIdle & remove redundant nextFrame (#11264)

2026-04-15 16:52:41 +00:00

imageCompare.spec.ts

test: address review nits for image compare E2E (#11260)

2026-04-15 10:50:44 -04:00

imageCrop.spec.ts

test: backfill e2e coverage gaps for toolkit widgets, minimap, mask editor, painter (#11183)

2026-04-16 09:48:13 +00:00

imagePastePriority.spec.ts

test: harden 82 Playwright specs for deterministic CI runs (#10967)

2026-04-09 20:50:56 -07:00

interaction.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

jobHistoryActions.spec.ts

test: Remove unnecessary setup, UseNewMenu and waitForNodes calls (#11237)

2026-04-14 17:40:23 +00:00

keybindingPresets.spec.ts

fix: remove redundant and counterproductive e2e timeout overrides (#11110)

2026-04-10 19:47:20 +00:00

keybindings.spec.ts

test: harden 82 Playwright specs for deterministic CI runs (#10967)

2026-04-09 20:50:56 -07:00

linearMode.spec.ts

test: Remove unnecessary setup, UseNewMenu and waitForNodes calls (#11237)

2026-04-14 17:40:23 +00:00

litegraphEvent.spec.ts

test: migrate browser_tests/ to @e2e/ path alias and add lint rule (#10958)

2026-04-08 11:28:59 -07:00

loadWorkflowInMedia.spec.ts

test: harden 82 Playwright specs for deterministic CI runs (#10967)

2026-04-09 20:50:56 -07:00

lodThreshold.spec.ts

test: harden 82 Playwright specs for deterministic CI runs (#10967)

2026-04-09 20:50:56 -07:00

maskEditor.spec.ts

test: backfill e2e coverage gaps for toolkit widgets, minimap, mask editor, painter (#11183)

2026-04-16 09:48:13 +00:00

menu.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

minimap.spec.ts

test: backfill e2e coverage gaps for toolkit widgets, minimap, mask editor, painter (#11183)

2026-04-16 09:48:13 +00:00

mobileBaseline.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

nodeBadge.spec.ts

test: harden 82 Playwright specs for deterministic CI runs (#10967)

2026-04-09 20:50:56 -07:00

nodeContextMenuOverflow.spec.ts

test: Add waitForWorkflowIdle & remove redundant nextFrame (#11264)

2026-04-15 16:52:41 +00:00

nodeDisplay.spec.ts

feat: add eslint-plugin-playwright via oxlint JS plugins (#11136)

2026-04-11 01:25:14 +00:00

nodeGhostPlacement.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

nodeHelp.spec.ts

test: Remove unnecessary setup, UseNewMenu and waitForNodes calls (#11237)

2026-04-14 17:40:23 +00:00

nodeLibraryEssentials.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

nodeSearchBox.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

nodeSearchBoxV2.spec.ts

feat: add eslint-plugin-playwright via oxlint JS plugins (#11136)

2026-04-11 01:25:14 +00:00

nodeSearchBoxV2Extended.spec.ts

feat: add eslint-plugin-playwright via oxlint JS plugins (#11136)

2026-04-11 01:25:14 +00:00

noteNode.spec.ts

test: migrate browser_tests/ to @e2e/ path alias and add lint rule (#10958)

2026-04-08 11:28:59 -07:00

outputHistory.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

painter.spec.ts

test: backfill e2e coverage gaps for toolkit widgets, minimap, mask editor, painter (#11183)

2026-04-16 09:48:13 +00:00

pasteImageContextMenu.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

performance.spec.ts

feat: add eslint-plugin-playwright via oxlint JS plugins (#11136)

2026-04-11 01:25:14 +00:00

primitiveNode.spec.ts

test: migrate browser_tests/ to @e2e/ path alias and add lint rule (#10958)

2026-04-08 11:28:59 -07:00

recordAudio.spec.ts

fix: remove redundant and counterproductive e2e timeout overrides (#11110)

2026-04-10 19:47:20 +00:00

releaseNotifications.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

remoteWidgets.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

rendererToggleStability.spec.ts

test: migrate browser_tests/ to @e2e/ path alias and add lint rule (#10958)

2026-04-08 11:28:59 -07:00

rerouteNode.spec.ts

test: Remove unnecessary setup, UseNewMenu and waitForNodes calls (#11237)

2026-04-14 17:40:23 +00:00

resultGallery.spec.ts

test: Remove unnecessary setup, UseNewMenu and waitForNodes calls (#11237)

2026-04-14 17:40:23 +00:00

rightClickMenu.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

rightSidePanelTabs.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

saveImageAndWebp.spec.ts

test: Auto wait for nodes after loadWorkflow in vue-node tests (#11238)

2026-04-14 17:30:49 +00:00

selectionRectangle.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

selectionToolbox.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

selectionToolboxActions.spec.ts

test: Add waitForWorkflowIdle & remove redundant nextFrame (#11264)

2026-04-15 16:52:41 +00:00

selectionToolboxMoreActions.spec.ts

test: expand E2E coverage for toolbox actions (#10968)

2026-04-14 23:20:45 +00:00

selectionToolboxSubmenus.spec.ts

test: Add waitForWorkflowIdle & remove redundant nextFrame (#11264)

2026-04-15 16:52:41 +00:00

subgraphPromotedWidgetPanel.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

templateFitView.spec.ts

test: migrate browser_tests/ to @e2e/ path alias and add lint rule (#10958)

2026-04-08 11:28:59 -07:00

templates.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

toastNotifications.spec.ts

test: Remove unnecessary setup, UseNewMenu and waitForNodes calls (#11237)

2026-04-14 17:40:23 +00:00

userSelectView.spec.ts

test: migrate browser_tests/ to @e2e/ path alias and add lint rule (#10958)

2026-04-08 11:28:59 -07:00

useSettingSearch.spec.ts

feat: add eslint-plugin-playwright via oxlint JS plugins (#11136)

2026-04-11 01:25:14 +00:00

versionMismatchWarnings.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

viewport.spec.ts

refactor: internalize nextFrame() into fixture/helper methods (#11166)

2026-04-15 15:25:47 +00:00

widget.spec.ts

feat: add eslint-plugin-playwright via oxlint JS plugins (#11136)

2026-04-11 01:25:14 +00:00

widgetCopyButton.spec.ts

test: remove redundant setup/settings now handled by @vue-nodes fixture (#11195)

2026-04-13 22:15:45 +00:00

workflowPersistence.spec.ts

test: Add waitForWorkflowIdle & remove redundant nextFrame (#11264)

2026-04-15 16:52:41 +00:00

workflowTabThumbnail.spec.ts

test: Simplify vue node/menu test setup (#11184)

2026-04-13 20:43:25 +00:00

zoomControls.spec.ts

feat: add eslint-plugin-playwright via oxlint JS plugins (#11136)

2026-04-11 01:25:14 +00:00