From 4a91330e301b8239ff3d947c4ca9f72d78c297f8 Mon Sep 17 00:00:00 2001 From: Christian Byrne Date: Fri, 12 Dec 2025 15:33:10 -0800 Subject: [PATCH] fix: flaky legacy context menu e2e test (#7373) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Summary Fixes this test which became flaky after removing the timeout. Now simply wait for the title text in the context menu, which has built-in timeout, then take snapshot. image ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-7373-WIP-fix-legacy-context-menu-e2e-test-2c66d73d3650819eb9baceb5bdebfbe8) by [Unito](https://www.unito.io) --- browser_tests/tests/nodeSearchBox.spec.ts | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/browser_tests/tests/nodeSearchBox.spec.ts b/browser_tests/tests/nodeSearchBox.spec.ts index ce8875def..2eea22329 100644 --- a/browser_tests/tests/nodeSearchBox.spec.ts +++ b/browser_tests/tests/nodeSearchBox.spec.ts @@ -260,6 +260,12 @@ test.describe('Release context menu', () => { test('Can trigger on link release', async ({ comfyPage }) => { await comfyPage.disconnectEdge() + const contextMenu = comfyPage.page.locator('.litecontextmenu') + // Wait for context menu with correct title (slot name | slot type) + // The title shows the output slot name and type from the disconnected link + await expect(contextMenu.locator('.litemenu-title')).toContainText( + 'CLIP | CLIP' + ) await comfyPage.page.mouse.move(10, 10) await comfyPage.nextFrame() await expect(comfyPage.canvas).toHaveScreenshot(