Merge pull request #796 from microsoft/psl-codefix

fix: fixed new suggestions by copilot

Author: Roopan-Microsoft

src/tests/backend/auth/conftest.py

@@ -5,7 +5,6 @@
 import pytest
 import sys
 import os
-from unittest.mock import MagicMock, patch
 import base64
 import json
 

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

@@ -16,9 +16,7 @@
 import asyncio
 import logging
 import importlib.util
-from unittest.mock import patch, MagicMock, AsyncMock, Mock
-from typing import Any, Dict, Optional, List, Union
-from dataclasses import dataclass
+from unittest.mock import patch, MagicMock
 
 # Add the src directory to sys.path for proper import
 src_path = os.path.join(os.path.dirname(__file__), '..', '..', '..', '..')

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

@@ -5,7 +5,6 @@
 including registration, unregistration, cleanup, and monitoring functionality.
 """
 
-import asyncio
 import logging
 import os
 import sys

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

@@ -8,7 +8,6 @@
 import os
 import sys
 import unittest
-from unittest import IsolatedAsyncioTestCase
 from unittest.mock import AsyncMock, Mock, patch
 
 # Add the backend directory to the Python path
@@ -162,7 +161,7 @@ def test_ad_token_provider(self, mock_config):
         self.assertEqual(token, "test-token-123")
         mock_credential.get_token.assert_called_once_with(mock_config.AZURE_COGNITIVE_SERVICES)
 
-class TestAzureConfigAsync(IsolatedAsyncioTestCase):
+class TestAzureConfigAsync(unittest.IsolatedAsyncioTestCase):
     """Async test cases for AzureConfig class."""
 
     @patch('backend.v4.config.settings.AzureOpenAIChatClient')
@@ -284,7 +283,7 @@ def test_overwrite_existing_team(self):
         self.assertEqual(config.get_current_team(user_id), team_config2)
 
 
-class TestOrchestrationConfig(IsolatedAsyncioTestCase):
+class TestOrchestrationConfig(unittest.IsolatedAsyncioTestCase):
     """Test cases for OrchestrationConfig class."""
 
     def test_orchestration_config_creation(self):
@@ -490,7 +489,7 @@ def test_cleanup_clarification(self):
         self.assertNotIn(request_id, config._clarification_events)
 
 
-class TestConnectionConfig(IsolatedAsyncioTestCase):
+class TestConnectionConfig(unittest.IsolatedAsyncioTestCase):
     """Test cases for ConnectionConfig class."""
 
     def test_connection_config_creation(self):

src/tests/backend/v4/magentic_agents/models/test_agent_models.py

@@ -1,6 +1,6 @@
 """Unit tests for backend.v4.magentic_agents.models.agent_models module."""
 import sys
-from unittest.mock import Mock, patch, MagicMock
+from unittest.mock import patch, MagicMock
 import pytest
 
 

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

@@ -7,8 +7,8 @@
 import logging
 import os
 import sys
+import unittest
 from typing import Any, Optional
-from unittest import IsolatedAsyncioTestCase
 from unittest.mock import Mock, AsyncMock, patch
 
 import pytest
@@ -225,7 +225,7 @@ def convert(plan_text, facts, team, task):
 messages = sys.modules['v4.models.messages']
 
 
-class TestHumanApprovalMagenticManager(IsolatedAsyncioTestCase):
+class TestHumanApprovalMagenticManager(unittest.IsolatedAsyncioTestCase):
     """Test cases for HumanApprovalMagenticManager class."""
 
     def setUp(self):
@@ -697,5 +697,4 @@ async def test_create_progress_ledger_websocket_error(self):
 
 
 if __name__ == '__main__':
-    import unittest
     unittest.main()