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

Unified Diff: tools/telemetry/telemetry/internal/results/results_options.py

Issue 1647513002: Delete tools/telemetry. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 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: tools/telemetry/telemetry/internal/results/results_options.py
diff --git a/tools/telemetry/telemetry/internal/results/results_options.py b/tools/telemetry/telemetry/internal/results/results_options.py
deleted file mode 100644
index e4386f6995b875e953ad1221b1f18ddf3817df17..0000000000000000000000000000000000000000
--- a/tools/telemetry/telemetry/internal/results/results_options.py
+++ /dev/null
@@ -1,171 +0,0 @@
-# Copyright 2014 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 optparse
-import os
-import sys
-
-from catapult_base import cloud_storage # pylint: disable=import-error
-
-from telemetry.core import util
-from telemetry.internal.results import buildbot_output_formatter
-from telemetry.internal.results import chart_json_output_formatter
-from telemetry.internal.results import csv_pivot_table_output_formatter
-from telemetry.internal.results import gtest_progress_reporter
-from telemetry.internal.results import html_output_formatter
-from telemetry.internal.results import json_output_formatter
-from telemetry.internal.results import page_test_results
-from telemetry.internal.results import progress_reporter
-
-# Allowed output formats. The default is the first item in the list.
-_OUTPUT_FORMAT_CHOICES = ('html', 'buildbot', 'gtest', 'json',
- 'chartjson', 'csv-pivot-table', 'none')
-
-
-# Filenames to use for given output formats.
-_OUTPUT_FILENAME_LOOKUP = {
- 'html': 'results.html',
- 'json': 'results.json',
- 'chartjson': 'results-chart.json',
- 'csv-pivot-table': 'results-pivot-table.csv'
-}
-
-
-def AddResultsOptions(parser):
- group = optparse.OptionGroup(parser, 'Results options')
- group.add_option('--output-format', action='append', dest='output_formats',
- choices=_OUTPUT_FORMAT_CHOICES, default=[],
- help='Output format. Defaults to "%%default". '
- 'Can be %s.' % ', '.join(_OUTPUT_FORMAT_CHOICES))
- group.add_option('-o', '--output',
- dest='output_file',
- default=None,
- help='Redirects output to a file. Defaults to stdout.')
- group.add_option('--output-dir', default=util.GetBaseDir(),
- help='Where to save output data after the run.')
- group.add_option('--output-trace-tag',
- default='',
- help='Append a tag to the key of each result trace. Use '
- 'with html, buildbot, csv-pivot-table output formats.')
- group.add_option('--reset-results', action='store_true',
- help='Delete all stored results.')
- group.add_option('--upload-results', action='store_true',
- help='Upload the results to cloud storage.')
- group.add_option('--upload-bucket', default='output',
- choices=cloud_storage.BUCKET_ALIAS_NAMES,
- help='Storage bucket to use for the uploaded results. ' +
- 'Defaults to output bucket. Supported values are: ' +
- ', '.join(cloud_storage.BUCKET_ALIAS_NAMES) + '.')
- group.add_option('--results-label',
- default=None,
- help='Optional label to use for the results of a run .')
- group.add_option('--suppress_gtest_report',
- default=False,
- help='Whether to suppress GTest progress report.')
- parser.add_option_group(group)
-
-
-def ProcessCommandLineArgs(parser, args):
- # TODO(ariblue): Delete this flag entirely at some future data, when the
- # existence of such a flag has been long forgotten.
- if args.output_file:
- parser.error('This flag is deprecated. Please use --output-dir instead.')
-
- try:
- os.makedirs(args.output_dir)
- except OSError:
- # Do nothing if the output directory already exists. Existing files will
- # get overwritten.
- pass
-
- args.output_dir = os.path.expanduser(args.output_dir)
-
-
-def _GetOutputStream(output_format, output_dir):
- assert output_format in _OUTPUT_FORMAT_CHOICES, 'Must specify a valid format.'
- assert output_format not in ('gtest', 'none'), (
- 'Cannot set stream for \'gtest\' or \'none\' output formats.')
-
- if output_format == 'buildbot':
- return sys.stdout
-
- assert output_format in _OUTPUT_FILENAME_LOOKUP, (
- 'No known filename for the \'%s\' output format' % output_format)
- output_file = os.path.join(output_dir, _OUTPUT_FILENAME_LOOKUP[output_format])
-
- # TODO(eakuefner): Factor this hack out after we rewrite HTMLOutputFormatter.
- if output_format == 'html':
- open(output_file, 'a').close() # Create file if it doesn't exist.
- return open(output_file, 'r+')
- else:
- return open(output_file, 'w+')
-
-
-def _GetProgressReporter(output_skipped_tests_summary, suppress_gtest_report):
- if suppress_gtest_report:
- return progress_reporter.ProgressReporter()
-
- return gtest_progress_reporter.GTestProgressReporter(
- sys.stdout, output_skipped_tests_summary=output_skipped_tests_summary)
-
-
-def CreateResults(benchmark_metadata, options,
- value_can_be_added_predicate=lambda v, is_first: True):
- """
- Args:
- options: Contains the options specified in AddResultsOptions.
- """
- if not options.output_formats:
- options.output_formats = [_OUTPUT_FORMAT_CHOICES[0]]
-
- output_formatters = []
- for output_format in options.output_formats:
- if output_format == 'none' or output_format == "gtest":
- continue
-
- output_stream = _GetOutputStream(output_format, options.output_dir)
- if output_format == 'csv-pivot-table':
- output_formatters.append(
- csv_pivot_table_output_formatter.CsvPivotTableOutputFormatter(
- output_stream, trace_tag=options.output_trace_tag))
- elif output_format == 'buildbot':
- output_formatters.append(
- buildbot_output_formatter.BuildbotOutputFormatter(
- output_stream, trace_tag=options.output_trace_tag))
- elif output_format == 'html':
- # TODO(chrishenry): We show buildbot output so that users can grep
- # through the results easily without needing to open the html
- # file. Another option for this is to output the results directly
- # in gtest-style results (via some sort of progress reporter),
- # as we plan to enable gtest-style output for all output formatters.
- output_formatters.append(
- buildbot_output_formatter.BuildbotOutputFormatter(
- sys.stdout, trace_tag=options.output_trace_tag))
- output_formatters.append(html_output_formatter.HtmlOutputFormatter(
- output_stream, benchmark_metadata, options.reset_results,
- options.upload_results, options.browser_type,
- options.results_label))
- elif output_format == 'json':
- output_formatters.append(json_output_formatter.JsonOutputFormatter(
- output_stream, benchmark_metadata))
- elif output_format == 'chartjson':
- output_formatters.append(
- chart_json_output_formatter.ChartJsonOutputFormatter(
- output_stream, benchmark_metadata))
- else:
- # Should never be reached. The parser enforces the choices.
- raise Exception('Invalid --output-format "%s". Valid choices are: %s'
- % (output_format, ', '.join(_OUTPUT_FORMAT_CHOICES)))
-
- # TODO(chrishenry): This is here to not change the output of
- # gtest. Let's try enabling skipped tests summary for gtest test
- # results too (in a separate patch), and see if we break anything.
- output_skipped_tests_summary = 'gtest' in options.output_formats
-
- reporter = _GetProgressReporter(output_skipped_tests_summary,
- options.suppress_gtest_report)
- return page_test_results.PageTestResults(
- output_formatters=output_formatters, progress_reporter=reporter,
- output_dir=options.output_dir,
- value_can_be_added_predicate=value_can_be_added_predicate)

Powered by Google App Engine
This is Rietveld 408576698