From 022e069a17922bbef12ebb0710389e7be1641877 Mon Sep 17 00:00:00 2001 From: bymyself Date: Tue, 23 Sep 2025 23:30:50 -0700 Subject: [PATCH] [test] add DOM property assertions for Vue node colors - addresses review feedback Adds CSS background-color property validation in addition to screenshot comparison to verify header and body colors are correctly applied to DOM elements. Co-authored-by: Myestery --- .../tests/vueNodes/nodeStates/colors.spec.ts | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/browser_tests/tests/vueNodes/nodeStates/colors.spec.ts b/browser_tests/tests/vueNodes/nodeStates/colors.spec.ts index cc42e8b51e..c902bc2fb5 100644 --- a/browser_tests/tests/vueNodes/nodeStates/colors.spec.ts +++ b/browser_tests/tests/vueNodes/nodeStates/colors.spec.ts @@ -28,6 +28,18 @@ test.describe('Vue Node Custom Colors', () => { await expect(comfyPage.canvas).toHaveScreenshot( 'vue-node-custom-color-blue.png' ) + + // Verify actual DOM properties for header and body colors + const nodeHeader = loadCheckpointNode.locator( + '[data-testid^="node-header-"]' + ) + const nodeBody = loadCheckpointNode + + // Header should have blue header color (#223) + await expect(nodeHeader).toHaveCSS('background-color', 'rgb(34, 34, 51)') + + // Node body should have blue body color (#335) + await expect(nodeBody).toHaveCSS('background-color', 'rgb(51, 51, 85)') }) // TODO: implement loading node colors from workflow in Vue system