From c808c55594dfca00b57ecc3b3d9eaa8f3a15a81a Mon Sep 17 00:00:00 2001 From: "phajdan.jr@chromium.org" Date: Tue, 26 Apr 2016 14:25:01 +0000 Subject: [PATCH] depot_tools: only add mock infra_paths when test data is enabled BUG=chromium:605919 Review URL: https://codereview.chromium.org/1917093004 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@300193 0039d316-1c4b-4281-b951-d872f2087c98 --- recipe_modules/infra_paths/api.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/recipe_modules/infra_paths/api.py b/recipe_modules/infra_paths/api.py index 1b902826f1..7fd3e1ccd7 100644 --- a/recipe_modules/infra_paths/api.py +++ b/recipe_modules/infra_paths/api.py @@ -29,9 +29,10 @@ class InfraPathsApi(recipe_api.RecipeApi): else: self.set_config('buildbot') - for path in self._test_data.get('exists', []): - assert isinstance(path.base, NamedBasePath) - self.m.path.mock_add_paths(self[path.base.name].join(*path.pieces)) + if self._test_data.enabled: + for path in self._test_data.get('exists', []): + assert isinstance(path.base, NamedBasePath) + self.m.path.mock_add_paths(self[path.base.name].join(*path.pieces)) def __getitem__(self, name): self._lazy_set_config()