Skip to content

Commit 7dc8b32

Browse files
authored
test: unflake dialog inline script test (#2934)
1 parent 8ad65d4 commit 7dc8b32

File tree

1 file changed

+11
-9
lines changed

1 file changed

+11
-9
lines changed

tests/sync/test_browsercontext_events.py

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -172,23 +172,25 @@ def test_dialog_event_should_work_with_inline_script_tag(
172172
) -> None:
173173
def handle_route(request: TestServerRequest) -> None:
174174
request.setHeader("content-type", "text/html")
175-
request.write(b"""<script>window.result = prompt('hey?')</script>""")
175+
request.write(b"<script>window.result = prompt('hey?')</script>")
176176
request.finish()
177177

178178
server.set_route("/popup.html", handle_route)
179179
page.goto(server.EMPTY_PAGE)
180180
page.set_content("<a href='popup.html' target=_blank>Click me</a>")
181181

182-
def handle_dialog(dialog: Dialog) -> None:
183-
assert dialog.message == "hey?"
184-
assert dialog.page == popup
185-
dialog.accept("hello")
182+
with (
183+
page.context.expect_event("dialog") as dialog_info,
184+
page.expect_popup() as popup_info,
185+
):
186+
page.click("a")
186187

187-
page.context.on("dialog", handle_dialog)
188+
dialog: Dialog = dialog_info.value
189+
popup: Page = popup_info.value
188190

189-
with page.expect_popup() as popup_info:
190-
page.click("a")
191-
popup = popup_info.value
191+
assert dialog.message == "hey?"
192+
assert dialog.page == popup
193+
dialog.accept("hello")
192194
assert popup.evaluate("window.result") == "hello"
193195

194196

0 commit comments

Comments
 (0)