fix: setDefaultTimeout

Author: mxschmitt

playwright/sync_base.py

@@ -84,6 +84,8 @@ def __str__(self) -> str:
         return self._impl_obj.__str__()
 
     def _sync(self, task: asyncio.Future) -> Any:
+        if not task:
+            return None
         g_self = greenlet.getcurrent()
         future = self._loop.create_task(task)
 

tests/test_sync.py

@@ -16,7 +16,7 @@
 
 import pytest
 
-from playwright import Error, sync_playwright
+from playwright import Error, TimeoutError, sync_playwright
 from playwright.sync_api import Browser, Page
 
 
@@ -214,3 +214,10 @@ def test_sync_playwright_multiple_times():
             with sync_playwright() as pw2:
                 assert pw1.chromium == pw2.chromium
         assert "Can only run one Playwright at a time." in exc.value.message
+
+
+def test_sync_set_default_timeout(page):
+    page.setDefaultTimeout(1)
+    with pytest.raises(TimeoutError) as exc:
+        page.waitForFunction("false")
+    assert "Timeout 1ms exceeded." in exc.value.message