From 69b32dd8702aca128a68a0b607c0384e20dde59b Mon Sep 17 00:00:00 2001
From: Haji Akhundov <hajiakhundov@optiver.com>
Date: Wed, 3 Dec 2025 21:06:32 +0100
Subject: [PATCH] Fix ruff linting errors: remove unused imports and variables

---
 .../scim/test_scim_transformations.py                    | 9 ---------
 .../management_endpoints/scim/test_scim_v2_endpoints.py  | 9 +++------
 2 files changed, 3 insertions(+), 15 deletions(-)

diff --git a/tests/test_litellm/proxy/management_endpoints/scim/test_scim_transformations.py b/tests/test_litellm/proxy/management_endpoints/scim/test_scim_transformations.py
index be9c79911d0..e5857a10967 100644
--- a/tests/test_litellm/proxy/management_endpoints/scim/test_scim_transformations.py
+++ b/tests/test_litellm/proxy/management_endpoints/scim/test_scim_transformations.py
@@ -1,14 +1,8 @@
-import asyncio
-import json
 import os
 import sys
-from litellm._uuid import uuid
-from typing import Optional, cast
 from unittest.mock import AsyncMock, MagicMock, patch
 
 import pytest
-from fastapi import HTTPException
-from fastapi.testclient import TestClient
 
 sys.path.insert(
     0, os.path.abspath("../../../")
@@ -19,10 +13,7 @@ from litellm.proxy.management_endpoints.scim.scim_transformations import (
     ScimTransformations,
 )
 from litellm.types.proxy.management_endpoints.scim_v2 import (
-    SCIMGroup,
-    SCIMPatchOp,
     SCIMPatchOperation,
-    SCIMUser,
 )
 
 
diff --git a/tests/test_litellm/proxy/management_endpoints/scim/test_scim_v2_endpoints.py b/tests/test_litellm/proxy/management_endpoints/scim/test_scim_v2_endpoints.py
index 15c4272fa90..cf0b9c58760 100644
--- a/tests/test_litellm/proxy/management_endpoints/scim/test_scim_v2_endpoints.py
+++ b/tests/test_litellm/proxy/management_endpoints/scim/test_scim_v2_endpoints.py
@@ -15,7 +15,6 @@ from litellm.proxy.management_endpoints.scim.scim_v2 import (
     update_user,
 )
 from litellm.types.proxy.management_endpoints.scim_v2 import (
-    SCIMFeature,
     SCIMGroup,
     SCIMMember,
     SCIMPatchOp,
@@ -428,7 +427,7 @@ async def test_update_user_success(mocker):
         "litellm.proxy.management_endpoints.scim.scim_v2._handle_team_membership_changes",
         AsyncMock()
     )
-    mock_transform = mocker.patch(
+    mocker.patch(
         "litellm.proxy.management_endpoints.scim.scim_v2.ScimTransformations.transform_litellm_user_to_scim_user",
         AsyncMock(return_value=response_scim_user)
     )
@@ -524,7 +523,7 @@ async def test_patch_user_success(mocker):
         "litellm.proxy.management_endpoints.scim.scim_v2._handle_team_membership_changes",
         AsyncMock()
     )
-    mock_transform = mocker.patch(
+    mocker.patch(
         "litellm.proxy.management_endpoints.scim.scim_v2.ScimTransformations.transform_litellm_user_to_scim_user",
         AsyncMock(return_value=response_scim_user)
     )
@@ -660,7 +659,7 @@ async def test_update_group_metadata_serialization_issue(mocker):
     )
     
     # Call the function that had the bug
-    result = await update_group(group_id=group_id, group=scim_group)
+    await update_group(group_id=group_id, group=scim_group)
     
     # Verify the team update was called
     mock_prisma_client.db.litellm_teamtable.update.assert_called_once()
@@ -696,7 +695,6 @@ async def test_team_membership_management(mocker):
     from litellm.proxy.management_endpoints.scim.scim_v2 import (
         _get_team_member_user_ids_from_team,
         _handle_group_membership_changes,
-        patch_team_membership,
     )
 
     # Mock team with members_with_roles as source of truth
@@ -772,7 +770,6 @@ async def test_update_group_e2e(mocker):
     from litellm.proxy.management_endpoints.scim.scim_transformations import (
         ScimTransformations,
     )
-    from litellm.proxy.utils import safe_dumps
 
     # Setup test data
     group_id = "test-team-123"