Closed Bug 1566198 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 17645 - [tools] Remove support for `support` filename flag

Categories

(Core :: DOM: Core & HTML, task, P4)

Product:
Core
Component:
DOM: Core & HTML
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla70
Tracking Flags:
Tracking Status
firefox70 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 17645 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/17645
Details from upstream follow.

Mike Pennisi <mike@mikepennisi.com> wrote:

[tools] Remove support for support filename flag

In terms of file interpretation by the WPT infrastructure, naming a file
with the -support flag is equivalent to placing it in a directory
named support, resources, or tools. The ability to express the
same intention with different mechanisms allows for variation that adds
no value and needless increases the complexity of the infrastructure and
its documentation. Further, the character sequence of -support is
prone to accidental opt-in: a number of tests in this suite currently
include that sequence and are thus not executed in automation.

Remove support for the flag.

Resolves gh-15749

Component: web-platform-tests → DOM: Core & HTML
Product: Testing → Core
Type: -- → task
Ran 25 tests and 21423 subtests OK : 24 PASS : 21419[android-em-7.0-x86_64-debug,android-em-7.0-x86_64-opt], 21421[linux32-shippable-opt,linux64-asan-opt,linux64-debug,linux64-opt,linux64-qr-debug,linux64-qr-opt,linux64-shippable-opt,linux64-shippable-qr-opt,windows10-64-debug,windows10-64-opt,windows10-64-qr-debug,windows10-64-qr-opt,windows10-64-shippable-opt,windows10-64-shippable-qr-opt,windows7-32-debug,windows7-32-opt,windows7-32-shippable-opt] FAIL : 2[linux32-shippable-opt,linux64-asan-opt,linux64-debug,linux64-opt,linux64-qr-debug,linux64-qr-opt,linux64-shippable-opt,linux64-shippable-qr-opt,windows10-64-debug,windows10-64-opt,windows10-64-qr-debug,windows10-64-qr-opt,windows10-64-shippable-opt,windows10-64-shippable-qr-opt,windows7-32-debug,windows7-32-opt,windows7-32-shippable-opt], 4[android-em-7.0-x86_64-debug,android-em-7.0-x86_64-opt] Tests that are disabled for instability: /html/syntax/parsing/DOMContentLoaded-defer.html
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/e6ae78f494c3 [wpt PR 17645] - [tools] Remove support for `support` filename flag, a=testonly

https://hg.mozilla.org/mozilla-central/rev/e6ae78f494c3

Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox70: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.