-
Notifications
You must be signed in to change notification settings - Fork 932
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
chore(roll): roll to Playwright 1.41.0-beta-1705101589000 #2225
Conversation
e057914
to
ac3211e
Compare
040dc89
to
a2ec2bd
Compare
a044c0d
to
9df606f
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.
I didn't look into most of the tests since a bunch of them just moved around.
from tests.utils import must | ||
|
||
|
||
def test_context_unroute_all_removes_all_handlers( |
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.
What about other tests for async version of unrouteAll?
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.
they are in tests/async/test_unroute_behavior.py.
Fixes #2176