hack: workaround JSHandle generated types

rwoll · rwoll · commit 7935cd6e4b17 · 2020-08-03T16:53:29.000-07:00
diff --git a/playwright/js_handle.py b/playwright/js_handle.py
@@ -93,6 +93,8 @@ async def jsonValue(self) -> Any:
 
 
 def serialize_value(value: Any, handles: List[JSHandle], depth: int) -> Any:
+    if type(value).__name__ == "JSHandle" and not isinstance(value, JSHandle):
+        value = value._impl_obj
     if isinstance(value, JSHandle):
         h = len(handles)
         handles.append(value._channel)
diff --git a/tests/async/test_dispatch_event.py b/tests/async/test_dispatch_event.py
@@ -137,7 +137,6 @@ async def test_should_be_atomic(selectors, page, utils):
     assert await page.evaluate("() => window._clicked")
 
 
-@pytest.mark.skip("TODO: Determine why there is a failure.")
 @pytest.mark.skip_browser("webkit")
 async def test_should_dispatch_drag_drop_events(page, server):
     await page.goto(server.PREFIX + "/drag-n-drop.html")
@@ -151,7 +150,6 @@ async def test_should_dispatch_drag_drop_events(page, server):
     )
 
 
-@pytest.mark.skip("TODO: Determine why there is a failure.")
 @pytest.mark.skip_browser("webkit")
 async def test_should_dispatch_drag_and_drop_events_element_handle(page, server):
     await page.goto(server.PREFIX + "/drag-n-drop.html")
