From d5a9ff2aac6f483b68acd17eb24af901b8be4f06 Mon Sep 17 00:00:00 2001 From: Vineeth Sai Date: Thu, 5 Nov 2026 16:32:02 -0800 Subject: [PATCH] test(agentic loop): drop the redundant callbacks reset The unit conftest already clears litellm.callbacks before every test, so the monkeypatch did nothing. Also drop the docstring, which only restated the test name. --- .../test_chat_completion_agentic_loop.py | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/tests/unit/litellm_core_utils/test_chat_completion_agentic_loop.py b/tests/unit/litellm_core_utils/test_chat_completion_agentic_loop.py index 7f09f93c977..d86128eae68 100644 --- a/tests/unit/litellm_core_utils/test_chat_completion_agentic_loop.py +++ b/tests/unit/litellm_core_utils/test_chat_completion_agentic_loop.py @@ -1044,12 +1044,7 @@ async def test_converted_stream_replays_the_final_answer_with_the_requested_usag @pytest.mark.parametrize( "prefix", (WEBSEARCH_INTERCEPTION_PREFIX, CODE_INTERPRETER_INTERCEPTION_PREFIX, HEADROOM_INTERCEPTION_PREFIX) ) -async def test_converted_stream_replays_the_original_reply_when_no_tool_is_called( - monkeypatch: pytest.MonkeyPatch, prefix: str -): - """A streamed request that was converted to a non-streaming call still gets a - stream back when the model answers without a tool call and no callback gates.""" - monkeypatch.setattr(litellm, "callbacks", []) +async def test_converted_stream_replays_the_original_reply_when_no_tool_is_called(prefix: str): logging_obj: Final = Logging( model="gpt-4o", messages=[{"role": "user", "content": "hello"}],