vrr/ouroboros
2
0
Fork 0
mirror of https://github.com/razzant/ouroboros.git synced 2026-08-06 00:59:46 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 8

fix(tests): patch the moved LLMClient at plan_review_runtime

Browse source 
PR #96 extracted the plan-review slot runner (and its LLMClient
construction site) into ouroboros/tools/plan_review_runtime.py, but
test_plan_review_slots_use_no_proxy still patched the re-exporting
plan_review module, failing quick-test on every push since the merge.
Patch the owning module; the call through the re-export is unchanged.

Co-authored-by: Ouroboros <311266734+ouroboros-agent@users.noreply.github.com>
This commit is contained in:
Anton Razzhigaev 2026-08-04 15:17:06 +03:00
parent dad44e3957
commit be865eaa9d
1 changed files with 4 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

6
tests/test_llm_no_proxy.py
View file

@ -357,7 +357,7 @@ def test_chat_remote_no_proxy_retries_openrouter_parameter_rejection():
def test_plan_review_slots_use_no_proxy(tmp_path):
"""plan_review's shared ReviewCoordinator path must pass no_proxy=True."""
from ouroboros.tools import plan_review
from ouroboros.tools import plan_review, plan_review_runtime
captured_kwargs = []
@ -378,7 +378,9 @@ def test_plan_review_slots_use_no_proxy(tmp_path):
"event_queue": None,
})()
with patch.object(plan_review, "LLMClient", return_value=FakeLLMClient()):
# The slot runner (and its LLMClient construction site) moved to
# plan_review_runtime; patch the owning module, not the re-exporting one.
with patch.object(plan_review_runtime, "LLMClient", return_value=FakeLLMClient()):
result = _asyncio.run(
plan_review._run_plan_review_slots(
fake_ctx,