Closed
Bug 1452672
Opened 6 years ago
Closed 6 years ago
[wpt-sync] Sync PR 10381 - Support adding Idlharness dependencies
Categories
(Testing :: web-platform-tests, enhancement, P4)
Testing
web-platform-tests
Tracking
(firefox62 fixed)
RESOLVED
FIXED
mozilla62
Tracking | Status | |
---|---|---|
firefox62 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 10381 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/w3c/web-platform-tests/pull/10381 Details from upstream follow. Luke Bjerring <lukebjerring@users.noreply.github.com> wrote: > Support adding Idlharness dependencies > > This is a request for feedback on a potential solution to https://github.com/w3c/web-platform-tests/issues/10347 > > The PR adds a new `idlharness.js` api: `add_dependency_idls(raw_idls, options)` > > Usage examples are in the couple of modified tests. > > Note that the `options` object was needed for adding deps in some cases, because some interfaces are re-declared in both `html` and `dedicated-workers.idl`, e.g. `WorkerNavigator`. > See https://github.com/w3c/web-platform-tests/blob/master/budget-api/interfaces.any.js for example.
Reporter | ||
Updated•6 years ago
|
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Reporter | ||
Updated•6 years ago
|
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Reporter | ||
Comment 1•6 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=6df33bc6f4e70954e595d4fed47a0771390ad074
Reporter | ||
Comment 2•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=fe358bb30707b090d901e32bc8e8f8c5146294f4
Reporter | ||
Comment 3•6 years ago
|
||
Ran 123 tests and 17864[linux32-debug], 17865[linux32-opt,linux64-debug,linux64-opt] subtests OK : 120[linux32-opt,linux64-debug,linux64-opt], 119[linux32-debug] PASS : 15946 FAIL : 1918[linux32-debug], 1919[linux32-opt,linux64-debug,linux64-opt] TIMEOUT: 3[linux32-opt,linux64-debug,linux64-opt], 4[linux32-debug] Existing tests that now have a worse result (e.g. they used to PASS and now FAIL): /html/dom/interfaces.worker.html: TIMEOUT[linux32-debug] /web-animations/interfaces/Animation/idlharness.html Animation interface: new Animation() must inherit property "oncancel" with the proper type: FAIL Animation interface: new Animation() must inherit property "onfinish" with the proper type: FAIL
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/8e5565a60bc7 [wpt PR 10381] - Support adding Idlharness dependencies, a=testonly https://hg.mozilla.org/integration/mozilla-inbound/rev/1e0bf152d713 [wpt PR 10381] - Update wpt metadata, a=testonly
Comment 5•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8e5565a60bc7 https://hg.mozilla.org/mozilla-central/rev/1e0bf152d713
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox62:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in
before you can comment on or make changes to this bug.
Description
•