commit | c164a706e74cffa2c0f15ecfa78b787cda26dc3f | [log] [tgz] |
---|---|---|
author | Garrett Beaty <gbeaty@chromium.org> | Thu Nov 05 23:40:14 2020 |
committer | LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> | Thu Nov 05 23:40:14 2020 |
tree | 8939b386f2aa331d8e85fdaf4ea7366ca51b5a8a | |
parent | 34ead5d40ba33beb2909d9085222c7491e3a96c0 [diff] |
Remove the no-op steps from MockTest. The no-op steps are only needed for one test, so rather than force all users of Mocktest to provide the api, which isn't suited for being stored in a spec in an upcoming change, just have the one test create a wrapper that creates the no-op steps. Bug: 1128746 Change-Id: I8a7098d85046527eecca6c6011eb7ad7fa81c04b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/2472141 Commit-Queue: Garrett Beaty <gbeaty@chromium.org> Auto-Submit: Garrett Beaty <gbeaty@chromium.org> Reviewed-by: Stephen Martinis <martiniss@chromium.org>
If you're here to make a change to ‘recipes’ (the code located in scripts/slave/recipes*
), please take a look at the README for more information pertaining to recipes.
The preferred style is PEP8 with two-space indent; that is, the Chromium Python style. Functions use lowercase_with_underscores
, with the exception of the special functions RunSteps
and RunTests
in recipes. Use yapf (git cl format --no-clang-format
) to autoformat new code.