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

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

Issue 16299003: Revert 203587 "Move CheckInstall to device status and fail on lo..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 7 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 | trunk/src/build/android/device_status_check.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/build/android/buildbot/bb_device_steps.py
===================================================================
--- trunk/src/build/android/buildbot/bb_device_steps.py (revision 203593)
+++ trunk/src/build/android/buildbot/bb_device_steps.py (working copy)
@@ -162,6 +162,15 @@
RunCmd(['chrome/test/chromedriver/run_buildbot_steps.py',
'--android-package=%s' % constants.CHROMIUM_TEST_SHELL_PACKAGE])
+
+def CheckInstall():
+ """Build bot step to see if adb install works on attached devices. """
+ buildbot_report.PrintNamedStep('Check device install')
+ # This step checks if apks can be installed on the devices.
+ args = ['--apk', 'build/android/CheckInstallApk-debug.apk']
+ RunCmd(['build/android/adb_install_apk.py'] + args, halt_on_failure=True)
+
+
def InstallApk(options, test, print_step=False):
"""Install an apk to all phones.
@@ -277,6 +286,9 @@
target = options.factory_properties.get('target', 'Debug')
RunCmd(['build/android/provision_devices.py', '-t', target])
+ # Check to see if devices can install apks.
+ CheckInstall()
+
if options.install:
test_obj = INSTRUMENTATION_TESTS[options.install]
InstallApk(options, test_obj, print_step=True)
« no previous file with comments | « no previous file | trunk/src/build/android/device_status_check.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698