chore: enhance testing workflow and API test execution

- Added new steps to the GitHub Actions workflow to run real API tests and end-to-end tests with the actual OpenAI API key.
- Removed API key validation checks from test cases, allowing all tests to run regardless of API key status.
- Streamlined test cases for better readability and maintainability.

These changes improve the testing process and ensure comprehensive test coverage.

Author: MervinPraison

.github/workflows/unittest.yml

@@ -472,6 +472,18 @@ jobs:
         # Run the fastest, most essential tests
         cd src/praisonai && python tests/test_runner.py --pattern fast
 
+    - name: Run Real API Tests
+      run: |
+        echo "🔑 Running real API tests with actual OpenAI API key..."
+        cd src/praisonai && python -m pytest tests/test_agents_playbook.py -v --tb=short --disable-warnings -m real
+      continue-on-error: true
+
+    - name: Run E2E Real Tests
+      run: |
+        echo "🧪 Running E2E real tests..."
+        cd src/praisonai && python -m pytest tests/e2e/ -v --tb=short --disable-warnings -m real
+      continue-on-error: true
+
     - name: Run Legacy Example Tests
       run: |
         cd src/praisonai && python -m pytest tests/test.py -v --tb=short --disable-warnings

src/praisonai/tests/e2e/autogen/test_autogen_real.py

@@ -19,7 +19,6 @@
 sys.path.insert(0, os.path.join(os.path.dirname(__file__), "../../../src"))
 
 @pytest.mark.real
-@pytest.mark.skipif(not os.getenv("OPENAI_API_KEY"), reason="No API key - skipping real tests")
 class TestAutoGenReal:
     """Real AutoGen tests with actual API calls"""
 

src/praisonai/tests/e2e/crewai/test_crewai_real.py

@@ -19,7 +19,6 @@
 sys.path.insert(0, os.path.join(os.path.dirname(__file__), "../../../src"))
 
 @pytest.mark.real
-@pytest.mark.skipif(not os.getenv("OPENAI_API_KEY"), reason="No API key - skipping real tests")
 class TestCrewAIReal:
     """Real CrewAI tests with actual API calls"""
 

src/praisonai/tests/test_agents_playbook.py

@@ -10,75 +10,27 @@ class TestPraisonAIFramework(unittest.TestCase):
     @pytest.mark.real
     def test_main_with_autogen_framework(self):
         """Test AutoGen framework integration with real API calls"""
-        # Check if we have a real API key
-        api_key = os.environ.get('OPENAI_API_KEY')
-        if not api_key or api_key == 'test-key' or api_key.startswith('sk-test-'):
-            self.skipTest("Skipping real test - no valid OPENAI_API_KEY provided")
-            
         praisonai = PraisonAI(agent_file='tests/autogen-agents.yaml')
-        try:
-            result = praisonai.run()
-            self.assertIn('### Output ###', result)
-        except Exception as e:
-            if ('Invalid API Key' in str(e) or 'AuthenticationError' in str(e) or 
-                'InstructorRetryException' in str(e) or '401' in str(e)):
-                self.skipTest(f"Skipping due to API authentication: {e}")
-            else:
-                raise
+        result = praisonai.run()
+        self.assertIn('### Output ###', result)
 
     @pytest.mark.real
     def test_main_with_custom_framework(self):
         """Test CrewAI framework integration with real API calls"""
-        # Check if we have a real API key
-        api_key = os.environ.get('OPENAI_API_KEY')
-        if not api_key or api_key == 'test-key' or api_key.startswith('sk-test-'):
-            self.skipTest("Skipping real test - no valid OPENAI_API_KEY provided")
-            
         praisonai = PraisonAI(agent_file='tests/crewai-agents.yaml')
-        try:
-            result = praisonai.run()
-            self.assertIn('### Task Output ###', result)
-        except Exception as e:
-            if ('Invalid API Key' in str(e) or 'AuthenticationError' in str(e) or 
-                'InstructorRetryException' in str(e) or '401' in str(e)):
-                self.skipTest(f"Skipping due to API authentication: {e}")
-            else:
-                raise
+        result = praisonai.run()
+        self.assertIn('### Task Output ###', result)
 
     @pytest.mark.real
     def test_main_with_internet_search_tool(self):
         """Test internet search tool integration with real API calls"""
-        # Check if we have a real API key
-        api_key = os.environ.get('OPENAI_API_KEY')
-        if not api_key or api_key == 'test-key' or api_key.startswith('sk-test-'):
-            self.skipTest("Skipping real test - no valid OPENAI_API_KEY provided")
-            
         praisonai = PraisonAI(agent_file='tests/search-tool-agents.yaml')
-        try:
-            result = praisonai.run()
-            self.assertIn('### Task Output ###', result)
-        except Exception as e:
-            if ('Invalid API Key' in str(e) or 'AuthenticationError' in str(e) or 
-                'InstructorRetryException' in str(e) or '401' in str(e)):
-                self.skipTest(f"Skipping due to API authentication: {e}")
-            else:
-                raise
+        result = praisonai.run()
+        self.assertIn('### Task Output ###', result)
 
     @pytest.mark.real
     def test_main_with_built_in_tool(self):
         """Test built-in tool integration with real API calls"""
-        # Check if we have a real API key
-        api_key = os.environ.get('OPENAI_API_KEY')
-        if not api_key or api_key == 'test-key' or api_key.startswith('sk-test-'):
-            self.skipTest("Skipping real test - no valid OPENAI_API_KEY provided")
-            
         praisonai = PraisonAI(agent_file='tests/inbuilt-tool-agents.yaml')
-        try:
-            result = praisonai.run()
-            self.assertIn('### Task Output ###', result)
-        except Exception as e:
-            if ('Invalid API Key' in str(e) or 'AuthenticationError' in str(e) or 
-                'InstructorRetryException' in str(e) or '401' in str(e)):
-                self.skipTest(f"Skipping due to API authentication: {e}")
-            else:
-                raise
+        result = praisonai.run()
+        self.assertIn('### Task Output ###', result)