From 50d1518df63a331f6191b7cd03c7a9751a2946c9 Mon Sep 17 00:00:00 2001 From: Teknium Date: Tue, 7 Apr 2026 17:44:04 -0700 Subject: [PATCH] fix(tests): update tool_progress_callback test calls to new 4-arg signature MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Follow-up to sroecker's PR #5918 — test mocks were using the old 3-arg callback signature (name, preview, args) instead of the new (event_type, name, preview, args, **kwargs). --- tests/gateway/test_api_server.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/gateway/test_api_server.py b/tests/gateway/test_api_server.py index 5bde076a68..3b2160615d 100644 --- a/tests/gateway/test_api_server.py +++ b/tests/gateway/test_api_server.py @@ -439,7 +439,7 @@ class TestChatCompletionsEndpoint: tp_cb = kwargs.get("tool_progress_callback") # Simulate tool progress before streaming content if tp_cb: - tp_cb("terminal", "ls -la", {"command": "ls -la"}) + tp_cb("tool.started", "terminal", "ls -la", {"command": "ls -la"}) if cb: await asyncio.sleep(0.05) cb("Here are the files.") @@ -476,8 +476,8 @@ class TestChatCompletionsEndpoint: cb = kwargs.get("stream_delta_callback") tp_cb = kwargs.get("tool_progress_callback") if tp_cb: - tp_cb("_thinking", "some internal state", {}) - tp_cb("web_search", "Python docs", {"query": "Python docs"}) + tp_cb("tool.started", "_thinking", "some internal state", {}) + tp_cb("tool.started", "web_search", "Python docs", {"query": "Python docs"}) if cb: await asyncio.sleep(0.05) cb("Found it.")