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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: build/android/test_runner.py

Issue 1003463002: Add json output to Junit tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed jbudorick's comments. Created 5 years, 9 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
« no previous file with comments | « build/android/pylib/results/json_results.py ('k') | testing/android/junit/BUILD.gn » ('j') | testing/android/junit/java/src/org/chromium/testing/local/JsonLogger.java » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/test_runner.py
diff --git a/build/android/test_runner.py b/build/android/test_runner.py
index 368ac06aff109931487a0f4b1497f16e292a2f4c..87f4cecabd12bc3010bae8e26f3ca1ef22c1118d 100755
--- a/build/android/test_runner.py
+++ b/build/android/test_runner.py
@@ -768,7 +768,13 @@ def _RunUIAutomatorTests(args, devices):
def _RunJUnitTests(args):
"""Subcommand of RunTestsCommand which runs junit tests."""
runner_factory, tests = junit_setup.Setup(args)
- _, exit_code = junit_dispatcher.RunTests(tests, runner_factory)
+ results, exit_code = junit_dispatcher.RunTests(tests, runner_factory)
kzaikin 2015/03/19 09:12:11 In fact your dispatcher returns bool as second tup
mikecase (-- gone --) 2015/03/23 20:38:44 Good catch. Fixed
+
+ report_results.LogFull(
+ results=results,
+ test_type='JUnit',
+ test_package=args.test_suite)
+
return exit_code
« no previous file with comments | « build/android/pylib/results/json_results.py ('k') | testing/android/junit/BUILD.gn » ('j') | testing/android/junit/java/src/org/chromium/testing/local/JsonLogger.java » ('J')

Powered by Google App Engine
This is Rietveld 408576698