From a24db8e52a2ae57166dd19b7560d13dbe6b68eef Mon Sep 17 00:00:00 2001 From: KeyArgo Date: Sat, 19 Sep 2026 20:09:15 -0600 Subject: [PATCH] test(cli): also assert no unrecognized-key warning on stderr The 'not a recognized config key' notice for set_config_value is emitted on stdout today, but asserting stderr too guards against a future stream move and makes the regression test robust to either channel. --- tests/hermes_cli/test_set_config_value.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/hermes_cli/test_set_config_value.py b/tests/hermes_cli/test_set_config_value.py index bbdd868c96..ced185a078 100644 --- a/tests/hermes_cli/test_set_config_value.py +++ b/tests/hermes_cli/test_set_config_value.py @@ -124,7 +124,9 @@ class TestConfigYamlRouting: registered config key (not flagged as unrecognized) and coerce to a real list.""" set_config_value("tools.tool_search.defer", '["todo_list", "skill_manage"]') - assert "not a recognized config key" not in capsys.readouterr().out + captured = capsys.readouterr() + assert "not a recognized config key" not in captured.out + assert "not a recognized config key" not in captured.err config = yaml.safe_load(_read_config(_isolated_hermes_home)) assert config["tools"]["tool_search"]["defer"] == ["todo_list", "skill_manage"]