Skip to content

Commit 776ce3a

Browse files
committed
fix: e2e test_node case-insensitive show-all check, retry reinstall
1 parent c91741b commit 776ce3a

1 file changed

Lines changed: 12 additions & 7 deletions

File tree

tests/e2e/test_e2e.py

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -109,20 +109,25 @@ def test_node(comfy_cli, workspace):
109109
break
110110
assert proc.returncode == 0, f"node install failed after 3 attempts:\n{proc.stderr}"
111111

112-
proc = exec(
113-
f"""
114-
{comfy_cli} node reinstall {node}
115-
"""
116-
)
117-
assert proc.returncode == 0
112+
for attempt in range(3):
113+
proc = exec(
114+
f"""
115+
{comfy_cli} node reinstall {node}
116+
"""
117+
)
118+
if proc.returncode == 0:
119+
break
120+
assert proc.returncode == 0, f"node reinstall failed after 3 attempts:\n{proc.stderr}"
118121

119122
proc = exec(
120123
f"""
121124
{comfy_cli} node show all
122125
"""
123126
)
124127
assert proc.returncode == 0
125-
assert node in proc.stdout
128+
# cm-cli may display the repo name (ComfyUI-AnimateDiff-Evolved) rather
129+
# than the registry id (comfyui-animatediff-evolved), so compare lowercase.
130+
assert node.lower() in proc.stdout.lower()
126131

127132
proc = exec(
128133
f"""

0 commit comments

Comments
 (0)