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

Unified Diff: scripts/slave/recipe_modules/chromium_android/api.py

Issue 979013002: Add authorize_adb_keys step that will add host key to device. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Add call to authorize script and add expectations. 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: scripts/slave/recipe_modules/chromium_android/api.py
diff --git a/scripts/slave/recipe_modules/chromium_android/api.py b/scripts/slave/recipe_modules/chromium_android/api.py
index 06a808b5ab7ef15ad3bbfb4bc5865153a22d6249..738e3842a835056490b8a9bd0ee3dc07150d3cde 100644
--- a/scripts/slave/recipe_modules/chromium_android/api.py
+++ b/scripts/slave/recipe_modules/chromium_android/api.py
@@ -293,10 +293,19 @@ class AndroidApi(recipe_api.RecipeApi):
env=self.m.chromium.get_env(),
infra_step=True)
+
+ def authorize_adb_devices(self):
+ cmd = [self.m.path['build'].join('scripts', 'slave', 'android',
+ 'authorize_adb_devices')]
Vadim Sh. 2015/03/05 05:20:26 authorize_adb_devices.py
+ return self.m.step('authorize_adb_devices', cmd, infra_step=True,
+ env=self.m.chromium.get_env())
+
+
def detect_and_setup_devices(self, restart_usb=False, skip_wipe=False,
disable_location=False, min_battery_level=None,
disable_network=False, disable_java_debug=False,
reboot_timeout=None):
+ self.authorize_adb_devices()
self.device_status_check(restart_usb=restart_usb)
self.provision_devices(
skip_wipe=skip_wipe, disable_location=disable_location,
@@ -622,6 +631,7 @@ class AndroidApi(recipe_api.RecipeApi):
def common_tests_setup_steps(self, perf_setup=False):
self.spawn_logcat_monitor()
+ self.authorize_adb_devices()
self.device_status_check()
if perf_setup:
kwargs = {
@@ -757,4 +767,4 @@ class AndroidApi(recipe_api.RecipeApi):
elif (step_result.retcode == EXIT_CODES['infra']):
step_result.presentation.status = self.m.step.EXCEPTION
elif (step_result.retcode == EXIT_CODES['warning']):
- step_result.presentation.status = self.m.step.WARNING
+ step_result.presentation.status = self.m.step.WARNING

Powered by Google App Engine
This is Rietveld 408576698