Revision 7e214e852492e619254c14c351a85ac97661d847 authored by Robert Ma on 19 March 2018, 15:34:37 UTC, committed by Robert Ma on 20 March 2018, 20:32:14 UTC
Similar to #10030 (which was for `wpt run`), this commit makes `wpt
check-stability` return a non-zero exit code when no tests run.

Also fix some code smells:

* `run` was redefined by `do_delayed_imports` to a different function.
  Rename the other `run` method in this module to `setup_and_run`.
* `retcode` was unnecessarily defined in the top scope. Remove it to
  avoid redefinition.
* `global logger` and the call to `get_parser` were extraneous in `run`
  (now `setup_and_run`) and hence are removed.
1 parent 4d10320
History

back to top