Always identify test names that lead to failures

recipe_modules/step/example.py

 # Copyright 2013 The LUCI Authors. All rights reserved.
 # Use of this source code is governed under the Apache License, Version 2.0
 # that can be found in the LICENSE file.
 
 from recipe_engine import recipe_api, config
 
 DEPS = [
   'path',
   'properties',
   'step',
@@ skipping 14 matching lines @@
 
 
 def RunSteps(
     api, bad_return, raise_infra_failure, access_invalid_data, timeout):
   if bad_return:
     return RETURN_SCHEMA.new(test_me='this should fail')
   elif timeout:
     # Timeout causes the recipe engine to raise an exception if your step takes
     # longer to run than you allow. Units are seconds.
     if timeout == 1:
+      raise Exception('fuu')

[iannucci, 2017/04/04 20:24:51]

did you mean to leave this in? should this be added as a different test?

[Paweł Hajdan Jr., 2017/04/04 20:32:03]

Ouch, indeed left by mistake. Also detected by presubmit. Removed now.

By the way, we do have test coverage for this - see changing tests.

       api.step('timeout', ['sleep', '20'], timeout=1)
     elif timeout == 2:
       try:
         api.step('caught timeout', ['sleep', '20'], timeout=1)
       except api.step.StepTimeout:
         return RETURN_SCHEMA(test_me=4)
 
 
   # TODO(martinis) change this
   # The api.step object is directly callable.
@@ skipping 126 matching lines @@
       api.test('timeout') +
       api.properties(timeout=1) +
       api.step_data('timeout', times_out_after=20)
     )
 
   yield (
       api.test('catch_timeout') +
       api.properties(timeout=2) +
       api.step_data('caught timeout', times_out_after=20)
     )