From 8f2c8d2dc88e6d3ead6c9472deed120627865a7c Mon Sep 17 00:00:00 2001 From: Kenny Van de Maele Date: Sun, 7 Jun 2026 17:23:06 +0200 Subject: [PATCH] fix(test): tolerate owner kwarg in compaction summary resolve_endpoint mock (#3304) #2996 made context_compactor call resolve_endpoint('utility', owner=owner), but the mock added by #2174 stubbed it as lambda which: ..., which rejects the owner kwarg. Each PR passed alone; merged on dev the two compaction tests fail with TypeError and the pytest job goes red. Widen the mock to lambda *a, **k. Co-authored-by: Claude Opus 4.8 --- tests/test_compaction_summary_failure.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/test_compaction_summary_failure.py b/tests/test_compaction_summary_failure.py index f69f7e58d..2a3020c42 100644 --- a/tests/test_compaction_summary_failure.py +++ b/tests/test_compaction_summary_failure.py @@ -48,7 +48,7 @@ class TestCompactionSummaryFailure: cc.get_context_length = lambda url, model: context_length cc.estimate_tokens = lambda msgs: 10000 # well over the threshold cc.llm_call_async = _boom - cc.resolve_endpoint = lambda which: (None, None, None) + cc.resolve_endpoint = lambda *a, **k: (None, None, None) cc._update_session_history = lambda *a, **k: None try: return asyncio.run(