diff options
author | Tristan Honscheid <honscheid@google.com> | 2023-01-19 15:55:39 -0700 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-01-23 19:42:38 +0000 |
commit | 67830158094d9a67e28a3bd196b51ec0d0617c22 (patch) | |
tree | f03e045c808c9c31254e84966af48807a2abab25 | |
parent | 9ae74605306a5d5cb8e6f209fe62e09208e4577f (diff) | |
download | chrome-ec-67830158094d9a67e28a3bd196b51ec0d0617c22.tar.gz |
docs: Update coverage requirement to 90%
Update the firmware testing requirements doc to reflect the new
90% threshold.
BUG=None
BRANCH=None
TEST=None, ironically
Change-Id: I35f90b1592db2203d3c6f7ae05071fdd23ad0695
Signed-off-by: Tristan Honscheid <honscheid@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4182781
Reviewed-by: Yuval Peress <peress@google.com>
Reviewed-by: Keith Short <keithshort@chromium.org>
-rw-r--r-- | docs/chromeos-ec-firmware-test-requirements.md | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/docs/chromeos-ec-firmware-test-requirements.md b/docs/chromeos-ec-firmware-test-requirements.md index 0a173769b3..2e8b461eb6 100644 --- a/docs/chromeos-ec-firmware-test-requirements.md +++ b/docs/chromeos-ec-firmware-test-requirements.md @@ -5,14 +5,13 @@ ## Overview The ChromeOS EC firmware requires that all C source and header file code changes -include tests covering at least 80% of any new or changed lines of code. +include tests covering at least 90% of any new or changed lines of code. A +non-blocking warning comment is posted on CLs that do not exceed 95% coverage +This change is in effect as of 2023-01-19. The Gerrit commit-queue enforces the coverage requirement with the `firmware-zephyr-cov-cq` builder. -The Google EC firmware team plans to gradually increase the code coverage -requirement from 80% to 95% for new or changed lines of code. - ## Rationale Starting 2H’22, all new Chromebook projects use the Zephyr RTOS as the |