Skip to content

Commit 34d026a

Browse files
[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 <[email protected]>
1 parent d4bdde6 commit 34d026a

File tree

1 file changed

+12
-0
lines changed

1 file changed

+12
-0
lines changed

browser_tests/tests/vueNodes/nodeStates/colors.spec.ts

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,18 @@ test.describe('Vue Node Custom Colors', () => {
2828
await expect(comfyPage.canvas).toHaveScreenshot(
2929
'vue-node-custom-color-blue.png'
3030
)
31+
32+
// Verify actual DOM properties for header and body colors
33+
const nodeHeader = loadCheckpointNode.locator(
34+
'[data-testid^="node-header-"]'
35+
)
36+
const nodeBody = loadCheckpointNode
37+
38+
// Header should have blue header color (#223)
39+
await expect(nodeHeader).toHaveCSS('background-color', 'rgb(34, 34, 51)')
40+
41+
// Node body should have blue body color (#335)
42+
await expect(nodeBody).toHaveCSS('background-color', 'rgb(51, 51, 85)')
3143
})
3244

3345
// TODO: implement loading node colors from workflow in Vue system

0 commit comments

Comments
 (0)