Closed Bug 1481911 Opened Last year Closed Last year

[wpt-sync] Sync PR 12360 - [wptrunner] Make test skipping explicit

Categories

(Testing :: web-platform-tests, enhancement, P4)

enhancement

Tracking

(firefox63 fixed)

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: wptsync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Mike Pennisi <mike@mikepennisi.com> wrote:
>  [wptrunner] Make test skipping explicit
>  
>  Explicitly report on skipped "jsshell" tests during execution. This
>  gives users an indication that the behavior is intentional and promotes
>  parity with the output generated by the `--list-tests` flag of the `wpt
>  run.  command.
>  
>  ---
>  
>  @jgraham @Ms2ger This removes a comment that states "we silently skip them," so
>  I expect there is a good reason they aren't being documented as "SKIP". In
>  that case, I can look in to updating the `--list-tests` feature to correctly
>  support the new feature. But could we also extend this comment to explain why
>  this needs to be done "silently"?
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/811e84ef1cd5
[wpt PR 12360] - [wptrunner] Make test skipping explicit, a=testonly
Result changes from PR not available.
https://hg.mozilla.org/mozilla-central/rev/811e84ef1cd5
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.