-
Notifications
You must be signed in to change notification settings - Fork 179
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Slight improvements to e2e test setup #1560
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
bc80957
to
7a475a0
Compare
@dvoytenko Any objections to merging this? Helps me with improving the docs around testing (#1476). |
Hmm Firefox currently seems to be hanging at the end due to |
44ad9a8
to
9cbc89f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm Firefox currently seems to be hanging at the end due to
usePreventWindowUnload
, despite us usingenablePageDialogAccept
inbeforeAll
(which doespage.on('dialog', (dialog) => dialog.accept())
Those will be disablable with #1176
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with one issue
See #1476 for docs.
This PR mainly adds some fixtures and bug fixes to the e2e tests.