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

scripts/slave/recipe_modules/path/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.
 
 import os
 from slave import recipe_api
 
 
 def path_method(api, name, base):
   """Returns a shortcut static method which functions like os.path.join but
@@ skipping 82 matching lines @@
     exists (list): Paths which should exist in the test case. Thes must be paths
       using the [*_ROOT] placeholders. ex. '[BUILD_ROOT]/scripts'.
   """
 
   OK_METHODS = ('abspath', 'basename', 'exists', 'join', 'pardir',
                 'sep', 'split', 'splitext')
 
   def __init__(self, **kwargs):
     super(PathApi, self).__init__(**kwargs)
 
-    if self._mock is None:  # pragma: no cover
+    if not self._test_data.enabled:  # pragma: no cover
       self._path_mod = os.path
       # e.g. /b/build/slave/<slavename>/build
       self.slave_build = path_method(
         self, 'slave_build', self.abspath(os.getcwd()))
 
       # e.g. /b
       r = self.abspath(self.join(self.slave_build(), *([self.pardir]*4)))
       for token in ('build_internal', 'build', 'depot_tools'):
         # e.g. /b/{token}
         setattr(self, token, path_method(self, token, self.join(r, token)))
       self.root = path_method(self, 'root', r)
     else:
-      self._path_mod = mock_path(self.m, self._mock.get('exists', []))
+      self._path_mod = mock_path(self.m, self._test_data.get('exists', []))
       self.slave_build = path_method(self, 'slave_build', '[SLAVE_BUILD_ROOT]')
       self.build_internal = path_method(
         self, 'build_internal', '[BUILD_INTERNAL_ROOT]')
       self.build = path_method(self, 'build', '[BUILD_ROOT]')
       self.depot_tools = path_method(self, 'depot_tools', '[DEPOT_TOOLS_ROOT]')
       self.root = path_method(self, 'root', '[ROOT]')
 
     # Because it only makes sense to call self.checkout() after
     # a checkout has been defined, make calls to self.checkout()
     # explode with a helpful message until that point.
@@ skipping 10 matching lines @@
 
     The checked out source is often a forest of trees possibly inside other
     trees.  One of these trees' root is designated as special/primary and
     this method builds paths inside of it.  For Chrome, that would be 'src'.
     This defaults to the special root of the first checkout.
     """
     return self._checkout(*args, **kwargs)
 
   def mock_add_paths(self, path):
     """For testing purposes, assert that |path| exists."""
-    if self._mock is not None:
+    if self._test_data.enabled:
       self._path_mod.mock_add_paths(path)
 
   def add_checkout(self, checkout, *pieces):
     """Assert that we have a source directory with this name. """
     checkout = self.join(checkout, *pieces)
     self.assert_absolute(checkout)
     if not self._checkouts:
       self._checkout = path_method(self, 'checkout', checkout)
     self._checkouts.append(checkout)
 
@@ skipping 32 matching lines @@
     if name in self.OK_METHODS:
       return getattr(self._path_mod, name)
     raise AttributeError("'%s' object has no attribute '%s'" %
                          (self._path_mod, name))  # pragma: no cover
 
   def __dir__(self):  # pragma: no cover
     # Used for helping out show_me_the_modules.py
     return self.__dict__.keys() + list(self.OK_METHODS)
 
   # TODO(iannucci): Custom paths?