Index: tools/telemetry/telemetry/test_runner.py |
diff --git a/tools/telemetry/telemetry/test_runner.py b/tools/telemetry/telemetry/test_runner.py |
index 0e2770a443a35021c1b22c29b995cbcf09c08af8..df6cda405955573f80995de8a468fab7b29f07bb 100644 |
--- a/tools/telemetry/telemetry/test_runner.py |
+++ b/tools/telemetry/telemetry/test_runner.py |
@@ -161,9 +161,8 @@ class Run(command_line.OptparseCommand): |
page_set_name = args.positional_args[1] |
page_set_class = _MatchPageSetName(page_set_name) |
if page_set_class is None: |
- parser.error( |
- 'Page set not found. Please specify the name of a valid page set.') |
- #TODO(ariblue): Print available page sets. |
+ parser.error("Page set %s not found. Available sets:\n%s" % |
+ (page_set_name, _AvailablePageSetNamesString())) |
class TestWrapper(benchmark.Benchmark): |
test = test_class |
@@ -233,6 +232,15 @@ def _MatchPageSetName(input_name): |
return None |
+def _AvailablePageSetNamesString(): |
+ result = "" |
+ for base_dir in config.base_paths: |
+ for p in discover.DiscoverClasses(base_dir, base_dir, page_set.PageSet, |
+ index_by_class_name=True).values(): |
+ result += p.Name() + "\n" |
+ return result |
+ |
+ |
def _MatchTestName(input_test_name, exact_matches=True): |
def _Matches(input_string, search_string): |
if search_string.startswith(input_string): |