diff options
| author | Jeremy Bettis <jbettis@google.com> | 2021-08-23 16:39:10 -0600 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2021-08-24 22:18:34 +0000 |
| commit | 439f32152d159838e181216e272411798a5e55fa (patch) | |
| tree | 41a1459c336d7659c9d2546ed1d9104491c5c6b6 | |
| parent | 851179f8e5da5507fa16c3faf9f1b8a54eff928c (diff) | |
| download | chrome-ec-439f32152d159838e181216e272411798a5e55fa.tar.gz | |
ec: Run tests in normal and coverage mode
As tests behave differently with coverage enabled, run the tests both
with and without coverage in the CQ.
BUG=None
TEST=./zephyr/firmware_builder.py --metrics=/tmp/metrics test ; echo $?
BRANCH=none
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Change-Id: I8188542ceaf3974f9debd8b574895313941a75e5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3115430
Commit-Queue: Jeremy Bettis <jbettis@chromium.org>
Tested-by: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
| -rwxr-xr-x | zephyr/firmware_builder.py | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/zephyr/firmware_builder.py b/zephyr/firmware_builder.py index 13f2d141e0..446ad20bee 100755 --- a/zephyr/firmware_builder.py +++ b/zephyr/firmware_builder.py @@ -137,14 +137,13 @@ def test(opts): # proceeding. subprocess.run([zephyr_dir / 'zmake' / 'run_tests.sh'], check=True) - if opts.code_coverage: - platform_ec = zephyr_dir.parent - build_dir = platform_ec / 'build/zephyr-coverage' - return subprocess.run( - ['zmake', '-D', 'coverage', build_dir], cwd=platform_ec).returncode - subprocess.run(['zmake', '-D', 'testall'], check=True) - return 0 + + # Run the test with coverage also, as sometimes they behave differently. + platform_ec = zephyr_dir.parent + build_dir = platform_ec / 'build/zephyr-coverage' + return subprocess.run( + ['zmake', '-D', 'coverage', build_dir], cwd=platform_ec).returncode def main(args): |