Skip to content
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

[WebDriver BiDi] addPreloadScript supports contexts #41932

Conversation

Lightning00Blade
Copy link
Contributor

@Lightning00Blade Lightning00Blade commented Sep 12, 2023

Change to BiDi spec at w3c/webdriver-bidi#517.

Closes #41974

webdriver/tests/bidi/script/add_preload_script/invalid.py Outdated Show resolved Hide resolved
webdriver/tests/bidi/script/add_preload_script/invalid.py Outdated Show resolved Hide resolved
webdriver/tests/support/fixtures_bidi.py Outdated Show resolved Hide resolved
webdriver/tests/support/fixtures_bidi.py Outdated Show resolved Hide resolved
@whimboo
Copy link
Contributor

whimboo commented Oct 10, 2023

Hi @Lightning00Blade, will you have a chance to continue on this PR?

Copy link
Contributor

@whimboo whimboo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just some comments, but otherwise the tests look great. Thanks!

@whimboo
Copy link
Contributor

whimboo commented Jan 8, 2024

@Lightning00Blade I think that this PR needs an admin merge.

@Lightning00Blade
Copy link
Contributor Author

CC @web-platform-tests/admins for admin merge request. As per the last comment, errors seem unrelated to this change.

@jgraham jgraham merged commit 1ac9693 into web-platform-tests:master Jan 12, 2024
40 of 43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add webdriver tests for "contexts" argument of "script.addPreloadScript"
9 participants