Merge pull request #802 from microsoft/psl-codeql

fix: Fixed the code quality issue

Author: Roopan-Microsoft

src/tests/services/test_orchestrator.py

@@ -1,10 +1,10 @@
 import base64
 import json
+import sys
 from unittest.mock import AsyncMock, MagicMock, patch
 
 import pytest
 
-import orchestrator
 from orchestrator import (_HARMFUL_PATTERNS_COMPILED,
                           _SYSTEM_PROMPT_PATTERNS_COMPILED,
                           PLANNING_INSTRUCTIONS, RAI_HARMFUL_CONTENT_RESPONSE,
@@ -841,7 +841,7 @@ def test_get_orchestrator_singleton():
         mock_builder.return_value = mock_builder_instance
 
         # Reset the singleton
-        orchestrator._orchestrator = None
+        sys.modules["orchestrator"]._orchestrator = None
 
         instance1 = get_orchestrator()
         instance2 = get_orchestrator()
@@ -1518,7 +1518,7 @@ async def test_get_chat_client_foundry_mode():
 def test_foundry_not_available():
     """Test when Foundry SDK is not available."""
     # Check that FOUNDRY_AVAILABLE is defined
-    assert hasattr(orchestrator, 'FOUNDRY_AVAILABLE')
+    assert hasattr(sys.modules["orchestrator"], 'FOUNDRY_AVAILABLE')
 
 # Tests for workflow event handling (lines 736-799, 841-895)
 # Note: These are integration-level tests that verify the workflow event