Some PR changes

agg23 · agg23 · commit 196f0da14e0f · 2025-04-24T12:43:30.000-07:00
diff --git a/tests/async/test_accessibility.py b/tests/async/test_accessibility.py
@@ -20,9 +20,9 @@
 
 
 async def test_accessibility_should_work(
-    page: Page, is_firefox: bool, is_chromium: bool
+    page: Page, is_firefox: bool, is_chromium: bool, is_webkit: bool
 ) -> None:
-    if not is_firefox and not is_chromium and sys.platform == "darwin":
+    if is_webkit and sys.platform == "darwin":
         pytest.skip("Test disabled on WebKit on macOS")
     await page.set_content(
         """<head>
diff --git a/tests/async/test_fetch_global.py b/tests/async/test_fetch_global.py
@@ -534,9 +534,9 @@ async def test_should_follow_max_redirects(
     def _handle_request(req: TestServerRequest) -> None:
         nonlocal redirect_count
         redirect_count += 1
-        req.setResponseCode(301),
-        req.setHeader("Location", server.EMPTY_PAGE),
-        req.finish(),
+        req.setResponseCode(301)
+        req.setHeader("Location", server.EMPTY_PAGE)
+        req.finish()
 
     server.set_route("/empty.html", _handle_request)
     request = await playwright.request.new_context(max_redirects=1)
diff --git a/tests/sync/test_accessibility.py b/tests/sync/test_accessibility.py
@@ -21,8 +21,10 @@
 
 
 def test_accessibility_should_work(
-    page: Page, is_firefox: bool, is_chromium: bool
+    page: Page, is_firefox: bool, is_chromium: bool, is_webkit: bool
 ) -> None:
+    if is_webkit and sys.platform == "darwin":
+        pytest.skip("Test disabled on WebKit on macOS")
     page.set_content(
         """<head>
       <title>Accessibility Test</title>
diff --git a/tests/sync/test_fetch_global.py b/tests/sync/test_fetch_global.py
@@ -369,9 +369,9 @@ def test_should_follow_max_redirects(playwright: Playwright, server: Server) ->
     def _handle_request(req: TestServerRequest) -> None:
         nonlocal redirect_count
         redirect_count += 1
-        req.setResponseCode(301),
-        req.setHeader("Location", server.EMPTY_PAGE),
-        req.finish(),
+        req.setResponseCode(301)
+        req.setHeader("Location", server.EMPTY_PAGE)
+        req.finish()
 
     server.set_route("/empty.html", _handle_request)
     request = playwright.request.new_context(max_redirects=1)
