From 78d338b9ee917b73468c38ba4633ed53de4942e6 Mon Sep 17 00:00:00 2001 From: kshitijk4poor <82637225+kshitijk4poor@users.noreply.github.com> Date: Tue, 15 Sep 2026 20:37:34 +0530 Subject: [PATCH] test(cli): drop the tautological mcp_servers assertion from the seed test `_config_mcp_servers` is `self.config.get("mcp_servers") or {}`; with the test's own `{"mcp_servers": {}}` it is `{}` whether or not the import fix is present, so the assertion discriminated nothing. One invariant per fix. --- tests/hermes_cli/test_cli_mcp_config_watch.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/hermes_cli/test_cli_mcp_config_watch.py b/tests/hermes_cli/test_cli_mcp_config_watch.py index 85cd3d7028..2858733879 100644 --- a/tests/hermes_cli/test_cli_mcp_config_watch.py +++ b/tests/hermes_cli/test_cli_mcp_config_watch.py @@ -205,6 +205,7 @@ def test_tui_init_run_state_seeds_config_sig_when_config_exists(monkeypatch): from hermes_cli.config import get_config_path import cli as cli_mod + # Bare object: skip the tool-callback / security wiring at the end of the init. monkeypatch.setenv("HERMES_DEFER_AGENT_STARTUP", "1") cfg_file = get_config_path() cfg_file.parent.mkdir(parents=True, exist_ok=True) @@ -215,7 +216,6 @@ def test_tui_init_run_state_seeds_config_sig_when_config_exists(monkeypatch): obj._tui_init_run_state() assert obj._config_sig == file_signature(cfg_file.stat()) - assert obj._config_mcp_servers == {} def test_pinned_mtime_same_size_replacement_triggers_reload(tmp_path):