refactor: enhance test_autoagents.py with improved mock configuration

- Updated the first response to use a malformed JSON string for clarity in error simulation.
- Introduced a serializable configuration for the mock client to ensure valid data handling in tests.
- Enhanced comments for better understanding of the test setup and scenarios.

These changes improve the clarity and robustness of the test cases while maintaining the integrity of the testing framework.

Author: MervinPraison

src/praisonai/tests/unit/test_autoagents.py

@@ -547,19 +547,8 @@ def test_retry_with_previous_response_in_prompt(self, mock_llm_class, mock_suppo
         # Mock LLM instance
         mock_llm = Mock()
 
-        # First response: invalid JSON that will fail validation
-        first_response = json.dumps({
-            "main_instruction": "Test",
-            "process_type": "sequential",
-            "agents": [{
-                "name": "Agent",
-                "role": "Role",
-                "goal": "Goal",
-                "backstory": "Story",
-                "tools": [],
-                "tasks": ["String task instead of object"]
-            }]
-        })
+        # First response: malformed JSON that will fail parsing
+        first_response = "{ invalid json structure"
 
         # Second response: valid config
         second_response = json.dumps(sample_valid_config.model_dump())
@@ -598,9 +587,32 @@ def test_custom_api_key_and_base_url(self, mock_get_client, mock_openai_class, m
         # Mock support for structured outputs
         mock_supports_structured.return_value = True
 
+        # Create a proper mock config with serializable data
+        serializable_config = AutoAgentsConfig(
+            main_instruction="Test",
+            process_type="sequential",
+            agents=[
+                AgentConfig(
+                    name="Test Agent",
+                    role="Tester",
+                    goal="Test goal",
+                    backstory="Test story",
+                    tools=[],
+                    tasks=[
+                        TaskConfig(
+                            name="Test Task",
+                            description="Test description",
+                            expected_output="Test output",
+                            tools=[]
+                        )
+                    ]
+                )
+            ]
+        )
+        
         # Mock OpenAI client instance
         mock_client = Mock(spec=OpenAIClient)
-        mock_client.parse_structured_output.return_value = sample_valid_config
+        mock_client.parse_structured_output.return_value = serializable_config
         mock_openai_class.return_value = mock_client
 
         custom_api_key = "custom-api-key"