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

Unified Diff: build/android/buildbot/bb_device_steps.py

Issue 235573010: Add telemetry tests to Android tester. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 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 | « no previous file | build/android/buildbot/bb_run_bot.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/buildbot/bb_device_steps.py
diff --git a/build/android/buildbot/bb_device_steps.py b/build/android/buildbot/bb_device_steps.py
index c6e5f268128b24a8b473683e95a5ab05dd204ac4..fda6ee2649852bd0a5a46397074c96b4edc892cb 100755
--- a/build/android/buildbot/bb_device_steps.py
+++ b/build/android/buildbot/bb_device_steps.py
@@ -77,8 +77,9 @@ INSTRUMENTATION_TESTS = dict((suite.name, suite) for suite in [
'webview:android_webview/test/data/device_files'),
])
-VALID_TESTS = set(['chromedriver', 'gpu', 'ui', 'unit', 'webkit',
- 'webkit_layout', 'webrtc_chromium', 'webrtc_native'])
+VALID_TESTS = set(['chromedriver', 'gpu', 'ui', 'unit', 'telemetry_tests',
tonyg 2014/04/12 01:37:29 Looks like these were alphabetical. Should we main
navabi 2014/04/12 01:55:01 Yes we should. Will fix. And I realized I should s
+ 'webkit', 'webkit_layout', 'webrtc_chromium',
+ 'webrtc_native'])
RunCmd = bb_utils.RunCmd
@@ -164,6 +165,19 @@ def RunChromeDriverTests(options):
'--update-log'])
+def RunTelemetryTests(_):
+ """Runs the telemetry unit tests and perf unit tests.
+
+ Args:
+ options: options object.
+ """
+ args = ['--browser', 'android-chromium-testshell']
+ bb_annotations.PrintNamedStep('telemetry_unittests')
+ RunCmd(['tools/telemetry/run_tests'] + args)
+ bb_annotations.PrintNamedStep('telemetry_perf_unittests')
+ RunCmd(['tools/perf/run_tests'] + args)
+
+
def InstallApk(options, test, print_step=False):
"""Install an apk to all phones.
@@ -483,6 +497,7 @@ def GetTestStepCmds():
('gpu', RunGPUTests),
('unit', RunUnitTests),
('ui', RunInstrumentationTests),
+ ('telemetry_tests', RunTelemetryTests),
tonyg 2014/04/12 01:37:29 ditto
('webkit', RunWebkitTests),
('webkit_layout', RunWebkitLayoutTests),
('webrtc_chromium', RunWebRTCChromiumTests),
« no previous file with comments | « no previous file | build/android/buildbot/bb_run_bot.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698