Skip to content

Conversation

@tanuja-intellicore
Copy link

@tanuja-intellicore tanuja-intellicore commented May 19, 2025

This pr is for fixing the below issue. I have validated this fix and its working as expected

Error: There is no handler registered for beforeunload popup displayed on the page XXXXXX
This might interfere with your test flow. You can use Taiko's beforeunload API to handle this popup.
Please visit https://docs.taiko.dev/#beforeunload for more details
at Chrome. (/Users/19596/suse-run-final/ng-acceptance-tests/node_modules/taiko/lib/handlers/pageHandler.js:81:13)
at Chrome.emit (node:events:519:35)
at Chrome.emit (node:domain:489:12)
at Chrome._handleMessage (/Users/19596/suse-run-final/ng-acceptance-tests/node_modules/chrome-remote-interface/lib/chrome.js:287:18)
at WebSocket. (/Users/19596/suse-run-final/ng-acceptance-tests/node_modules/chrome-remote-interface/lib/chrome.js:240:22)
at WebSocket.emit (node:events:507:28)
at WebSocket.emit (node:domain:489:12)
at Receiver.receiverOnMessage (/Users/19596/suse-run-final/ng-acceptance-tests/node_modules/chrome-remote-interface/node_modules/ws/lib/websocket.js:1070:20)
at Receiver.emit (node:events:507:28)
at Receiver.emit (node:domain:489:12)
at Receiver.dataMessage (/Users/19596/suse-run-final/ng-acceptance-tests/node_modules/chrome-remote-interface/node_modules/ws/lib/receiver.js:517:14)
at Receiver.getData (/Users/19596/suse-run-final/ng-acceptance-tests/node_modules/chrome-remote-interface/node_modules/ws/lib/receiver.js:435:17)
at Receiver.startLoop (/Users/19596/suse-run-final/ng-acceptance-tests/node_modules/chrome-remote-interface/node_modules/ws/lib/receiver.js:143:22)
at Receiver._write (/Users/19596/suse-run-final/ng-acceptance-tests/node_modules/chrome-remote-interface/node_modules/ws/lib/receiver.js:78:10)
at writeOrBuffer (node:internal/streams/writable:570:12)
at _write (node:internal/streams/writable:499:10)

@getgauge/taiko-contributors

@tanuja-intellicore
Copy link
Author

@getgauge/taiko-contributors Please check the pr

Signed-off-by: Tanuja Viswanadhapalli <tanuja.viswanadhapalli@omnicell.com>
Signed-off-by: Tanuja Viswanadhapalli <tanuja.viswanadhapalli@omnicell.com>
Signed-off-by: Tanuja Viswanadhapalli <tanuja.viswanadhapalli@omnicell.com>
Signed-off-by: Tanuja Viswanadhapalli <tanuja.viswanadhapalli@omnicell.com>
@zabil
Copy link
Member

zabil commented May 21, 2025

@tanuja-intellicore thanks for the PR can you look at the test failures?

https://github.com/getgauge/taiko/actions/runs/15137304825/job/42613244355?pr=2753

@tanuja-intellicore
Copy link
Author

I have seen the failures and most of them are failing with beforeunload issue. I have raised the pr to accept the beforeunload after closing the browser

@winst0niuss
Copy link
Contributor

@tanuja-intellicore, @zabil
This PR can be closed. The fixes were completed in #2763 and merged into v1.4.7140.0.7339.82.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants