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

Side by Side Diff: scripts/slave/recipe_modules/chromium_android/example.py

Issue 2107003002: Revert "[Android] Move to new device status and recovery scripts." (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 5 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 unified diff | Download patch
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 from recipe_engine.types import freeze 5 from recipe_engine.types import freeze
6 6
7 DEPS = [ 7 DEPS = [
8 'adb', 8 'adb',
9 'chromium', 9 'chromium',
10 'chromium_android', 10 'chromium_android',
(...skipping 136 matching lines...) Expand 10 before | Expand all | Expand 10 after
147 api.chromium.output_dir.join('chrome_public_apk', 'classes.dex.zip')) 147 api.chromium.output_dir.join('chrome_public_apk', 'classes.dex.zip'))
148 148
149 if config.get('specific_install'): 149 if config.get('specific_install'):
150 api.chromium_android.adb_install_apk('Chrome.apk', devices=['abc123']) 150 api.chromium_android.adb_install_apk('Chrome.apk', devices=['abc123'])
151 151
152 api.adb.root_devices() 152 api.adb.root_devices()
153 api.chromium_android.spawn_logcat_monitor() 153 api.chromium_android.spawn_logcat_monitor()
154 154
155 failure = False 155 failure = False
156 try: 156 try:
157 # detect_and_setup_devices contains device recovery, provisiong, and status. 157 # TODO(luqui): remove redundant cruft, need one consistent API.
158 api.chromium_android.detect_and_setup_devices() 158 api.chromium_android.detect_and_setup_devices()
159 api.chromium_android.device_status_check() 159
160 # Needed to test all flags to provision devices. 160 api.chromium_android.device_status_check(
161 restart_usb=config.get('restart_usb', False))
162
161 api.chromium_android.provision_devices( 163 api.chromium_android.provision_devices(
162 skip_wipe=config.get('skip_wipe', False), 164 skip_wipe=config.get('skip_wipe', False),
163 disable_location=config.get('disable_location', False), 165 disable_location=config.get('disable_location', False),
164 min_battery_level=config.get('min_battery_level'), 166 min_battery_level=config.get('min_battery_level'),
165 max_battery_temp=config.get('max_battery_temp'), 167 max_battery_temp=config.get('max_battery_temp'),
166 reboot_timeout=1800, 168 reboot_timeout=1800,
167 remove_system_webview=config.get('remove_system_webview', False), 169 remove_system_webview=config.get('remove_system_webview', False),
168 disable_system_chrome=config.get('disable_system_chrome', False)) 170 disable_system_chrome=config.get('disable_system_chrome', False))
169 171
170 except api.step.StepFailure as f: 172 except api.step.StepFailure as f:
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
239 slavename='tehslave', 241 slavename='tehslave',
240 repo_name='src/repo', 242 repo_name='src/repo',
241 patch_url='https://the.patch.url/the.patch', 243 patch_url='https://the.patch.url/the.patch',
242 repo_url='svn://svn.chromium.org/chrome/trunk/src', 244 repo_url='svn://svn.chromium.org/chrome/trunk/src',
243 revision='4f4b02f6b7fa20a3a25682c457bbc8ad589c8a00', 245 revision='4f4b02f6b7fa20a3a25682c457bbc8ad589c8a00',
244 internal=True) 246 internal=True)
245 247
246 for buildername in BUILDERS: 248 for buildername in BUILDERS:
247 yield api.test('%s_basic' % buildername) + properties_for(buildername) 249 yield api.test('%s_basic' % buildername) + properties_for(buildername)
248 250
249 yield (api.test('tester_no_devices_during_recovery') + 251 yield (api.test('tester_no_devices') +
250 properties_for('tester') + 252 properties_for('tester') +
251 api.step_data('device_recovery', retcode=1)) 253 api.step_data('device_status_check', retcode=1))
252 254
253 yield (api.test('tester_no_devices_during_status') + 255 yield (api.test('tester_other_device_failure') +
254 properties_for('tester') + 256 properties_for('tester') +
255 api.step_data('device_status', retcode=1)) 257 api.step_data('device_status_check', retcode=2))
256
257 yield (api.test('tester_other_device_failure_during_recovery') +
258 properties_for('tester') +
259 api.step_data('device_recovery', retcode=2))
260
261 yield (api.test('tester_other_device_failure_during_status') +
262 properties_for('tester') +
263 api.step_data('device_status', retcode=2))
264 258
265 yield (api.test('tester_with_step_warning') + 259 yield (api.test('tester_with_step_warning') +
266 properties_for('tester') + 260 properties_for('tester') +
267 api.step_data('unittests', retcode=88)) 261 api.step_data('unittests', retcode=88))
268 262
269 yield (api.test('tester_blacklisted_devices') + 263 yield (api.test('tester_blacklisted_devices') +
270 properties_for('tester') + 264 properties_for('tester') +
271 api.override_step_data('provision_devices', 265 api.override_step_data('provision_devices',
272 api.json.output(['abc123', 'def456']))) 266 api.json.output(['abc123', 'def456'])))
273 267
274 yield (api.test('tester_offline_devices') + 268 yield (api.test('tester_offline_devices') +
275 properties_for('tester') + 269 properties_for('tester') +
276 api.override_step_data('device_status', 270 api.override_step_data('device_status_check',
277 api.json.output([{}, {}]))) 271 api.json.output([{}, {}])))
278 272
279 yield (api.test('perf_tests_failure') + 273 yield (api.test('perf_tests_failure') +
280 properties_for('perf_runner') + 274 properties_for('perf_runner') +
281 api.step_data('perf_test.foo', retcode=1)) 275 api.step_data('perf_test.foo', retcode=1))
282 276
283 yield (api.test('gerrit_refs') + 277 yield (api.test('gerrit_refs') +
284 api.properties.generic( 278 api.properties.generic(
285 buildername='gerrit_try_builder', 279 buildername='gerrit_try_builder',
286 slavename='testslave', 280 slavename='testslave',
(...skipping 109 matching lines...) Expand 10 before | Expand all | Expand 10 after
396 properties_for('tester') + 390 properties_for('tester') +
397 api.path.exists( 391 api.path.exists(
398 api.path['cache'].join('known_android_devices')) + 392 api.path['cache'].join('known_android_devices')) +
399 api.override_step_data('fix_device_file_format.read_device_file', 393 api.override_step_data('fix_device_file_format.read_device_file',
400 api.raw_io.output('["device1", "device2"]'))) 394 api.raw_io.output('["device1", "device2"]')))
401 395
402 yield (api.test('tombstones_m53') + 396 yield (api.test('tombstones_m53') +
403 properties_for('tester') + 397 properties_for('tester') +
404 api.override_step_data('get version (2)', 398 api.override_step_data('get version (2)',
405 api.raw_io.output('MAJOR=53\nMINOR=0\nBUILD=2800\nPATCH=0\n'))) 399 api.raw_io.output('MAJOR=53\nMINOR=0\nBUILD=2800\nPATCH=0\n')))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698