fix tests

Author: SentienceDEV

tests/unit/conftest.py

@@ -9,7 +9,8 @@
 pure unit/contract tests without requiring Playwright.
 
 IMPORTANT:
-- These stubs are only active during pytest runs (via conftest import order).
+- These stubs are ONLY loaded when Playwright is NOT installed.
+- When Playwright IS installed, real Playwright is used for all tests.
 - Integration/E2E tests that need real Playwright should install Playwright and will
   typically run in separate environments.
 """
@@ -20,52 +21,61 @@
 import types
 
 
-def _ensure_module(name: str) -> types.ModuleType:
-    if name in sys.modules:
-        return sys.modules[name]
-    mod = types.ModuleType(name)
-    sys.modules[name] = mod
-    return mod
+def _ensure_playwright_stubs() -> None:
+    """
+    Provide minimal `playwright.*` stubs so the SDK can be imported in environments
+    where Playwright isn't installed (e.g., constrained CI/sandbox).
 
+    This is only intended to support pure unit/contract tests that don't actually
+    launch browsers.
+    """
 
-# Create top-level playwright module and submodules
-playwright_mod = _ensure_module("playwright")
-async_api_mod = _ensure_module("playwright.async_api")
-sync_api_mod = _ensure_module("playwright.sync_api")
+    def _ensure_module(name: str) -> types.ModuleType:
+        if name in sys.modules:
+            return sys.modules[name]
+        mod = types.ModuleType(name)
+        sys.modules[name] = mod
+        return mod
 
+    # Create top-level playwright module and submodules
+    playwright_mod = _ensure_module("playwright")
+    async_api_mod = _ensure_module("playwright.async_api")
+    sync_api_mod = _ensure_module("playwright.sync_api")
 
-class _Dummy:
-    """Placeholder type used for Playwright classes in unit tests."""
+    class _Dummy:
+        """Placeholder type used for Playwright classes in unit tests."""
 
+    # Minimal symbols imported by `sentience.browser`
+    async_api_mod.BrowserContext = _Dummy
+    async_api_mod.Browser = _Dummy
+    async_api_mod.Page = _Dummy
+    async_api_mod.Playwright = _Dummy
+    async_api_mod.PlaywrightContextManager = _Dummy
 
-# Minimal symbols imported by `sentience.browser`
-async_api_mod.BrowserContext = _Dummy
-async_api_mod.Browser = _Dummy
-async_api_mod.Page = _Dummy
-async_api_mod.Playwright = _Dummy
-async_api_mod.PlaywrightContextManager = _Dummy
+    async def _async_playwright():
+        raise RuntimeError("Playwright is not available in this unit-test environment.")
 
+    async_api_mod.async_playwright = _async_playwright
 
-async def _async_playwright():
-    raise RuntimeError("Playwright is not available in this unit-test environment.")
+    sync_api_mod.BrowserContext = _Dummy
+    sync_api_mod.Browser = _Dummy
+    sync_api_mod.Page = _Dummy
+    sync_api_mod.Playwright = _Dummy
+    sync_api_mod.PlaywrightContextManager = _Dummy
 
+    def _sync_playwright():
+        raise RuntimeError("Playwright is not available in this unit-test environment.")
 
-async_api_mod.async_playwright = _async_playwright
+    sync_api_mod.sync_playwright = _sync_playwright
 
-sync_api_mod.BrowserContext = _Dummy
-sync_api_mod.Browser = _Dummy
-sync_api_mod.Page = _Dummy
-sync_api_mod.Playwright = _Dummy
-sync_api_mod.PlaywrightContextManager = _Dummy
+    # Expose submodules on the top-level module for completeness
+    playwright_mod.async_api = async_api_mod
+    playwright_mod.sync_api = sync_api_mod
 
 
-def _sync_playwright():
-    raise RuntimeError("Playwright is not available in this unit-test environment.")
-
-
-sync_api_mod.sync_playwright = _sync_playwright
-
-
-# Expose submodules on the top-level module for completeness
-playwright_mod.async_api = async_api_mod
-playwright_mod.sync_api = sync_api_mod
+# Only load stubs if Playwright is NOT available
+# This prevents overwriting real Playwright when it IS installed
+try:
+    import playwright  # noqa: F401
+except ImportError:
+    _ensure_playwright_stubs()