diff options
| author | Tristan Honscheid <honscheid@google.com> | 2022-08-04 15:01:02 -0600 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-08-05 21:48:50 +0000 |
| commit | 1e869d9431dbae9b0753bf506ff08425d0afe6c1 (patch) | |
| tree | f0e6cdde1d68f458fd26dda93ad9c7b0850ac26f /.gitlab-ci.yml | |
| parent | 8837bd3f14c0af1d221316bf1126ec22066fe054 (diff) | |
| download | chrome-ec-1e869d9431dbae9b0753bf506ff08425d0afe6c1.tar.gz | |
zephyr: tests: remove BUILD.py for Zephyr EC unit tests
We have completed transitioning tests to Twister and verified parity in
coverage reports between this and the previous version. This CL
removes the BUILD.py files from our tests that declare the zmake
projects and removes the call to `zmake -D test -a --no-rebuild` from
`zephyr/firmware-builder.py` The `EC_ZEPHYR_MERGED` metric is now
derived from Twister's coverage data instead. Made corresponding changes
to `.gitlab-ci.yml`
This CL was tested by comparing the total coverage and list of covered
files before and after the change.
BUG=b:240093007
BRANCH=None
TEST=zephyr/firmware-builder.py --code-coverage {build|test|bundle}
Signed-off-by: Tristan Honscheid <honscheid@google.com>
Change-Id: I865a22a44f56760790444499c7588f7aec299374
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3811956
Reviewed-by: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Yuval Peress <peress@google.com>
Diffstat (limited to '.gitlab-ci.yml')
| -rw-r--r-- | .gitlab-ci.yml | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index a3f8c4cfbc..77872288df 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -230,12 +230,11 @@ zephyr_coverage: stage: test needs: ["seed_cache"] script: - - zmake --zephyr-base "${ZEPHYR_BASE}" - --modules-dir "${MODULES_DIR}" -l DEBUG test - --coverage --host-tests-only + - ${EC_DIR}/twister --coverage --outdir "${EC_DIR}/twister-out/" + -p native_posix artifacts: paths: - - build/zephyr/all_tests.info + - ${EC_DIR}/twister-out/coverage.info expire_in: 1 week coverage: '/lines\.*: \d+\.\d+%/' @@ -259,7 +258,7 @@ merged_coverage: needs: ["ec_coverage", "zephyr_coverage", "zephyr_boards_coverage"] script: - lcov --rc lcov_branch_coverage=1 -o build/merged.info - -a build/coverage/lcov.info -a build/zephyr/all_tests.info + -a build/coverage/lcov.info -a "${EC_DIR}/twister-out/coverage.info" - lcov --rc lcov_branch_coverage=1 -o build/merged_no_zephyr.info -r build/merged.info "${ZEPHYR_BASE}/**" "${MODULES_DIR}/**" @@ -302,8 +301,8 @@ testall: stage: test needs: ["seed_cache"] script: - - zmake --zephyr-base "${ZEPHYR_BASE}" - --modules-dir "${MODULES_DIR}" -l DEBUG test --host-tests-only + - ${EC_DIR}/twister --coverage --outdir "${EC_DIR}/twister-out/" + -p native_posix twister_coverage: stage: test |