tradefed_test: relax the condition to alert the result as inaccurate.
If the tradefed command itself is recognizing the inaccuracy,
consider it as ok.
BUG=b:115944638
TEST=some random cheets_CTS tests
Change-Id: Ifae3044a75cb94b1666a2b1652262c8144cce638
Reviewed-on: https://chromium-review.googlesource.com/1229814
Tested-by: Kazuhiro Inaba <kinaba@chromium.org>
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
diff --git a/server/cros/tradefed_test.py b/server/cros/tradefed_test.py
index 6ac4a0d..ec687b7 100644
--- a/server/cros/tradefed_test.py
+++ b/server/cros/tradefed_test.py
@@ -1098,7 +1098,6 @@
# minutes).
waived_tests, acc = self._run_and_parse_tradefed(
commands)
- accurate.append(acc)
result = self._run_tradefed_list_results()
if not result:
logging.error('Did not find any test results. Retry.')
@@ -1108,6 +1107,10 @@
waived = len(waived_tests)
last_session_id, passed, failed, all_done = result
+ # If the result is |acc|urate according to the log, or the
+ # inaccuracy is recognized by tradefed (not all_done), then
+ # it is fine.
+ accurate.append(acc or not all_done)
if failed < waived:
logging.error(
'Error: Internal waiver bookkeeping has become '