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

Unified Diff: build/android/pylib/browsertests/dispatch.py

Issue 18323020: Updates the test runner script exit codes (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 6 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/browsertests/dispatch.py
diff --git a/build/android/pylib/browsertests/dispatch.py b/build/android/pylib/browsertests/dispatch.py
index 8457cf2f618986b8f1bad9770d3273bf9eabf35a..5daba5cc379853289bcf1f7d1ae6756923e9adc6 100644
--- a/build/android/pylib/browsertests/dispatch.py
+++ b/build/android/pylib/browsertests/dispatch.py
@@ -23,7 +23,15 @@ from common import unittest_util
def Dispatch(options):
frankf 2013/07/03 21:26:37 Future TODO: There's a lot of duplicaiton in dispa
gkanwar 2013/07/03 23:15:18 Agreed. Should I file a bug for this perhaps?
- """Dispatches all content_browsertests."""
+ """Dispatches all content_browsertests.
+
+ Args:
+ options: optparse.Options object containing command-line options
+ Returns:
+ base_test_result.TestRunResults object with the results of running the tests
+ Raises:
+ Exception: Failed to reset the test server port.
+ """
attached_devices = []
if options.test_device:
@@ -88,7 +96,7 @@ def Dispatch(options):
flakiness_server=options.flakiness_dashboard_server)
report_results.PrintAnnotation(test_results)
- return len(test_results.GetNotPass())
+ return test_results
def _FilterTests(all_enabled_tests):

Powered by Google App Engine
This is Rietveld 408576698