commit 64131ede3235dfcd468b89785833ccb6fa344660
author Kazuhiro Inaba <kinaba@chromium.org> Tue Apr 04 20:47:38 2017 +0900
committer Kazuhiro Inaba <kinaba@chromium.org> Wed Apr 05 12:16:59 2017 +0000
tree 9df52b4f2f6bbe9d193ba7d72c28b78a13a76962
parent a64d4b08e2bcbea6213288474b2f6ebdf35341dd

cheets_CTS_N: Don't throw an error when all failures are waived.

When (passed=0, failed=waived>0) it should be regarded as success.

BUG=None
TEST=cheets_CTS_N.CtsSystemUiHostTestCases

Change-Id: I0192e5012280ae36c2289aa155e99e2d346fd0f9
Reviewed-on: https://chromium-review.googlesource.com/467187
Tested-by: Kazuhiro Inaba <kinaba@chromium.org>
Trybot-Ready: Kazuhiro Inaba <kinaba@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
(cherry picked from commit 19bfd2ef42eb77d0bd29ae1ac2b0203eb3110eb1)
Reviewed-on: https://chromium-review.googlesource.com/468494
Reviewed-by: Kazuhiro Inaba <kinaba@chromium.org>

server/site_tests/cheets_CTS_N/cheets_CTS_N.py

diff --git a/server/site_tests/cheets_CTS_N/cheets_CTS_N.py b/server/site_tests/cheets_CTS_N/cheets_CTS_N.py
index f4744ea..141cf3c 100644
--- a/server/site_tests/cheets_CTS_N/cheets_CTS_N.py
+++ b/server/site_tests/cheets_CTS_N/cheets_CTS_N.py
@@ -418,7 +418,7 @@
             # The DUT has rebooted at this point and is in a clean state.
 
         # Final classification of test results.
-        if total_passed == 0 or failed > waived:
+        if total_passed + waived == 0 or failed > waived:
             raise error.TestFail(
                 'Failed: after %d retries giving up. '
                 'passed=%d, failed=%d, notexecuted=%d, waived=%d. %s' %