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

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

Issue 2181693004: Add a check that the Chromedriver prebuilt name is expected. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Changed to Sam's impl 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/chromedriver/example.py
diff --git a/scripts/slave/recipe_modules/chromedriver/example.py b/scripts/slave/recipe_modules/chromedriver/example.py
index 4052550e54c979793a21e95220a11505cb3664b6..397377dd0120e01c4b53f624f37cf67d8437cdc3 100644
--- a/scripts/slave/recipe_modules/chromedriver/example.py
+++ b/scripts/slave/recipe_modules/chromedriver/example.py
@@ -13,6 +13,7 @@ DEPS = [
'commit_position',
'depot_tools/bot_update',
'depot_tools/gclient',
+ 'recipe_engine/json',
'recipe_engine/platform',
'recipe_engine/properties',
'recipe_engine/raw_io',
@@ -106,3 +107,15 @@ def GenTests(api):
got_revision_cp='refs/heads/master@{#3333333333}') +
api.step_data(
'Download Test Results Log.gsutil download results log', retcode=1))
+
+ yield (
+ api.test('%s_unexpected_prebuilt' % sanitize('Android ChromeDriver')) +
+ api.properties.generic(
+ buildername='Android ChromeDriver Tests Example',
+ slavename='slavename') +
+ api.properties(
+ parent_build_archive_url='gs://test-domain/test-archive.zip',
+ got_revision_cp='refs/heads/master@{#3333333333}') +
+ api.step_data(
+ 'Download Prebuilts.listdir get prebuilt filename',
+ api.json.output(['rNone.zip'])))

Powered by Google App Engine
This is Rietveld 408576698