refactor: remove unused imports from test files for improved code quality

Author: Kingshuk-Microsoft

src/tests/backend/common/config/test_app_config.py

@@ -13,7 +13,6 @@
 import os
 import logging
 from unittest.mock import patch, MagicMock, AsyncMock
-from azure.identity import ManagedIdentityCredential
 
 # Add the source root directory to the Python path for imports
 import sys

src/tests/backend/common/database/test_cosmosdb.py

@@ -4,7 +4,6 @@
 import logging
 import sys
 import os
-from typing import Dict, List, Optional
 from unittest.mock import AsyncMock, Mock, patch
 import pytest
 

src/tests/backend/common/database/test_database_base.py

@@ -4,7 +4,7 @@
 import os
 from abc import ABC
 from typing import Any, Dict, List, Optional, Type
-from unittest.mock import Mock, patch
+from unittest.mock import Mock
 import pytest
 
 # Add the backend directory to the Python path

src/tests/backend/v4/common/services/test_base_api_service.py

@@ -13,7 +13,6 @@
 import sys
 import importlib.util
 from unittest.mock import patch, MagicMock, AsyncMock, Mock
-from typing import Dict, Optional, Union
 import aiohttp
 from aiohttp import ClientTimeout, ClientSession
 

src/tests/backend/v4/common/services/test_mcp_service.py

@@ -14,8 +14,7 @@
 import sys
 import asyncio
 import importlib.util
-from unittest.mock import patch, MagicMock, Mock
-from typing import Dict, Optional
+from unittest.mock import patch, MagicMock
 from aiohttp import ClientTimeout, ClientError
 
 # Add the src directory to sys.path for proper import

src/tests/backend/v4/common/services/test_team_service.py

@@ -19,7 +19,6 @@
 import uuid
 import importlib.util
 from unittest.mock import patch, MagicMock, AsyncMock
-from typing import Dict, Optional, List, Tuple
 from datetime import datetime, timezone
 
 # Add the src directory to sys.path for proper import

src/tests/backend/v4/config/test_settings.py

@@ -432,7 +432,7 @@ async def cancel_task():
         cancel_task_handle = asyncio.create_task(cancel_task())
 
         with self.assertRaises(asyncio.CancelledError):
-            result = await task
+            await task
 
         await cancel_task_handle
 
@@ -454,7 +454,7 @@ async def cancel_task():
         with self.assertRaises(asyncio.CancelledError):
             await task
 
-        _ = await cancel_task_handle
+        await cancel_task_handle
 
     def test_cleanup_approval(self):
         """Test cleanup approval."""

src/tests/backend/v4/magentic_agents/test_foundry_agent.py

@@ -2,7 +2,7 @@
 
 import sys
 import os
-from unittest.mock import Mock, patch, AsyncMock, call
+from unittest.mock import Mock, patch, AsyncMock
 import pytest
 
 # Add the backend directory to the Python path

src/tests/backend/v4/magentic_agents/test_magentic_agent_factory.py

@@ -2,7 +2,7 @@
 import logging
 import sys
 from types import SimpleNamespace
-from unittest.mock import Mock, patch, AsyncMock
+from unittest.mock import Mock, AsyncMock
 import pytest
 
 # Mock the dependencies before importing the module under test

src/tests/backend/v4/orchestration/test_human_approval_manager.py

@@ -7,7 +7,6 @@
 import os
 import sys
 import unittest
-from typing import Optional
 from unittest.mock import Mock, AsyncMock, patch
 
 # Add the backend directory to the Python path