1546303 - Missing tests for Execute (Async) Script properties

Status: RESOLVED FIXED

(Testing :: geckodriver, enhancement, P1)

Description

[Andreas Tolfsen ❲:ato❳]

We are missing quite basic tests for interacting with properties
of various kinds (content attributes and IDL attributes).

Comment 1

[Andreas Tolfsen ❲:ato❳]

I’ve not found any irregularities in this area with Marionette’s
implementation, but I recently went looking for tests to verify
that everything worked as expected and couldn’t find any, so I wrote
some.

Comment 2

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

Don't we want to have them as wdspec tests? Or are those specific to Marionette client?

Comment 3

[Andreas Tolfsen ❲:ato❳]

Attachment: bug 1546303: webdriver: add WPT tests for properties in evaluated scripts;

This adds a basic set of tests for interacting with various kinds
of DOM properties from the WebDriver commands Execute Script and
Execute Async Script.

Comment 4

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

Actually the patch covers wdspec.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4a753a263c38
webdriver: add WPT tests for properties in evaluated scripts; r=whimboo

Comment 6

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/4a753a263c38

Comment 7

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

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

Comment 8

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

Upstream PR merged