Make step status EXCEPTION when test runner has infrastructure failure.

scripts/slave/recipe_modules/chromium_android/config.py

 # Copyright 2013 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.
 
 import types
 
 from slave.recipe_config import config_item_context, ConfigGroup
 from slave.recipe_config import ConfigList, Dict, List, Single, Static
 from slave.recipe_config_types import Path
 
@@ skipping 11 matching lines @@
     run_tree_truth = Single(bool, required=False, empty_val=True),
     deps_file = Single(basestring, required=False, empty_val='.DEPS.git'),
     internal_dir_name = Single(basestring, required=False),
     # deps_dir: where to checkout the gclient deps file
     deps_dir = Single(basestring, required=False, empty_val=REPO_NAME),
     managed = Single(bool, required=False, empty_val=True),
     extra_deploy_opts = List(inner_type=basestring),
     tests = List(inner_type=basestring),
     cr_build_android = Static(Path('[CHECKOUT]', 'build', 'android')),
     test_runner = Single(Path),
+    test_runner_exit_codes = Dict(value_type=int),

[luqui, 2015/03/02 23:40:58]

This doesn't feel right to me.  I would prefer to hardcode this dictionary in
the test_runner function -- keeps the implementation details from leaking all
over the place.

Unless you expect different configurations to use different dictionaries, which
I find unlikely.

[mikecase (-- gone --), 2015/03/03 00:46:47]

Done.

     gclient_custom_deps = Dict(value_type=(basestring, types.NoneType)),
     storage_bucket = Single(basestring),
     channel = Single(basestring, empty_val='chrome'),
     upload_dest_prefix = Single(basestring, empty_val=''),
     gclient_custom_vars = Dict(value_type=(basestring, types.NoneType)),
     coverage = Single(bool, required=False, empty_val=False),
   )
 
 
 VAR_TEST_MAP = {
   'INTERNAL': [True, False],
   'REPO_NAME': ['src/clank'],
   'REPO_URL': ['<hidden>'],  # supplied in build properties
   'BUILD_CONFIG': ['Debug', 'Release'],
 }
 
 def TEST_NAME_FORMAT(kwargs):  # pragma: no cover
   name = 'repo-%(REPO_NAME)s-from-url-%(REPO_URL)s' % kwargs
   if kwargs['INTERNAL']:
     return name + '-internal'
   else:
     return name
 
 config_ctx = config_item_context(BaseConfig, VAR_TEST_MAP, TEST_NAME_FORMAT)
 
 @config_ctx(is_root=True)
 def base_config(c):
   c.internal_dir_name = 'clank'
   c.test_runner = Path('[CHECKOUT]', 'build', 'android', 'test_runner.py')
+  c.test_runner_exit_codes = {'ERROR': 1, 'INFRA': 87, 'WARNING': 88}
 
 @config_ctx()
 def main_builder(c):
   pass
 
 @config_ctx()
 def trybot_builder(c):
   pass
 
 @config_ctx()
@@ skipping 128 matching lines @@
 def oilpan_builder(c):
   pass
 
 @config_ctx()
 def perf(c):
   pass
 
 @config_ctx()
 def webview_perf(c):
   pass