Refactor WCA provider test to avoid global state patching

Updated TestHasWCAProvidersWithWCAProvider to patch settings and app config locally within the test method, removing global state modifications in setUp and tearDown. This improves test isolation and reliability.

Author: mabashian

ansible_ai_connect/main/tests/test_utils.py

@@ -25,32 +25,23 @@
 
 from unittest.mock import patch
 
-from django.apps import apps
-from django.test import override_settings
-
 from ansible_ai_connect.ai.api.model_pipelines.factory import ModelPipelineFactory
 from ansible_ai_connect.ai.api.model_pipelines.tests import mock_config
 from ansible_ai_connect.main.utils import has_wca_providers
 from ansible_ai_connect.test_utils import WisdomServiceLogAwareTestCase
 
 
-@override_settings(ANSIBLE_AI_MODEL_MESH_CONFIG=mock_config("wca"))
 class TestHasWCAProvidersWithWCAProvider(WisdomServiceLogAwareTestCase):
-    def setUp(self):
-        super().setUp()
-        # Patch the pipeline factory to avoid test interference
-        self.pipeline_factory_patcher = patch.object(
-            apps.get_app_config("ai"), "_pipeline_factory", ModelPipelineFactory()
-        )
-        self.pipeline_factory_patcher.start()
-
-    def tearDown(self):
-        self.pipeline_factory_patcher.stop()
-        super().tearDown()
-
     def test_returns_true_with_wca_provider(self):
         """Test that has_wca_providers returns True when WCA provider is configured."""
-        self.assertTrue(has_wca_providers())
+        # Create a factory with WCA config directly, without affecting global state
+        with patch(
+            "ansible_ai_connect.main.settings.base.ANSIBLE_AI_MODEL_MESH_CONFIG", mock_config("wca")
+        ):
+            factory = ModelPipelineFactory()
+            with patch("ansible_ai_connect.main.utils.apps.get_app_config") as mock_app:
+                mock_app.return_value._pipeline_factory = factory
+                self.assertTrue(has_wca_providers())
 
 
 # @override_settings(ANSIBLE_AI_MODEL_MESH_CONFIG=mock_config("wca-onprem"))