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

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

Issue 2103553002: Check that device file exists first before reverting. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: rebase Created 4 years, 6 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 c49bae08d6c334cff7d03627b37deb900f6ed717..42185c6825a28dbf7b093a6912fbe32918b951c8 100644
--- a/scripts/slave/recipe_modules/chromium_android/example.py
+++ b/scripts/slave/recipe_modules/chromium_android/example.py
@@ -381,8 +381,15 @@ 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['build'].join('site_config', '.known_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