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

Unified Diff: build/android/pylib/results/report_results.py

Issue 2392643003: Removes files from //build that we don't need (Closed)
Patch Set: Created 4 years, 2 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
« no previous file with comments | « build/android/pylib/results/json_results_test.py ('k') | build/android/pylib/screenshot.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/pylib/results/report_results.py
diff --git a/build/android/pylib/results/report_results.py b/build/android/pylib/results/report_results.py
deleted file mode 100644
index 4fc6aa0b5cf959bf8e680241f80d563528487066..0000000000000000000000000000000000000000
--- a/build/android/pylib/results/report_results.py
+++ /dev/null
@@ -1,114 +0,0 @@
-# Copyright (c) 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.
-
-"""Module containing utility functions for reporting results."""
-
-import logging
-import os
-import re
-
-from pylib import constants
-from pylib.results.flakiness_dashboard import results_uploader
-
-
-def _LogToFile(results, test_type, suite_name):
- """Log results to local files which can be used for aggregation later."""
- log_file_path = os.path.join(constants.GetOutDirectory(), 'test_logs')
- if not os.path.exists(log_file_path):
- os.mkdir(log_file_path)
- full_file_name = os.path.join(
- log_file_path, re.sub(r'\W', '_', test_type).lower() + '.log')
- if not os.path.exists(full_file_name):
- with open(full_file_name, 'w') as log_file:
- print >> log_file, '\n%s results for %s build %s:' % (
- test_type, os.environ.get('BUILDBOT_BUILDERNAME'),
- os.environ.get('BUILDBOT_BUILDNUMBER'))
- logging.info('Writing results to %s.' % full_file_name)
-
- logging.info('Writing results to %s.' % full_file_name)
- with open(full_file_name, 'a') as log_file:
- shortened_suite_name = suite_name[:25] + (suite_name[25:] and '...')
- print >> log_file, '%s%s' % (shortened_suite_name.ljust(30),
- results.GetShortForm())
-
-
-def _LogToFlakinessDashboard(results, test_type, test_package,
- flakiness_server):
- """Upload results to the flakiness dashboard"""
- logging.info('Upload results for test type "%s", test package "%s" to %s' %
- (test_type, test_package, flakiness_server))
-
- try:
- if test_type == 'Instrumentation':
- if flakiness_server == constants.UPSTREAM_FLAKINESS_SERVER:
- assert test_package in ['ContentShellTest',
- 'ChromePublicTest',
- 'ChromeShellTest',
- 'ChromeSyncShellTest',
- 'AndroidWebViewTest']
- dashboard_test_type = ('%s_instrumentation_tests' %
- test_package.lower().rstrip('test'))
- # Downstream server.
- else:
- dashboard_test_type = 'Chromium_Android_Instrumentation'
-
- elif test_type == 'Unit test':
- dashboard_test_type = test_package
-
- else:
- logging.warning('Invalid test type')
- return
-
- results_uploader.Upload(
- results, flakiness_server, dashboard_test_type)
-
- except Exception as e:
- logging.error(e)
-
-
-def LogFull(results, test_type, test_package, annotation=None,
- flakiness_server=None):
- """Log the tests results for the test suite.
-
- The results will be logged three different ways:
- 1. Log to stdout.
- 2. Log to local files for aggregating multiple test steps
- (on buildbots only).
- 3. Log to flakiness dashboard (on buildbots only).
-
- Args:
- results: An instance of TestRunResults object.
- test_type: Type of the test (e.g. 'Instrumentation', 'Unit test', etc.).
- test_package: Test package name (e.g. 'ipc_tests' for gtests,
- 'ContentShellTest' for instrumentation tests)
- annotation: If instrumenation test type, this is a list of annotations
- (e.g. ['Smoke', 'SmallTest']).
- flakiness_server: If provider, upload the results to flakiness dashboard
- with this URL.
- """
- if not results.DidRunPass():
- logging.critical('*' * 80)
- logging.critical('Detailed Logs')
- logging.critical('*' * 80)
- for line in results.GetLogs().splitlines():
- logging.critical(line)
- logging.critical('*' * 80)
- logging.critical('Summary')
- logging.critical('*' * 80)
- for line in results.GetGtestForm().splitlines():
- logging.critical(line)
- logging.critical('*' * 80)
-
- if os.environ.get('BUILDBOT_BUILDERNAME'):
- # It is possible to have multiple buildbot steps for the same
- # instrumenation test package using different annotations.
- if annotation and len(annotation) == 1:
- suite_name = annotation[0]
- else:
- suite_name = test_package
- _LogToFile(results, test_type, suite_name)
-
- if flakiness_server:
- _LogToFlakinessDashboard(results, test_type, test_package,
- flakiness_server)
« no previous file with comments | « build/android/pylib/results/json_results_test.py ('k') | build/android/pylib/screenshot.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698