Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(555)

Unified Diff: build/android/pylib/base/base_test_result.py

Issue 2541093004: [Android] Try harder to run every gtest within each try. (Closed)
Patch Set: Tweak NOTRUN handling. Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: build/android/pylib/base/base_test_result.py
diff --git a/build/android/pylib/base/base_test_result.py b/build/android/pylib/base/base_test_result.py
index 333211fa3219480837e4bbc38d5140e70ac57da7..dd309af8abdf098b57e6e2d531c0c8b8379760ce 100644
--- a/build/android/pylib/base/base_test_result.py
+++ b/build/android/pylib/base/base_test_result.py
@@ -9,18 +9,33 @@ import threading
class ResultType(object):
"""Class enumerating test types."""
+ # The test passed.
PASS = 'PASS'
+
+ # The test was intentionally skipped.
SKIP = 'SKIP'
+
+ # The test failed.
FAIL = 'FAIL'
+
+ # The test caused the containing process to crash.
CRASH = 'CRASH'
+
+ # The test timed out.
TIMEOUT = 'TIMEOUT'
+
+ # The test ran, but we couldn't determine what happened.
UNKNOWN = 'UNKNOWN'
+ # The test did not run.
+ NOTRUN = 'NOTRUN'
+
@staticmethod
def GetTypes():
"""Get a list of all test types."""
return [ResultType.PASS, ResultType.SKIP, ResultType.FAIL,
- ResultType.CRASH, ResultType.TIMEOUT, ResultType.UNKNOWN]
+ ResultType.CRASH, ResultType.TIMEOUT, ResultType.UNKNOWN,
+ ResultType.NOTRUN]
shenghuazhang 2016/12/01 19:46:40 Seems like the 'NOTRUN' ResultType isn't used by a
jbudorick 2016/12/02 01:45:21 NOTRUN was more functional in an earlier patchset;
class BaseTestResult(object):

Powered by Google App Engine
This is Rietveld 408576698