From f20537708c424361a621b0fdc8d2cede8146f516 Mon Sep 17 00:00:00 2001
From: yryzhan <yryzhan@users.noreply.github.com>
Date: Wed, 20 May 2026 16:34:12 +0200
Subject: [PATCH] test(bedrock): remove dead code from
 test_tool_choice_stripped_when_no_tools

Remove unused patch.object wrapper (no-op pass-through) and unused
messages variable as flagged by Greptile review.
---
 .../chat/test_converse_transformation.py      | 32 +++++++------------
 1 file changed, 11 insertions(+), 21 deletions(-)

diff --git a/tests/test_litellm/llms/bedrock/chat/test_converse_transformation.py b/tests/test_litellm/llms/bedrock/chat/test_converse_transformation.py
index 25f3a9e0990..87a7194ddea 100644
--- a/tests/test_litellm/llms/bedrock/chat/test_converse_transformation.py
+++ b/tests/test_litellm/llms/bedrock/chat/test_converse_transformation.py
@@ -4586,34 +4586,24 @@ def test_tool_choice_stripped_when_no_tools():
     it must be popped before _transform_inference_params to avoid
     ValidationException from Bedrock.
     """
-    from unittest.mock import MagicMock, patch
-
     from litellm.llms.bedrock.chat.converse_transformation import (
         AmazonConverseConfig,
     )
 
     config = AmazonConverseConfig()
-    inference_params = {
+    optional_params = {
         "temperature": 0.7,
         "tool_choice": {"auto": {}},
     }
 
-    with patch.object(
-        config,
-        "_transform_request",
-        wraps=config._transform_request,
-    ):
-        messages = [{"role": "user", "content": [{"text": "Hello"}]}]
-        optional_params = dict(inference_params)
+    result = config.transform_request(
+        model="anthropic.claude-3-sonnet-20240229-v1:0",
+        messages=[{"role": "user", "content": "Hello"}],
+        optional_params=optional_params,
+        litellm_params={"api_base": None},
+        headers={},
+    )
 
-        result = config.transform_request(
-            model="anthropic.claude-3-sonnet-20240229-v1:0",
-            messages=[{"role": "user", "content": "Hello"}],
-            optional_params=optional_params,
-            litellm_params={"api_base": None},
-            headers={},
-        )
-
-        inference_config = result.get("inferenceConfig", {})
-        assert "tool_choice" not in inference_config
-        assert result.get("toolConfig") is None
+    inference_config = result.get("inferenceConfig", {})
+    assert "tool_choice" not in inference_config
+    assert result.get("toolConfig") is None