DasPoschi/hermes-webui
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test(#4022): update sprint42 runtime-route assertion for target_model threading

Browse Source 
#4022 now calls resolve_runtime_provider(requested=..., target_model=...) so
per-model base_url normalization applies; the sprint42 test pinned the old
single-arg signature. Updated the assertion to expect target_model (pre-existing
test, not a behavior regression — confirmed it passes on master only because the
call signature changed here).
This commit is contained in:
nesquena-hermes
2026-06-12 07:57:26 +00:00
parent 005b2ca460
commit c35c1c6e70
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
tests/test_sprint42.py
View File

@@ -256,7 +256,11 @@ class TestRuntimeRouteInjection(unittest.TestCase):
stream_id=fake_stream_id,
)
resolve_runtime_provider.assert_called_once_with(requested="openai-codex")
# #4022: the resolver is now called with the target model too so per-model
# base_url normalization (e.g. OpenCode-Go /v1 stripping) is applied.
resolve_runtime_provider.assert_called_once_with(
requested="openai-codex", target_model="gpt-5.4"
)
init_kwargs = captured["init_kwargs"]
self.assertEqual(init_kwargs["api_mode"], "codex_responses")
self.assertEqual(init_kwargs["acp_command"], "codex")