From 01b4ad0dbbdffa3cf561fd8ba45bb3374188bd06 Mon Sep 17 00:00:00 2001 From: Christian Byrne Date: Sun, 12 Oct 2025 17:01:45 -0700 Subject: [PATCH] [test] add browser test for control+a selection of Vue nodes (#6031) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Summary Adds test case ensuring you can select all Vue nodes with `Ctrl`+`a`, mirroring coverage of similar behavior when using Litegraph nodes. ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-6031-test-add-browser-test-for-control-a-selection-of-Vue-nodes-28a6d73d365081079860c3a083a946ef) by [Unito](https://www.unito.io) --- .../tests/vueNodes/interactions/node/select.spec.ts | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/browser_tests/tests/vueNodes/interactions/node/select.spec.ts b/browser_tests/tests/vueNodes/interactions/node/select.spec.ts index 4402236d2..98b0a63f6 100644 --- a/browser_tests/tests/vueNodes/interactions/node/select.spec.ts +++ b/browser_tests/tests/vueNodes/interactions/node/select.spec.ts @@ -50,15 +50,23 @@ test.describe('Vue Node Selection', () => { }) } + test('should select all nodes with ctrl+a', async ({ comfyPage }) => { + const initialCount = await comfyPage.vueNodes.getNodeCount() + expect(initialCount).toBeGreaterThan(0) + + await comfyPage.canvas.press('Control+a') + + const selectedCount = await comfyPage.vueNodes.getSelectedNodeCount() + expect(selectedCount).toBe(initialCount) + }) + test('should select pinned node without dragging', async ({ comfyPage }) => { const PIN_HOTKEY = 'p' const PIN_INDICATOR = '[data-testid="node-pin-indicator"]' - // Select a node by clicking its title const checkpointNodeHeader = comfyPage.page.getByText('Load Checkpoint') await checkpointNodeHeader.click() - // Pin it using the hotkey (as a user would) await comfyPage.page.keyboard.press(PIN_HOTKEY) const checkpointNode = comfyPage.vueNodes.getNodeByTitle('Load Checkpoint')