From 7ced2ee394fbd704ae66ba5219e5664a3079376e Mon Sep 17 00:00:00 2001 From: teknium1 <127238744+teknium1@users.noreply.github.com> Date: Sat, 18 Jul 2026 15:26:50 -0700 Subject: [PATCH] test(web): unpin HERMES_HONCHO_HOST in profile-param memory config test The suite-wide conftest now pins HERMES_HONCHO_HOST=hermes (597615ade, after this branch was written), which preempts profile-driven host resolution and made the salvaged test write to the 'hermes' host key instead of 'hermes_worker'. Drop the override inside the test like the other custom host-resolution tests do. --- tests/hermes_cli/test_web_server.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/hermes_cli/test_web_server.py b/tests/hermes_cli/test_web_server.py index 12e1530aab92..fd9a414d948d 100644 --- a/tests/hermes_cli/test_web_server.py +++ b/tests/hermes_cli/test_web_server.py @@ -1235,6 +1235,9 @@ class TestWebServerEndpoints: # A ?profile= save must land in that profile's config, not the serving # process's — same contract as the skills/toolsets endpoints. monkeypatch.setenv("HOME", str(tmp_path)) + # The suite pins HERMES_HONCHO_HOST=hermes; this test exercises + # profile-driven host resolution, so drop the override explicitly. + monkeypatch.delenv("HERMES_HONCHO_HOST", raising=False) from hermes_constants import get_hermes_home from hermes_cli.profiles import get_profile_dir