test: fix LLM config to use provider string instead of object

Author: firefox-669

tests/conftest.py

@@ -31,7 +31,7 @@ def mock_llm_config():
         'openhands': {
             'model': 'mock-gpt-4',
             'api_key': 'test-key',
-            'llm_provider': MockLLMProvider(),
+            'provider': 'mock',  # Add provider type
         },
         'monitor': {
             'quality_threshold': 0.8,

tests/test_orchestrator.py

@@ -85,7 +85,7 @@ def config(self):
             'openhands': {
                 'model': 'mock-gpt-4',
                 'api_key': 'test-key',
-                'llm_provider': MockLLMProvider(),
+                'provider': 'mock',
             },
             'monitor': {'quality_threshold': 0.8},
             'governance': {'enable_gatekeeping': True}
@@ -106,7 +106,7 @@ def test_init_with_custom_config(self):
             'openhands': {
                 'model': 'mock-claude-3',
                 'api_key': 'test-key',
-                'llm_provider': MockLLMProvider(),
+                'provider': 'mock',
             },
             'monitor': {'quality_threshold': 0.9},
             'governance': {'enable_gatekeeping': False},
@@ -165,7 +165,7 @@ def orchestrator(self):
             'openhands': {
                 'model': 'mock-gpt-4',
                 'api_key': 'test-key',
-                'llm_provider': MockLLMProvider(),
+                'provider': 'mock',
             },
             'monitor': {'quality_threshold': 0.8},
         }
@@ -206,7 +206,7 @@ def orchestrator(self):
             'openhands': {
                 'model': 'mock-gpt-4',
                 'api_key': 'test-key',
-                'llm_provider': MockLLMProvider(),
+                'provider': 'mock',
             },
             'monitor': {'quality_threshold': 0.8},
         }