tests: remove bloat tests

Author: dbschmigelski

tests/strands/tools/mcp/test_mcp_client.py

@@ -422,93 +422,6 @@ def test_call_tool_sync_without_structured_content(mock_transport, mock_session)
         assert result.get("structuredContent") is None
 
 
-@pytest.mark.asyncio
-async def test_call_tool_async_without_structured_content(mock_transport, mock_session):
-    """Test that call_tool_async works correctly when no structured content is provided."""
-    mock_content = MCPTextContent(type="text", text="Test message")
-    mock_result = MCPCallToolResult(isError=False, content=[mock_content])  # No structuredContent
-    mock_session.call_tool.return_value = mock_result
-
-    with MCPClient(mock_transport["transport_callable"]) as client:
-        with (
-            patch("asyncio.run_coroutine_threadsafe") as mock_run_coroutine_threadsafe,
-            patch("asyncio.wrap_future") as mock_wrap_future,
-        ):
-            mock_future = MagicMock()
-            mock_run_coroutine_threadsafe.return_value = mock_future
-
-            async def mock_awaitable():
-                return mock_result
-
-            mock_wrap_future.return_value = mock_awaitable()
-
-            result = await client.call_tool_async(
-                tool_use_id="test-123", name="test_tool", arguments={"param": "value"}
-            )
-
-        assert result["status"] == "success"
-        assert result["toolUseId"] == "test-123"
-        assert len(result["content"]) == 1
-        assert result["content"][0]["text"] == "Test message"
-        # structuredContent should be None when not provided by MCP
-        assert result.get("structuredContent") is None
-
-
-def test_structured_content_validation():
-    """Test that structured content is properly validated and handled."""
-    # Test that MCPToolResult can be created with structured content
-    structured_data = {"result": {"value": 42}, "metadata": {"timestamp": "2023-01-01"}}
-
-    result = MCPToolResult(
-        status="success",
-        toolUseId="test-123",
-        content=[{"text": "Operation completed"}],
-        structuredContent=structured_data,
-    )
-
-    # Validate that structured content is accessible
-    assert "structuredContent" in result
-    assert result["structuredContent"] == structured_data
-    assert result["structuredContent"]["result"]["value"] == 42
-    assert result["structuredContent"]["metadata"]["timestamp"] == "2023-01-01"
-
-    # Test that regular content is still accessible
-    assert result["content"][0]["text"] == "Operation completed"
-
-    # Test that MCPToolResult without structured content works
-    result_no_structured = MCPToolResult(status="success", toolUseId="test-456", content=[{"text": "Simple operation"}])
-
-    assert result_no_structured.get("structuredContent") is None
-    assert result_no_structured["content"][0]["text"] == "Simple operation"
-
-
-def test_structured_content_separation_from_content_array(mock_transport, mock_session):
-    """Test that structured content is NOT added to the content array (new behavior)."""
-    mock_content = MCPTextContent(type="text", text="Tool executed successfully")
-    structured_content = {"data": {"items": [1, 2, 3]}, "pagination": {"hasMore": False, "total": 3}}
-
-    mock_session.call_tool.return_value = MCPCallToolResult(
-        isError=False, content=[mock_content], structuredContent=structured_content
-    )
-
-    with MCPClient(mock_transport["transport_callable"]) as client:
-        result = client.call_tool_sync(tool_use_id="test-789", name="data_tool", arguments={})
-
-        # Verify the content array only contains the original MCP content
-        assert len(result["content"]) == 1
-        assert result["content"][0]["text"] == "Tool executed successfully"
-
-        # Verify no JSON object was added to the content array (old behavior)
-        for content_item in result["content"]:
-            assert "json" not in content_item
-
-        # Verify structured content is in its own field (new behavior)
-        assert "structuredContent" in result
-        assert result["structuredContent"] == structured_content
-        assert result["structuredContent"]["data"]["items"] == [1, 2, 3]
-        assert result["structuredContent"]["pagination"]["hasMore"] is False
-
-
 def test_exception_when_future_not_running():
     """Test exception handling when the future is not running."""
     # Create a client.with a mock transport