Closed
Bug 1515890
Opened 5 years ago
Closed 5 years ago
[wpt-sync] Sync PR 14625 - Make resources tests work better with positional arguments
Categories
(Testing :: web-platform-tests, enhancement, P4)
Testing
web-platform-tests
Tracking
(firefox67 fixed)
RESOLVED
FIXED
mozilla67
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 14625 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/web-platform-tests/wpt/pull/14625 Details from upstream follow. James Graham <james@hoppipolla.co.uk> wrote: > Make resources tests work better with positional arguments > > Previously we did two slightly broken things; we applied the > supplied arguments before the mandatory arguments, which prevented > passing -- to seperate named arguments from positional arguments, and > we depending on explictly passing in tests/ to avoid collecting HTML > files in the resources/test/ directory as tests. > > To fix the latter we restrict the paths that can form tests in the > collector rather than relying on specific command line arguments.
Assignee | ||
Comment 1•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=1a2afb19a0d09339e5e742fd73a91c4ac0c5f864
Assignee | ||
Comment 2•5 years ago
|
||
The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/5e9d910d33ec [wpt PR 14625] - Make resources tests work better with positional arguments, a=testonly
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/mozilla-inbound/rev/a3338f338d40 [wpt PR 14625] - Make resources tests work better with positional arguments, a=testonly
Comment 5•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in
before you can comment on or make changes to this bug.
Description
•