hermes-admin/hermes-agent
1
0
Fork 0
Code Issues Pull Requests Actions 4 Packages Projects Releases Wiki Activity

test(cli): also assert no unrecognized-key warning on stderr

Browse Source 
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.
This commit is contained in:
KeyArgo
2026-09-19 20:09:15 -06:00
committed by Teknium
parent d6beebcc57
commit a24db8e52a
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
tests/hermes_cli/test_set_config_value.py
View File

@@ -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"]