Missing tests for Execute (Async) Script properties

RESOLVED FIXED in Firefox 68

Status

enhancement
P1
normal
RESOLVED FIXED
a month ago
27 days ago

People

(Reporter: ato, Assigned: ato)

Tracking

Version 3
mozilla68
Points:
---

Firefox Tracking Flags

(firefox68 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

a month ago

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

Assignee

Comment 1

a month ago

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.

Assignee: nobody → ato
Status: NEW → ASSIGNED
Priority: -- → P1

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

Assignee

Comment 3

a month ago

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.

Actually the patch covers wdspec.

Component: Marionette → geckodriver
Assignee

Updated

a month ago
Type: defect → enhancement

Comment 5

a month ago
Pushed by atolfsen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4a753a263c38
webdriver: add WPT tests for properties in evaluated scripts; r=whimboo

Comment 6

a month ago
bugherder
Status: ASSIGNED → RESOLVED
Last Resolved: a month ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/16503 for changes under testing/web-platform/tests
Upstream PR merged
You need to log in before you can comment on or make changes to this bug.