1822779 - [wdspec] Remove tentative suffix from "script.addPreloadScript" and "script.removePreloadScript" tests

Status: RESOLVED FIXED

(Remote Protocol :: WebDriver BiDi, task, P1)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

We are going to change the script.addPreloadScript so that it no longer takes an expression but a function as argument. The relevant work is covered by the following Webdriver BiDi Pull:

https://github.com/w3c/webdriver-bidi/pull/381

Depending on when it lands we should do a quick follow-up and update our implementation and tests. So it will be a M6 or M7 related bug.

Comment 1

[Alexandra Borovova [:Sasha]]

Actually, our implementation already takes a function as an argument. But as soon as the spec PR is merged, we can remove tentative postfix from wdspec test file names.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Good point. I forgot that and should have checked. Nevertheless that gives me lesser worry regarding backward incompatibility. I'm going to update the summary accordingly.

Comment 3

[Alexandra Borovova [:Sasha]]

Attachment: Bug 1822779 - [wdspec] Remove tentative suffix from "script.addPreloadScript" and "script.removePreloadScript" tests.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8f3e5ea7e15f
[wdspec] Remove tentative suffix from "script.addPreloadScript" and "script.removePreloadScript" tests. r=webdriver-reviewers,whimboo

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/39335 for changes under testing/web-platform/tests

Comment 6

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/8f3e5ea7e15f

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot