Skip to content

Commit 06f4cef

Browse files
committed
fix(e2e): adapt tests for Manager v4 output format
- test_node: check display name (ComfyUI-AnimateDiff-Evolved) instead of registry ID in show all output, since cm_cli uses display names - test_manager_installed: remove direct python import check (system Python doesn't have cm_cli — it's in workspace venv); node show all success already proves Manager is functional
1 parent e023abe commit 06f4cef

1 file changed

Lines changed: 3 additions & 11 deletions

File tree

tests/e2e/test_e2e.py

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,8 @@ def test_node(comfy_cli, workspace):
118118
"""
119119
)
120120
assert proc.returncode == 0
121-
assert node in proc.stdout
121+
# Manager v4 show all uses display names, not registry IDs
122+
assert "ComfyUI-AnimateDiff-Evolved" in proc.stdout
122123

123124
proc = exec(
124125
f"""
@@ -161,16 +162,7 @@ def test_manager_installed(comfy_cli, workspace):
161162
{comfy_cli} node show all
162163
"""
163164
)
164-
assert proc.returncode == 0
165-
166-
# Check cm_cli is importable (Manager v4 installed as pip package)
167-
proc = exec(
168-
"""
169-
python -c "import cm_cli; print('cm_cli OK')"
170-
"""
171-
)
172-
assert proc.returncode == 0, f"cm_cli import failed: {proc.stderr}"
173-
assert "cm_cli OK" in proc.stdout
165+
assert proc.returncode == 0, f"node show all failed: {proc.stderr}"
174166

175167

176168
@e2e_test

0 commit comments

Comments
 (0)