Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(42)

Unified Diff: scripts/slave/recipe_modules/path/api.py

Issue 24650004: Fix hack for test presentation in path api. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: rebase Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: scripts/slave/recipe_modules/path/api.py
diff --git a/scripts/slave/recipe_modules/path/api.py b/scripts/slave/recipe_modules/path/api.py
index 39a786e02657063589c6ead00465a5e3c189fd3b..19de71445ffb22fcf6a064d47ebd40b6cbf94777 100644
--- a/scripts/slave/recipe_modules/path/api.py
+++ b/scripts/slave/recipe_modules/path/api.py
@@ -18,11 +18,7 @@ def PathTostring(api, test):
base_path = api.c.dynamic_paths[path.base]
elif path.base in api.c.base_paths:
if test.enabled:
- # TODO(iannucci): Remove special case in followup cl
- if path.base == 'root':
- base_path = '[ROOT]'
- else:
- base_path = '[%s_ROOT]' % path.base.upper()
+ base_path = '[%s]' % path.base.upper()
else: # pragma: no cover
base_path = api.join(*api.c.base_paths[path.base])
assert base_path, 'Could not get base %r for path' % path.base

Powered by Google App Engine
This is Rietveld 408576698