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

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

Issue 950893004: Add exit code for infra failures to testrunner script. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Made CL more general. BaseError now has is_infra_error property. Created 5 years, 10 months 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_error.py
diff --git a/build/android/pylib/base/base_error.py b/build/android/pylib/base/base_error.py
new file mode 100644
index 0000000000000000000000000000000000000000..31eaa54c446c52c589fd699b10e18a16a10caabc
--- /dev/null
+++ b/build/android/pylib/base/base_error.py
@@ -0,0 +1,16 @@
+# Copyright 2015 The Chromium Authors. All rights reserved.
jbudorick 2015/02/23 22:13:00 changed my mind, put this in pylib/utils/
jbudorick 2015/02/23 22:16:03 (reasoning here: we're going to have to eventually
mikecase (-- gone --) 2015/02/23 22:29:27 Done.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+
+class BaseError(Exception):
+ """Base error for all test runner errors."""
+
+ def __init__(self, message, is_infra_error=False):
+ super(BaseError, self).__init__(message)
+ self._is_infra_error = is_infra_error
+
+ @property
+ def is_infra_error(self):
+ """Property to indicate if error was caused by an infrastructure issue."""
+ return self._is_infra_error
« no previous file with comments | « no previous file | build/android/pylib/constants.py » ('j') | build/android/pylib/remote/device/remote_device_environment.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698