Open Bug 1515897 Opened 5 years ago Updated 2 years ago

[wpt-sync] Sync PR 14630 - Plumb empty test_list when using --affected

Categories

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

Product:
Testing
Component:
web-platform-tests
Type:
enhancement

Tracking

(Not tracked)

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream error]) 

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

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

Luke Bjerring <lukebjerring@google.com> wrote:
>  Plumb empty test_list when using --affected
>  
>  This changes the filtering setup for `TestLoader` to treat an empty `include` list differently than a null one. Also changes the plumbing for wptcommandline to forward an empty `test_list` when `--affected` was specified, and to not pop `--affected` from the args to be able to make this check later.
>  The error-code of the run is also affected, with a value of `0` for no executed tests.
>  Executing without specifying a whitelist of test paths has the same behaviour as before (runs the full test suite as per the manifest.)
>  The effect on the output is that we still produce a report, which has `results: []`, rather than no report file, for the case that `--affected` produces no test results.
>  
>  Fixes https://github.com/web-platform-tests/wpt/issues/14471
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.