test: cleanup tests by removing print statements (microsoft#633)

mxschmitt · web-flow · commit 1894c6b5c47c · 2021-04-20T17:24:46.000+02:00
diff --git a/tests/async/test_browsercontext_proxy.py b/tests/async/test_browsercontext_proxy.py
@@ -73,7 +73,6 @@ async def test_should_work_with_ip_port_notion(context_factory, server):
 
 async def test_should_authenticate(context_factory, server):
     def handler(req):
-        print(req)
         auth = req.getHeader("proxy-authorization")
         if not auth:
             req.setHeader(
@@ -102,7 +101,6 @@ def handler(req):
 
 async def test_should_authenticate_with_empty_password(context_factory, server):
     def handler(req):
-        print(req)
         auth = req.getHeader("proxy-authorization")
         if not auth:
             req.setHeader(
diff --git a/tests/async/test_launcher.py b/tests/async/test_launcher.py
@@ -61,25 +61,6 @@ async def test_browser_type_launch_should_reject_if_executable_path_is_invalid(
     assert "browser was not found" in exc.value.message
 
 
-@pytest.mark.skip()
-async def test_browser_type_launch_server_should_return_child_process_instance(
-    browser_type, launch_arguments
-):
-    browser_server = await browser_type.launchServer(**launch_arguments)
-    assert browser_server.pid > 0
-    await browser_server.close()
-
-
-@pytest.mark.skip()
-async def test_browser_type_launch_server_should_fire_close_event(
-    browser_type, launch_arguments
-):
-    browser_server = await browser_type.launchServer(**launch_arguments)
-    close_event = asyncio.Future()
-    browser_server.on("close", lambda: close_event.set_result(None))
-    await asyncio.gather(close_event, browser_server.close())
-
-
 async def test_browser_type_executable_path_should_work(browser_type, browser_channel):
     if browser_channel:
         return
diff --git a/tests/async/test_proxy.py b/tests/async/test_proxy.py
@@ -74,7 +74,6 @@ async def test_should_work_with_ip_port_notion(browser_factory, server):
 
 async def test_should_authenticate(browser_factory, server):
     def handler(req):
-        print(req)
         auth = req.getHeader("proxy-authorization")
         if not auth:
             req.setHeader(
@@ -103,7 +102,6 @@ def handler(req):
 
 async def test_should_authenticate_with_empty_password(browser_factory, server):
     def handler(req):
-        print(req)
         auth = req.getHeader("proxy-authorization")
         if not auth:
             req.setHeader(
diff --git a/tests/sync/test_element_handle.py b/tests/sync/test_element_handle.py
@@ -182,7 +182,6 @@ def test_owner_frame_for_iframe_elements(page, server, utils):
     utils.attach_frame(page, "frame1", server.EMPTY_PAGE)
     frame = page.main_frame
     element_handle = frame.evaluate_handle('document.querySelector("#frame1")')
-    print(element_handle)
     assert element_handle.owner_frame() == frame
 
 
diff --git a/tests/sync/test_har.py b/tests/sync/test_har.py
@@ -55,7 +55,6 @@ def test_should_include_content(browser, server, tmpdir):
         log = data["log"]
 
         content1 = log["entries"][0]["response"]["content"]
-        print(content1)
         assert content1["encoding"] == "base64"
         assert content1["mimeType"] == "text/html"
         s = base64.b64decode(content1["text"]).decode()
