From a92412ede151374212782b0bf47aa8df82e1c19d Mon Sep 17 00:00:00 2001 From: Teknium <127238744+teknium1@users.noreply.github.com> Date: Wed, 19 Aug 2026 23:02:30 -0700 Subject: [PATCH] test: mock load_config_readonly in memory status gate tests check_memory_requirements() reads the readonly config loader; the salvaged tests only patched load_config, so the gate saw the real config. --- tests/hermes_cli/test_memory_setup.py | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/tests/hermes_cli/test_memory_setup.py b/tests/hermes_cli/test_memory_setup.py index 674a9b51d1..84bb7ec02d 100644 --- a/tests/hermes_cli/test_memory_setup.py +++ b/tests/hermes_cli/test_memory_setup.py @@ -100,9 +100,11 @@ def test_install_dependencies_force_reinstalls_versioned_specs(tmp_path, monkeyp def test_cmd_status_memory_tool_gate_disabled(capsys, monkeypatch): """When both memory stores are disabled, Memory status reports memory tool as disabled.""" + _cfg = {"memory": {"memory_enabled": False, "user_profile_enabled": False}} + monkeypatch.setattr("hermes_cli.config.load_config", lambda: _cfg) + # check_memory_requirements() reads the readonly loader, not load_config. monkeypatch.setattr( - "hermes_cli.config.load_config", - lambda: {"memory": {"memory_enabled": False, "user_profile_enabled": False}}, + "hermes_cli.config.load_config_readonly", lambda: _cfg, raising=False ) monkeypatch.setattr(memory_setup, "_get_available_providers", lambda: []) @@ -116,9 +118,10 @@ def test_cmd_status_memory_tool_gate_disabled(capsys, monkeypatch): def test_cmd_status_memory_tool_gate_enabled(capsys, monkeypatch): """When at least one memory store is enabled, Memory status reports memory tool as enabled.""" + _cfg = {"memory": {"memory_enabled": True, "user_profile_enabled": False}} + monkeypatch.setattr("hermes_cli.config.load_config", lambda: _cfg) monkeypatch.setattr( - "hermes_cli.config.load_config", - lambda: {"memory": {"memory_enabled": True, "user_profile_enabled": False}}, + "hermes_cli.config.load_config_readonly", lambda: _cfg, raising=False ) monkeypatch.setattr(memory_setup, "_get_available_providers", lambda: [])