diff options
author | Aaron Massey <aaronmassey@google.com> | 2022-07-07 17:43:17 -0600 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-07-08 18:40:31 +0000 |
commit | a103b8df46557cda86911ac36a084f08e600ad02 (patch) | |
tree | 13f52b0ab2699594947f31981bc44474faa8d991 | |
parent | 42692e6c86c6258b921d1a22a146a4b7f9d28345 (diff) | |
download | chrome-ec-a103b8df46557cda86911ac36a084f08e600ad02.tar.gz |
util: Correct shuffle_test false positive errors
The shuffle_test.sh Zephyr test utility script greps for a user provided
failure string. When the script doesn't find such failure string via
grep it erroneously reports this as "Error/timeout".
Make the utility script only report "Error/timeout" after finding the
user provided failure string.
BRANCH=none
BUG=b:236726670
TEST=shuffle tests with and without failures
Signed-off-by: Aaron Massey <aaronmassey@google.com>
Change-Id: I989bc89cf8877e29aafea1a0f5c7f279ba1bd356
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3751777
Reviewed-by: Yuval Peress <peress@google.com>
-rwxr-xr-x | util/shuffle_test.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/util/shuffle_test.sh b/util/shuffle_test.sh index 69a6f694f3..662aaa7bce 100755 --- a/util/shuffle_test.sh +++ b/util/shuffle_test.sh @@ -28,7 +28,7 @@ while [ "${loop_count}" -gt 0 ]; do if [ "${status}" -eq 124 ]; then echo " Timeout" result="timed-out" - elif [ "${status}" -ne 0 ]; then + elif [ "${status}" -eq 0 ]; then echo " Error/timeout" result="exit-code-${status}" fi |