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

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

Issue 23926012: Revert 221736 "[android] Adds constants.GetOutDirectory() and co..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: 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 unified diff | Download patch | Annotate | Revision Log
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 12
13 import flakiness_dashboard_results_uploader 13 import flakiness_dashboard_results_uploader
14 14
15 15
16 def _LogToFile(results, test_type, suite_name): 16 def _LogToFile(results, test_type, suite_name):
17 """Log results to local files which can be used for aggregation later.""" 17 """Log results to local files which can be used for aggregation later."""
18 log_file_path = os.path.join(constants.GetOutDirectory(), 'test_logs') 18 log_file_path = os.path.join(constants.DIR_SOURCE_ROOT, 'out',
19 constants.GetBuildType(), 'test_logs')
19 if not os.path.exists(log_file_path): 20 if not os.path.exists(log_file_path):
20 os.mkdir(log_file_path) 21 os.mkdir(log_file_path)
21 full_file_name = os.path.join( 22 full_file_name = os.path.join(
22 log_file_path, re.sub('\W', '_', test_type).lower() + '.log') 23 log_file_path, re.sub('\W', '_', test_type).lower() + '.log')
23 if not os.path.exists(full_file_name): 24 if not os.path.exists(full_file_name):
24 with open(full_file_name, 'w') as log_file: 25 with open(full_file_name, 'w') as log_file:
25 print >> log_file, '\n%s results for %s build %s:' % ( 26 print >> log_file, '\n%s results for %s build %s:' % (
26 test_type, os.environ.get('BUILDBOT_BUILDERNAME'), 27 test_type, os.environ.get('BUILDBOT_BUILDERNAME'),
27 os.environ.get('BUILDBOT_BUILDNUMBER')) 28 os.environ.get('BUILDBOT_BUILDNUMBER'))
28 logging.info('Writing results to %s.' % full_file_name) 29 logging.info('Writing results to %s.' % full_file_name)
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
100 # instrumenation test package using different annotations. 101 # instrumenation test package using different annotations.
101 if annotation and len(annotation) == 1: 102 if annotation and len(annotation) == 1:
102 suite_name = annotation[0] 103 suite_name = annotation[0]
103 else: 104 else:
104 suite_name = test_package 105 suite_name = test_package
105 _LogToFile(results, test_type, suite_name) 106 _LogToFile(results, test_type, suite_name)
106 107
107 if flakiness_server: 108 if flakiness_server:
108 _LogToFlakinessDashboard(results, test_type, test_package, 109 _LogToFlakinessDashboard(results, test_type, test_package,
109 flakiness_server) 110 flakiness_server)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698