test: unskip tests (#840)

mxschmitt · web-flow · commit 6b2898d7c11b · 2021-08-08T16:23:54.000-07:00
diff --git a/tests/async/test_browsercontext_storage_state.py b/tests/async/test_browsercontext_storage_state.py
@@ -15,12 +15,8 @@
 import asyncio
 import json
 
-import pytest
-
 
 async def test_should_capture_local_storage(context, is_webkit, is_win):
-    if is_webkit and is_win:
-        pytest.skip()
     page1 = await context.new_page()
     await page1.route(
         "**/*", lambda route: asyncio.create_task(route.fulfill(body="<html></html>"))
@@ -44,8 +40,6 @@ async def test_should_capture_local_storage(context, is_webkit, is_win):
 
 
 async def test_should_set_local_storage(browser, is_webkit, is_win):
-    if is_webkit and is_win:
-        pytest.skip()
     context = await browser.new_context(
         storage_state={
             "origins": [
diff --git a/tests/async/test_network.py b/tests/async/test_network.py
@@ -198,6 +198,8 @@ def handle(request):
     assert cast(Response, response).request.headers == server_headers
 
 
+# TODO: update once fixed https://github.com/microsoft/playwright/issues/6690
+@pytest.mark.xfail
 async def test_request_headers_should_get_the_same_headers_as_the_server_cors(
     page: Page, server, is_webkit, is_win
 ):
@@ -562,11 +564,10 @@ async def test_network_events_should_support_redirects(page, server):
 
 
 async def test_request_is_navigation_request_should_work(page, server):
-    pytest.skip(msg="test")
     requests = {}
 
     def handle_request(request):
-        requests[request.url().split("/").pop()] = request
+        requests[request.url.split("/").pop()] = request
 
     page.on("request", handle_request)
     server.set_redirect("/rrredirect", "/frames/one-frame.html")
diff --git a/tests/sync/test_browsercontext_storage_state.py b/tests/sync/test_browsercontext_storage_state.py
@@ -14,12 +14,8 @@
 
 import json
 
-import pytest
-
 
 def test_should_capture_local_storage(context, is_webkit, is_win):
-    if is_webkit and is_win:
-        pytest.skip()
     page1 = context.new_page()
     page1.route("**/*", lambda route: route.fulfill(body="<html></html>"))
     page1.goto("https://www.example.com")
@@ -41,8 +37,6 @@ def test_should_capture_local_storage(context, is_webkit, is_win):
 
 
 def test_should_set_local_storage(browser, is_webkit, is_win):
-    if is_webkit and is_win:
-        pytest.skip()
     context = browser.new_context(
         storage_state={
             "origins": [
