Support running wpt with chaos mode

RESOLVED FIXED in Firefox 58

Status

Testing
web-platform-tests
RESOLVED FIXED
a month ago
a month ago

People

(Reporter: jgraham, Unassigned)

Tracking

Version 3
mozilla58
Points:
---

Firefox Tracking Flags

(firefox58 fixed)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

a month ago
Allow running wpt tests under chaos mode using a --chaos flag.
Comment hidden (mozreview-request)

Comment 2

a month ago
mozreview-review
Comment on attachment 8918821 [details]
Bug 1408977 - Allow running wpt test with chos mode using --chaos[=flag],

https://reviewboard.mozilla.org/r/189682/#review195120

This looks fine - thanks!

::: testing/web-platform/tests/tools/wptrunner/wptrunner/wptcommandline.py:203
(Diff revision 1)
>                               help="Disable reftest runner implemented inside Marionette")
>      gecko_group.add_argument("--reftest-screenshot", dest="reftest_screenshot", action="store",
>                               choices=["always", "fail", "unexpected"], default="unexpected",
>                               help="With --reftest-internal, when to take a screenshot")
> +    gecko_group.add_argument("--chaos", dest="chaos_mode_flags", action="store",
> +                             nargs="?", const=3, type=int,

Did you make the default 3 because that's what --verify uses? That's reasonable, but be aware that's (hopefully) a temporary condition. I hope to enable all chaos mode features for --verify (bug 1390884), eventually.
Attachment #8918821 - Flags: review?(gbrown) → review+
Comment hidden (mozreview-request)

Comment 4

a month ago
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/autoland/rev/917b66c1e16d
Allow running wpt test with chos mode using --chaos[=flag], r=gbrown
https://hg.mozilla.org/mozilla-central/rev/917b66c1e16d
Status: NEW → RESOLVED
Last Resolved: a month ago
status-firefox58: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.