fix: address reviewer feedback - improve skip guards and test parametrization

- Fix Ollama skip guard: move requests import inside try-catch for consistency
- Remove unnecessary google.generativeai import guard in Gemini example (litellm handles via REST)
- Parametrize compute provider test to reduce duplication and cover more providers

Co-authored-by: Mervin Praison <MervinPraison@users.noreply.github.com>

Author: praisonai-triage-agent[bot]

examples/python/managed-agents/provider/runtime_local_gemini.py

@@ -11,11 +11,8 @@
     print("[skip] GEMINI_API_KEY or GOOGLE_API_KEY not set")
     sys.exit(0)
 
-try:
-    import google.generativeai
-except ImportError:
-    print("[skip] google-generativeai SDK not installed")
-    sys.exit(0)
+# Note: google.generativeai SDK is not required - litellm handles Gemini via REST API
+# Only GEMINI_API_KEY/GOOGLE_API_KEY is needed for the gemini/ model prefix
 
 # Heavy imports only after skip-guards pass
 from praisonai import Agent

examples/python/managed-agents/provider/runtime_local_ollama.py

@@ -5,15 +5,20 @@
 """
 import os
 import sys
-import requests
 
 # Skip guards - exit cleanly if prerequisites not met  
-# Check if Ollama daemon is running
+# Check if requests is available and Ollama daemon is running
+try:
+    import requests
+except ImportError:
+    print("[skip] requests library not installed")
+    sys.exit(0)
+
 try:
     response = requests.get("http://localhost:11434/api/tags", timeout=2)
-    if response.status_code != 200:
-        raise Exception("Ollama not responding")
-except Exception:
+except requests.RequestException:
+    response = None
+if response is None or response.status_code != 200:
     print("[skip] Ollama daemon not running at localhost:11434")
     sys.exit(0)
 

tests/unit/integrations/test_backend_semantics.py

@@ -94,22 +94,15 @@ def test_managed_agent_deprecation_warnings():
             assert "LocalAgent" in str(w[0].message)
 
 
-def test_managed_agent_compute_provider_warnings():
-    """Test that ManagedAgent(provider='modal'|'e2b'|...) emits DeprecationWarning and returns LocalManagedAgent."""
+@pytest.mark.parametrize("provider", ["modal", "e2b", "docker", "flyio", "daytona"])
+def test_managed_agent_compute_provider_warnings(provider):
+    """Test that ManagedAgent(provider=<compute>) emits DeprecationWarning and returns LocalManagedAgent."""
     from praisonai.integrations.managed_agents import ManagedAgent
     from praisonai.integrations.managed_local import LocalManagedAgent
 
     # Compute providers should emit DeprecationWarning and return LocalManagedAgent (backward compatibility)
     with pytest.warns(DeprecationWarning, match="compute.*deprecated"):
-        obj = ManagedAgent(provider="modal")
-    assert isinstance(obj, LocalManagedAgent)
-    
-    with pytest.warns(DeprecationWarning, match="compute.*deprecated"):
-        obj = ManagedAgent(provider="e2b")
-    assert isinstance(obj, LocalManagedAgent)
-    
-    with pytest.warns(DeprecationWarning, match="compute.*deprecated"):
-        obj = ManagedAgent(provider="docker")
+        obj = ManagedAgent(provider=provider)
     assert isinstance(obj, LocalManagedAgent)