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

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

Issue 2122313002: Move known_devices file to ~/.android (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: with expectations 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 side-by-side diff with in-line comments
Download patch
Index: scripts/slave/recipe_modules/chromium_android/example.py
diff --git a/scripts/slave/recipe_modules/chromium_android/example.py b/scripts/slave/recipe_modules/chromium_android/example.py
index ff66f9a559c6478e1dc42c6708eb44983ed5e10d..b127f746887e24429b3a2dd643a133b091492669 100644
--- a/scripts/slave/recipe_modules/chromium_android/example.py
+++ b/scripts/slave/recipe_modules/chromium_android/example.py
@@ -157,6 +157,7 @@ def RunSteps(api, buildername):
# TODO(luqui): remove redundant cruft, need one consistent API.
api.chromium_android.detect_and_setup_devices()
+ api.path.mock_add_paths(api.chromium_android.known_devices_file)
api.chromium_android.device_status_check(
restart_usb=config.get('restart_usb', False))
@@ -381,15 +382,8 @@ def GenTests(api):
'Invalid CTS output here...',
stream='stdout')))
- yield (api.test('device_file_match') +
- properties_for('tester') +
- api.path.exists(
- api.path['build'].join('site_config', '.known_devices')))
-
yield (api.test('device_file_format_mismatch') +
properties_for('tester') +
- api.path.exists(
- api.path['cache'].join('known_android_devices')) +
api.override_step_data('fix_device_file_format.read_device_file',
api.raw_io.output('["device1", "device2"]')))

Powered by Google App Engine
This is Rietveld 408576698