test: update test_cli assertions for mempalace-mcp entry point
Three assertions in test_mcp_command_* were still checking for the old `python -m mempalace.mcp_server` output string. Update to match the new `mempalace-mcp` command printed by cmd_mcp(). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
committed by
Igor Lins e Silva
parent
982d421510
commit
9e53228ea3
+3
-3
@@ -334,9 +334,9 @@ def test_mcp_command_prints_setup_guidance(monkeypatch, capsys):
|
|||||||
|
|
||||||
captured = capsys.readouterr()
|
captured = capsys.readouterr()
|
||||||
assert "MemPalace MCP quick setup:" in captured.out
|
assert "MemPalace MCP quick setup:" in captured.out
|
||||||
assert "claude mcp add mempalace -- python -m mempalace.mcp_server" in captured.out
|
assert "claude mcp add mempalace -- mempalace-mcp" in captured.out
|
||||||
assert "\nOptional custom palace:\n" in captured.out
|
assert "\nOptional custom palace:\n" in captured.out
|
||||||
assert "python -m mempalace.mcp_server --palace /path/to/palace" in captured.out
|
assert "mempalace-mcp --palace /path/to/palace" in captured.out
|
||||||
assert "[--palace /path/to/palace]" not in captured.out
|
assert "[--palace /path/to/palace]" not in captured.out
|
||||||
assert captured.err == ""
|
assert captured.err == ""
|
||||||
|
|
||||||
@@ -349,7 +349,7 @@ def test_mcp_command_uses_custom_palace_path_when_provided(monkeypatch, capsys):
|
|||||||
captured = capsys.readouterr()
|
captured = capsys.readouterr()
|
||||||
expanded = str(Path("~/tmp/my palace").expanduser())
|
expanded = str(Path("~/tmp/my palace").expanduser())
|
||||||
|
|
||||||
assert "python -m mempalace.mcp_server --palace" in captured.out
|
assert "mempalace-mcp --palace" in captured.out
|
||||||
assert expanded in captured.out
|
assert expanded in captured.out
|
||||||
assert "Optional custom palace:" not in captured.out
|
assert "Optional custom palace:" not in captured.out
|
||||||
assert "[--palace /path/to/palace]" not in captured.out
|
assert "[--palace /path/to/palace]" not in captured.out
|
||||||
|
|||||||
Reference in New Issue
Block a user