Refactor the way that TestApi works so that it is actually useful.

scripts/slave/recipe_modules/gclient/api.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.
 
 from slave import recipe_api
 
 GIT_DEFAULT_WHITELIST = frozenset((
   'tools_build',
 ))
 
 def jsonish_to_python(spec, is_top=False):
   ret = ''
   if is_top:  # We're the 'top' level, so treat this dict as a suite.
     ret = '\n'.join(
       '%s = %s' % (k, jsonish_to_python(spec[k])) for k in sorted(spec)
     )
   else:
     if isinstance(spec, dict):
       ret += '{'
       ret += ', '.join(
-        "%s: %s" % (repr(str(k)), jsonish_to_python(spec[k])) for k in sorted(spec))
+        "%s: %s" % (repr(str(k)), jsonish_to_python(spec[k]))
+        for k in sorted(spec)
+      )
       ret += '}'
     elif isinstance(spec, list):
       ret += '['
       ret += ', '.join(jsonish_to_python(x) for x in spec)
       ret += ']'
     elif isinstance(spec, basestring):
       ret = repr(str(spec))
     else:
       ret = repr(spec)
   return ret
@@ skipping 122 matching lines @@
         self.m.path.build('scripts', 'slave', 'gclient_safe_revert.py'),
         ['.', self.m.path.depot_tools('gclient', wrapper=True)],
         **kwargs
     )
 
   def runhooks(self, args=None, **kwargs):
     """Return a 'gclient runhooks' step."""
     args = args or []
     assert isinstance(args, (list, tuple))
     return self('runhooks', ['runhooks'] + list(args), **kwargs)