bymyself
c5138692bf
fix: import comfyExpect from ComfyPage in all vueNodes tests
2025-11-11 10:56:06 -08:00
bymyself
24be8d123b
fix: resolve circular dependency by removing comfyPageFixture re-export
...
- Remove comfyPageFixture re-export from ComfyPage.ts
- Update all test imports to use comfyPageFixture from its own file
- Update README.md example to show correct import path
- Fixes: comfyPageFixture → LocalhostComfyPage → ComfyPage circular import
2025-11-06 22:55:25 -07:00
Christian Byrne
bb83b0107c
fix Vue node border styles in different states (executing, error, selected) ( #6018 )
...
- Use exact tokens from Figma
- Fix issue in which node is stuck in `executing` state after it errors
┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-6018-fix-Vue-node-border-styles-in-different-states-executing-error-selected-2896d73d365081f39000fc3e42811f0d )
by [Unito](https://www.unito.io )
2025-10-11 12:20:06 -07:00
Christian Byrne
a0c06bd723
[test] add browser test for missing vue nodes error state ( #5768 )
...
## Summary
Added browser test to verify Vue nodes display error state when workflow
contains missing/unknown nodes, complementing
- https://github.com/Comfy-Org/ComfyUI_frontend/pull/5758
## Changes
- **What**: Added [Playwright
test](https://playwright.dev/docs/writing-tests ) for Vue nodes error
state handling with missing nodes
- **Test Coverage**: Validates `border-error` class application on nodes
with `UNKNOWN NODE` text
## Review Focus
Test reliability when loading workflows with missing nodes and dialog
interaction timing.
┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-5768-test-add-browser-test-for-missing-vue-nodes-error-state-2796d73d365081aea187cdbc7920a643 )
by [Unito](https://www.unito.io )
2025-09-25 11:16:19 -07:00
