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

Side by Side Diff: build/android/pylib/utils/report_results.py

Issue 727543003: [Android] Fix new pylint errors in build/android/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 6 years, 1 month 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 unified diff | Download patch
« no previous file with comments | « build/android/pylib/utils/parallelizer.py ('k') | build/android/symbolize.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 """Module containing utility functions for reporting results.""" 5 """Module containing utility functions for reporting results."""
6 6
7 import logging 7 import logging
8 import os 8 import os
9 import re 9 import re
10 10
11 from pylib import constants 11 from pylib import constants
12 from pylib.utils import flakiness_dashboard_results_uploader 12 from pylib.utils import flakiness_dashboard_results_uploader
13 13
14 14
15 def _LogToFile(results, test_type, suite_name): 15 def _LogToFile(results, test_type, suite_name):
16 """Log results to local files which can be used for aggregation later.""" 16 """Log results to local files which can be used for aggregation later."""
17 log_file_path = os.path.join(constants.GetOutDirectory(), 'test_logs') 17 log_file_path = os.path.join(constants.GetOutDirectory(), 'test_logs')
18 if not os.path.exists(log_file_path): 18 if not os.path.exists(log_file_path):
19 os.mkdir(log_file_path) 19 os.mkdir(log_file_path)
20 full_file_name = os.path.join( 20 full_file_name = os.path.join(
21 log_file_path, re.sub('\W', '_', test_type).lower() + '.log') 21 log_file_path, re.sub(r'\W', '_', test_type).lower() + '.log')
22 if not os.path.exists(full_file_name): 22 if not os.path.exists(full_file_name):
23 with open(full_file_name, 'w') as log_file: 23 with open(full_file_name, 'w') as log_file:
24 print >> log_file, '\n%s results for %s build %s:' % ( 24 print >> log_file, '\n%s results for %s build %s:' % (
25 test_type, os.environ.get('BUILDBOT_BUILDERNAME'), 25 test_type, os.environ.get('BUILDBOT_BUILDERNAME'),
26 os.environ.get('BUILDBOT_BUILDNUMBER')) 26 os.environ.get('BUILDBOT_BUILDNUMBER'))
27 logging.info('Writing results to %s.' % full_file_name) 27 logging.info('Writing results to %s.' % full_file_name)
28 28
29 logging.info('Writing results to %s.' % full_file_name) 29 logging.info('Writing results to %s.' % full_file_name)
30 with open(full_file_name, 'a') as log_file: 30 with open(full_file_name, 'a') as log_file:
31 shortened_suite_name = suite_name[:25] + (suite_name[25:] and '...') 31 shortened_suite_name = suite_name[:25] + (suite_name[25:] and '...')
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
103 # instrumenation test package using different annotations. 103 # instrumenation test package using different annotations.
104 if annotation and len(annotation) == 1: 104 if annotation and len(annotation) == 1:
105 suite_name = annotation[0] 105 suite_name = annotation[0]
106 else: 106 else:
107 suite_name = test_package 107 suite_name = test_package
108 _LogToFile(results, test_type, suite_name) 108 _LogToFile(results, test_type, suite_name)
109 109
110 if flakiness_server: 110 if flakiness_server:
111 _LogToFlakinessDashboard(results, test_type, test_package, 111 _LogToFlakinessDashboard(results, test_type, test_package,
112 flakiness_server) 112 flakiness_server)
OLDNEW
« no previous file with comments | « build/android/pylib/utils/parallelizer.py ('k') | build/android/symbolize.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698