Christian Byrne aee2e6e6dd test: add e2e tests for nested SubgraphNode input target resolution (#11405) ...

*PR Created by the Glary-Bot Agent*

---

## Summary

- Adds four Playwright tests targeting `resolveSubgraphInputTarget`
lines 20-31 — the `inputNode.isSubgraphNode()` branch where the target
widget is a PromotedWidgetView
- These lines had 0% e2e coverage because no existing test loaded a
multi-level nested subgraph with VueNodes enabled
- Tests use the existing `subgraph-nested-promotion.json` workflow (node
5 → Sub 0 → node 6 → Sub 1), which has outer SubgraphNode inputs
connecting to an inner SubgraphNode

## Test cases

| Test | Coverage target | Mechanism |
|---|---|---|
| Nested SubgraphNode promoted widgets render without resolution
failures | Lines 20-31 (via VueNodes rendering) | Console warning
collection + widget count assertion |
| Subgraph input resolves through inner SubgraphNode with
PromotedWidgetView | Lines 20-31 (graph structure verification) |
`page.evaluate` walks link chain, asserts `isSubgraphNode() === true`
and `isPromotedWidgetView === true` |
| Promoted widgets from inner SubgraphNode carry correct source identity
| Lines 24-31 (source identity) | Asserts widgets with `sourceNodeId ===
'6'` have correct `sourceWidgetName` |
| Serialize and reload preserves nested promoted widget resolution |
Lines 20-31 (persistence) | `serializeAndReload()` + polled widget count
comparison |

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-11405-test-add-e2e-tests-for-nested-SubgraphNode-input-target-resolution-3476d73d365081ab932edc8a01c55c40)
by [Unito](https://www.unito.io)

---------

Co-authored-by: Glary-Bot <glary-bot@users.noreply.github.com>
Co-authored-by: Alexander Brown <drjkl@comfy.org>
Co-authored-by: Connor Byrne <c.byrne@comfy.org>

2026-05-04 13:53:16 -07:00

..

canvasSettings.spec.ts-snapshots

test: add tests for canvas related settings (#11604)

2026-04-28 08:52:49 +00:00

collapsedNodeLinks.spec.ts-snapshots

refactor: inline node footer layout to fix selection bounding box (#10741)

2026-04-19 04:58:34 +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

test: add E2E tests for publish flow wizard (#10770)

2026-04-28 00:13:43 +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

fix(load3d): suppress error toast on 404 when loading output model file (#11807)

2026-05-01 18:49:31 -04: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

feat: polish settings dialog layout and keybinding display (#11241)

2026-04-17 20:22:39 -07: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

test: add Vue node error/validation ring e2e coverage (#11727)

2026-05-01 09:35:29 +00:00

queue

test: add test for MaxHistoryItems setting (#11750)

2026-04-29 19:10:31 +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 dropdown chevron color (#11335)

2026-04-17 20:37:22 +00: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

test: add Vue node error/validation ring e2e coverage (#11727)

2026-05-01 09:35:29 +00:00

subgraph

test: add e2e tests for nested SubgraphNode input target resolution (#11405)

2026-05-04 13:53:16 -07: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

test: add Vue node error/validation ring e2e coverage (#11727)

2026-05-01 09:35:29 +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

refactor: harden ChangeTracker lifecycle with self-defending API (#10816)

2026-04-16 12:54:12 +00:00

actionBarButtons.spec.ts

test: add E2E tests for ActionBarButtons toolbar component (#11561)

2026-04-28 19:13:12 -04:00

appModeArrange.spec.ts

refactor: consolidate browser_tests/helpers/ into fixtures/ (#11411)

2026-04-28 02:18:31 +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: consolidate browser_tests/helpers/ into fixtures/ (#11411)

2026-04-28 02:18:31 +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

refactor: consolidate browser_tests/helpers/ into fixtures/ (#11411)

2026-04-28 02:18:31 +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: extract asset api browser fixture (#11279)

2026-04-20 18:37:45 +00: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

refactor: consolidate browser_tests/helpers/ into fixtures/ (#11411)

2026-04-28 02:18:31 +00:00

bottomPanelShortcuts.spec.ts

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

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

browserTabTitle.spec.ts

refactor: harden ChangeTracker lifecycle with self-defending API (#10816)

2026-04-16 12:54:12 +00:00

builderReorder.spec.ts

refactor: consolidate browser_tests/helpers/ into fixtures/ (#11411)

2026-04-28 02:18:31 +00:00

builderSaveFlow.spec.ts

refactor: consolidate browser_tests/helpers/ into fixtures/ (#11411)

2026-04-28 02:18:31 +00:00

canvasLayoutSettings.spec.ts

test: Canvas grid, ctx menu scaling and group padding settings (#11721)

2026-04-28 17:21:55 +00:00

canvasModeSelector.spec.ts

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

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

canvasSettings.spec.ts

test: extract title editor test component (#11605)

2026-04-28 09:46:25 -04:00

changeTracker.spec.ts

refactor: harden ChangeTracker lifecycle with self-defending API (#10816)

2026-04-16 12:54:12 +00:00

changeTrackerLoadGuard.spec.ts

refactor: harden ChangeTracker lifecycle with self-defending API (#10816)

2026-04-16 12:54:12 +00:00

cloud-asset-default.spec.ts

Fix naming strategy for multi-job asset exports (#11610)

2026-04-27 20:44:32 +00:00

cloud.spec.ts

refactor: fix lint errors in tests (#11182)

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

collapsedNodeLinks.spec.ts

fix: re-sync collapsed node slot positions after subgraph fitView (#11240)

2026-04-16 12:38:01 +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

refactor: consolidate browser_tests/helpers/ into fixtures/ (#11411)

2026-04-28 02:18:31 +00:00

errorOverlay.spec.ts

test: add Vue node error/validation ring e2e coverage (#11727)

2026-05-01 09:35:29 +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

fileInputReselection.spec.ts

fix: reset file input value after selection to allow same-file reupload (#11417)

2026-04-20 17:35:17 -07: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: add selection paste, rename, and batch rename browser tests (#11367)

2026-05-04 20:30:42 +00:00

groupNode.spec.ts

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

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

groupSelectChildren.spec.ts

test: add selection paste, rename, and batch rename browser tests (#11367)

2026-05-04 20:30:42 +00:00

helpCenter.spec.ts

test: Add tests for help center (#11475)

2026-04-20 22:43:28 +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

keyboardShortcutActions.spec.ts

test: combine duplicated undo/redo and settings dialog E2E tests with test.step (#11835)

2026-05-04 12:54:52 -07:00

linearMode.spec.ts

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

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

linkNodeInteractionSettings.spec.ts

test: add tests for link related settings (#11612)

2026-04-28 22:02:42 +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

loginButton.spec.ts

test: add E2E tests for LoginButton toolbar component (#11562)

2026-04-28 08:33:42 -04: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: extend minimap e2e for level 1& level4 (#11192)

2026-04-23 22:17:16 -04: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

fix: ensure escape key/graph navigation cancels ghost node placement (#11779)

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

nodeHelp.spec.ts

refactor: consolidate browser_tests/helpers/ into fixtures/ (#11411)

2026-04-28 02:18:31 +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: Node search - Improve category tree on mobile with collapse (#11687)

2026-04-29 20:52:58 +00:00

nodeSearchBoxV2Extended.spec.ts

feat: Search - add ghost node following setting and increase opacity (#11365)

2026-04-28 22:02:33 +00:00

nodeTemplates.spec.ts

test: e2e coverage for node templates (#11564)

2026-04-23 19:33:18 +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

refactor: consolidate browser_tests/helpers/ into fixtures/ (#11411)

2026-04-28 02:18:31 +00:00

pasteImageContextMenu.spec.ts

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

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

performance.spec.ts

refactor: consolidate browser_tests/helpers/ into fixtures/ (#11411)

2026-04-28 02:18:31 +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

queueNotificationBanners.spec.ts

test: add queue notification banners lifecycle browser tests (#11366)

2026-05-02 05:40:45 +00: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: extract title editor test component (#11605)

2026-04-28 09:46:25 -04:00

saveImageAndWebp.spec.ts

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

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

selectionBoundingBox.spec.ts

refactor: consolidate browser_tests/helpers/ into fixtures/ (#11411)

2026-04-28 02:18:31 +00:00

selectionRectangle.spec.ts

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

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

selectionToolbox.spec.ts

feat: add plum/ink color primitives and standardize design tokens (#11139)

2026-04-30 16:15:53 -07:00

selectionToolboxActions.spec.ts

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

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

selectionToolboxMoreActions.spec.ts

test: add selection paste, rename, and batch rename browser tests (#11367)

2026-05-04 20:30:42 +00:00

selectionToolboxRename.spec.ts

test: add selection paste, rename, and batch rename browser tests (#11367)

2026-05-04 20:30:42 +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

templateFilteringCount.spec.ts

fix: move template distribution filter from v-show to data pipeline (#11418)

2026-04-23 02:42:04 +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

topbarMenuCommands.spec.ts

test: combine duplicated undo/redo and settings dialog E2E tests with test.step (#11835)

2026-05-04 12:54:52 -07: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

webcamCapture.spec.ts

fix: fix webcam node not showing preview in nodes 2.0 (#11549)

2026-04-22 11:29:08 -07: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

refactor: harden ChangeTracker lifecycle with self-defending API (#10816)

2026-04-16 12:54:12 +00:00

workflowSettings.spec.ts

test: add test coverage for workflow save settings (#11763)

2026-05-01 09:09:31 +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