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

Side by Side Diff: build/android/pylib/results/json_results.py

Issue 2548503002: Revert of (Reland) Insert logcat as part of test result for android instrumentation tests. (Closed)
Patch Set: Created 4 years 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
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 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 import collections 5 import collections
6 import itertools 6 import itertools
7 import json 7 import json
8 8
9 from pylib.base import base_test_result 9 from pylib.base import base_test_result
10 10
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
95 results_iterable = test_run_result.GetAll() 95 results_iterable = test_run_result.GetAll()
96 96
97 for r in results_iterable: 97 for r in results_iterable:
98 result_dict = { 98 result_dict = {
99 'status': status_as_string(r.GetType()), 99 'status': status_as_string(r.GetType()),
100 'elapsed_time_ms': r.GetDuration(), 100 'elapsed_time_ms': r.GetDuration(),
101 'output_snippet': r.GetLog(), 101 'output_snippet': r.GetLog(),
102 'losless_snippet': '', 102 'losless_snippet': '',
103 'output_snippet_base64:': '', 103 'output_snippet_base64:': '',
104 'tombstones': r.GetTombstones() or '', 104 'tombstones': r.GetTombstones() or '',
105 'logcat_url': r.GetLogcatUrl() or '',
106 } 105 }
107 iteration_data[r.GetName()].append(result_dict) 106 iteration_data[r.GetName()].append(result_dict)
108 107
109 all_tests = all_tests.union(set(iteration_data.iterkeys())) 108 all_tests = all_tests.union(set(iteration_data.iterkeys()))
110 per_iteration_data.append(iteration_data) 109 per_iteration_data.append(iteration_data)
111 110
112 return { 111 return {
113 'global_tags': [], 112 'global_tags': [],
114 'all_tests': sorted(list(all_tests)), 113 'all_tests': sorted(list(all_tests)),
115 # TODO(jbudorick): Add support for disabled tests within base_test_result. 114 # TODO(jbudorick): Add support for disabled tests within base_test_result.
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
158 testsuite_runs = json_results['per_iteration_data'] 157 testsuite_runs = json_results['per_iteration_data']
159 for testsuite_run in testsuite_runs: 158 for testsuite_run in testsuite_runs:
160 for test, test_runs in testsuite_run.iteritems(): 159 for test, test_runs in testsuite_run.iteritems():
161 results_list.extend( 160 results_list.extend(
162 [base_test_result.BaseTestResult(test, 161 [base_test_result.BaseTestResult(test,
163 string_as_status(tr['status']), 162 string_as_status(tr['status']),
164 duration=tr['elapsed_time_ms']) 163 duration=tr['elapsed_time_ms'])
165 for tr in test_runs]) 164 for tr in test_runs])
166 return results_list 165 return results_list
167 166
OLDNEW
« no previous file with comments | « build/android/pylib/local/device/local_device_instrumentation_test_run.py ('k') | build/android/test_runner.pydeps » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698