From 62ee34570e6e5a8b9f016754632a6b0852819bbf Mon Sep 17 00:00:00 2001 From: teknium1 <127238744+teknium1@users.noreply.github.com> Date: Fri, 24 Jul 2026 14:16:37 -0700 Subject: [PATCH] test: accept kwargs in managed_uv fixture fakes The runtime-repair change passes repair_observer= to update_managed_uv/ ensure_uv; the autouse fixture fakes had zero-arg signatures and raised TypeError through the mock. Sibling test file to the PR's own suite. --- tests/hermes_cli/test_update_autostash.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/hermes_cli/test_update_autostash.py b/tests/hermes_cli/test_update_autostash.py index c112b83d373..d89d830b140 100644 --- a/tests/hermes_cli/test_update_autostash.py +++ b/tests/hermes_cli/test_update_autostash.py @@ -25,13 +25,13 @@ def _patch_managed_uv(request): # resolve_uv delegates to shutil.which("uv") so that test patches # on shutil.which flow through naturally. - def _fake_resolve_uv(): + def _fake_resolve_uv(**kwargs): return shutil.which("uv") - def _fake_ensure_uv(): + def _fake_ensure_uv(**kwargs): return shutil.which("uv") - def _fake_update_managed_uv(): + def _fake_update_managed_uv(**kwargs): return None # never actually self-update in tests with patch("hermes_cli.managed_uv.resolve_uv", side_effect=_fake_resolve_uv), \