From 107843877f43d52fe2f308582eaee6e7bd872b55 Mon Sep 17 00:00:00 2001 From: Brooklyn Nicholson Date: Fri, 24 Jul 2026 10:20:36 -0500 Subject: [PATCH] test(sessions): use mock.patch for the config gate, matching file idiom --- tests/hermes_cli/test_web_server.py | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/tests/hermes_cli/test_web_server.py b/tests/hermes_cli/test_web_server.py index 9c6299a6541..82b48c5ac71 100644 --- a/tests/hermes_cli/test_web_server.py +++ b/tests/hermes_cli/test_web_server.py @@ -1945,17 +1945,11 @@ class TestWebServerEndpoints: ws._last_auto_archive_check.clear() # The helper imports load_config lazily from hermes_cli.config; patch there. - import hermes_cli.config as _config_mod - - def _cfg(): - return {"sessions": {"auto_archive": True, "auto_archive_days": 3, "min_interval_hours": 0}} - - prev = _config_mod.load_config - _config_mod.load_config = _cfg # type: ignore[assignment] + cfg = {"sessions": {"auto_archive": True, "auto_archive_days": 3, "min_interval_hours": 0}} try: - listed = self.client.get("/api/sessions").json()["sessions"] + with patch("hermes_cli.config.load_config", return_value=cfg): + listed = self.client.get("/api/sessions").json()["sessions"] finally: - _config_mod.load_config = prev # type: ignore[assignment] ws._last_auto_archive_check.clear() assert all(s["id"] != "stale-serve" for s in listed)