recipe_tryjob: Allow test bypass.

scripts/slave/recipes/infra/recipe_roll_tryjob.py

 # Copyright (c) 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Rolls recipes.cfg dependencies."""
 
 DEPS = [
   'recipe_tryjob',
   'luci_config',
+  'recipe_engine/json',
   'recipe_engine/properties',
   'recipe_engine/raw_io',
   'recipe_engine/step',
 ]
 
 from recipe_engine.recipe_api import Property
 
 
 def get_auth_token(api, service_account=None):
   """
@@ skipping 83 matching lines @@
 
   yield (
       api.test('one_patch') +
       api.luci_config.get_projects(('recipe_engine', 'build')) +
       api.luci_config.get_project_config(
           'build', 'recipes.cfg',
           api.recipe_tryjob.make_recipe_config('build')) +
       api.luci_config.get_project_config(
           'recipe_engine', 'recipes.cfg',
           api.recipe_tryjob.make_recipe_config('recipe_engine')) +
-      api.properties(patches="build:https://fake.code.review/123456#ps1")
+      api.properties(patches="build:https://fake.code.review/123456#ps1") +
+      api.override_step_data(
+          'git_cl description (build)', stdout=api.raw_io.output(
+              "")) +
+      api.override_step_data(
+          'parse description', api.json.output(
+              {}))
   )
 
   yield (
+      api.test('three_patches') +
+      api.luci_config.get_projects(('recipe_engine', 'build', 'depot_tools')) +
+      api.luci_config.get_project_config(
+          'build', 'recipes.cfg',
+          api.recipe_tryjob.make_recipe_config(
+              'build', ['depot_tools', 'recipe_engine'])) +
+      api.luci_config.get_project_config(
+          'recipe_engine', 'recipes.cfg',
+          api.recipe_tryjob.make_recipe_config('recipe_engine')) +
+      api.luci_config.get_project_config(
+          'depot_tools', 'recipes.cfg',
+          api.recipe_tryjob.make_recipe_config(
+              'depot_tools', ['recipe_engine'])) +
+      api.properties(
+          patches="depot_tools:https://fake.code.review/123456#ps1,"
+                  "build:https://fake.code.review/789999#ps2") +
+      api.override_step_data(
+          'git_cl description (build)', stdout=api.raw_io.output(
+              "")) +
+      api.override_step_data(
+          'git_cl description (depot_tools)', stdout=api.raw_io.output(
+              "")) +
+      api.override_step_data(
+          'parse description', api.json.output(
+              {})) +
+      api.override_step_data(
+          'parse description (2)', api.json.output(
+              {}))
+  )
+
+  yield (
+      api.test('three_patches_fail_not_ok') +
+      api.luci_config.get_projects(('recipe_engine', 'build', 'depot_tools')) +
+      api.luci_config.get_project_config(
+          'build', 'recipes.cfg',
+          api.recipe_tryjob.make_recipe_config(
+              'build', ['depot_tools', 'recipe_engine'])) +
+      api.luci_config.get_project_config(
+          'recipe_engine', 'recipes.cfg',
+          api.recipe_tryjob.make_recipe_config('recipe_engine')) +
+      api.luci_config.get_project_config(
+          'depot_tools', 'recipes.cfg',
+          api.recipe_tryjob.make_recipe_config(
+              'depot_tools', ['recipe_engine'])) +
+      api.properties(
+          patches="depot_tools:https://fake.code.review/123456#ps1,"
+                  "build:https://fake.code.review/789999#ps2") +
+      api.override_step_data(
+          'git_cl description (build)', stdout=api.raw_io.output(
+              "")) +
+      api.override_step_data(
+          'git_cl description (depot_tools)', stdout=api.raw_io.output(
+              "")) +
+      api.override_step_data(
+          'parse description', api.json.output(
+              {})) +
+      api.override_step_data(
+          'parse description (2)', api.json.output(
+              {})) +
+      api.override_step_data("build tests", retcode=1)
+  )
+
+  yield (
+      api.test('three_patches_fail_ok') +
+      api.luci_config.get_projects(('recipe_engine', 'build', 'depot_tools')) +
+      api.luci_config.get_project_config(
+          'build', 'recipes.cfg',
+          api.recipe_tryjob.make_recipe_config(
+              'build', ['depot_tools', 'recipe_engine'])) +
+      api.luci_config.get_project_config(
+          'recipe_engine', 'recipes.cfg',
+          api.recipe_tryjob.make_recipe_config('recipe_engine')) +
+      api.luci_config.get_project_config(
+          'depot_tools', 'recipes.cfg',
+          api.recipe_tryjob.make_recipe_config(
+              'depot_tools', ['recipe_engine'])) +
+      api.properties(
+          patches="depot_tools:https://fake.code.review/123456#ps1,"
+                  "build:https://fake.code.review/789999#ps2") +
+      api.override_step_data(
+          'git_cl description (build)', stdout=api.raw_io.output(
+              "foo")) +
+      api.override_step_data(
+          'git_cl description (depot_tools)', stdout=api.raw_io.output(
+              "Recipe-Tryjob-Bypass-Reason: best reason")) +
+      api.override_step_data(
+          'parse description', api.json.output(
+              {'Recipe-Tryjob-Bypass-Reason': ['Best Reason']})) +
+      api.override_step_data(
+          'parse description (2)', api.json.output(
+              {'Recipe-Tryjob-Bypass-Reason': []})) +
+      api.override_step_data("build tests", retcode=1)
+  )
+
+  yield (
       api.test('bad_patches') +
       api.properties(
           patches="build:https://f.e.w/1#ps1,build:https://f.e.w/1#ps1")
   )
 
   yield (
       api.test('deps') +
       api.luci_config.get_projects(('recipe_engine', 'build')) +
       api.luci_config.get_project_config(
           'build', 'recipes.cfg',
           api.recipe_tryjob.make_recipe_config('build', ['recipe_engine'])) +
       api.luci_config.get_project_config(
           'recipe_engine', 'recipes.cfg',
           api.recipe_tryjob.make_recipe_config('recipe_engine')) +
       api.properties(
-          patches="recipe_engine:https://fake.code.review/123456#ps1")
+          patches="recipe_engine:https://fake.code.review/123456#ps1") +
+      api.override_step_data(
+          'parse description', api.json.output({})) +
+      api.override_step_data(
+          'git_cl description (recipe_engine)', stdout=api.raw_io.output(
+              "foo"))
   )
 
   yield (
       api.test('tryjob') +
       api.properties(
         rietveld="https://fake.code.review",
         issue='12345678',
         patchset='1',
         patch_project="build",
       ) +
       api.luci_config.get_projects(('recipe_engine', 'build')) +
       api.luci_config.get_project_config(
           'build', 'recipes.cfg',
           api.recipe_tryjob.make_recipe_config('build', ['recipe_engine'])) +
       api.luci_config.get_project_config(
           'recipe_engine', 'recipes.cfg',
-          api.recipe_tryjob.make_recipe_config('recipe_engine'))
+          api.recipe_tryjob.make_recipe_config('recipe_engine')) +
+      api.override_step_data(
+          'git_cl description (build)', stdout=api.raw_io.output(
+              "")) +
+      api.override_step_data(
+          'parse description', api.json.output(
+              {}))
   )
 
   yield (
       api.test('tryjob_dont_test_untouched_code') +
       api.properties(
         rietveld="https://fake.code.review",
         issue='12345678',
         patchset='1',
         patch_project="build",
       ) +
       api.luci_config.get_projects(('recipe_engine', 'build', 'foobar')) +
       api.luci_config.get_project_config(
           'build', 'recipes.cfg',
           api.recipe_tryjob.make_recipe_config('build', ['recipe_engine'])) +
       api.luci_config.get_project_config(
           'recipe_engine', 'recipes.cfg',
-          api.recipe_tryjob.make_recipe_config('recipe_engine'))
+          api.recipe_tryjob.make_recipe_config('recipe_engine')) +
+      api.override_step_data(
+          'git_cl description (build)', stdout=api.raw_io.output(
+              "")) +
+      api.override_step_data(
+          'parse description', api.json.output(
+              {}))
   )