From 33b6c4c6509387d7585570f5b39fecb1ebbc4b5c Mon Sep 17 00:00:00 2001
From: teknium1 <127238744+teknium1@users.noreply.github.com>
Date: Tue, 15 Sep 2026 12:12:12 -0700
Subject: [PATCH] test(config): point dict-value and deep-creation tests at
 schema keys

`display.tool_progress_overrides` (deprecated) and the fabricated `agent.max_iterations`
were unknown sub-keys under known sections, the shape `hermes config set` now refuses
before writing. The tests exercise value-shape parsing and section creation, not those
keys, so they use `mcp_servers.demo.env` and `agent.max_turns` instead.
---
 tests/hermes_cli/test_config_dotted_key_names.py | 4 ++--
 tests/hermes_cli/test_config_set_list_values.py  | 8 ++++----
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/tests/hermes_cli/test_config_dotted_key_names.py b/tests/hermes_cli/test_config_dotted_key_names.py
index 92c5fb5138..968e92147b 100644
--- a/tests/hermes_cli/test_config_dotted_key_names.py
+++ b/tests/hermes_cli/test_config_dotted_key_names.py
@@ -344,9 +344,9 @@ class TestBackwardCompatibility:
         assert cfg["custom_providers"][0]["name"] == "p2"
 
     def test_deep_creation_without_siblings_unchanged(self, _isolated_hermes_home):
-        set_config_value("agent.max_iterations", "50")
+        set_config_value("agent.max_turns", "50")
         saved = _read_config(_isolated_hermes_home)
-        assert saved["agent"]["max_iterations"] == 50
+        assert saved["agent"]["max_turns"] == 50
 
     def test_greedy_never_beats_exact_nested_structure(self):
         """A literal dotted key never shadows the plain-split path when the
diff --git a/tests/hermes_cli/test_config_set_list_values.py b/tests/hermes_cli/test_config_set_list_values.py
index 80c15a6afd..022f13ae08 100644
--- a/tests/hermes_cli/test_config_set_list_values.py
+++ b/tests/hermes_cli/test_config_set_list_values.py
@@ -36,9 +36,9 @@ def test_list_literal_is_parsed_to_list(user_home):
 def test_mapping_literal_is_parsed_to_dict(user_home):
     from hermes_cli.config import set_config_value, read_raw_config
 
-    set_config_value("display.tool_progress_overrides", '{"terminal": "off"}')
+    set_config_value("mcp_servers.demo.env", '{"terminal": "off"}')
     raw = read_raw_config()
-    assert raw["display"]["tool_progress_overrides"] == {"terminal": "off"}
+    assert raw["mcp_servers"]["demo"]["env"] == {"terminal": "off"}
 
 
 def test_yaml_flow_list_is_parsed(user_home):
@@ -97,11 +97,11 @@ def test_multiline_yaml_mapping_is_parsed(user_home):
     from hermes_cli.config import set_config_value, read_raw_config
 
     set_config_value(
-        "display.tool_progress_overrides",
+        "mcp_servers.demo.env",
         "terminal: off\nbrowser: on",
     )
     raw = read_raw_config()
-    assert raw["display"]["tool_progress_overrides"] == {
+    assert raw["mcp_servers"]["demo"]["env"] == {
         "terminal": False,
         "browser": True,
     }