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

Side by Side Diff: tools/test.dart

Issue 14139011: Print the failure summary by default (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 8 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env dart 1 #!/usr/bin/env dart
2 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 2 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
3 // for details. All rights reserved. Use of this source code is governed by a 3 // for details. All rights reserved. Use of this source code is governed by a
4 // BSD-style license that can be found in the LICENSE file. 4 // BSD-style license that can be found in the LICENSE file.
5 5
6 /** 6 /**
7 * This file is the entrypoint of the dart test suite. This suite is used 7 * This file is the entrypoint of the dart test suite. This suite is used
8 * to test: 8 * to test:
9 * 9 *
10 * 1. the dart vm 10 * 1. the dart vm
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 new Path('sdk/lib/_internal/dartdoc'), 72 new Path('sdk/lib/_internal/dartdoc'),
73 new Path('tools/dom/docs'), 73 new Path('tools/dom/docs'),
74 ]; 74 ];
75 75
76 void testConfigurations(List<Map> configurations) { 76 void testConfigurations(List<Map> configurations) {
77 var startTime = new DateTime.now(); 77 var startTime = new DateTime.now();
78 // Extract global options from first configuration. 78 // Extract global options from first configuration.
79 var firstConf = configurations[0]; 79 var firstConf = configurations[0];
80 var maxProcesses = firstConf['tasks']; 80 var maxProcesses = firstConf['tasks'];
81 var progressIndicator = firstConf['progress']; 81 var progressIndicator = firstConf['progress'];
82 // TODO(kustermann): Remove this option once the buildbots don't use it
83 // anymore.
82 var failureSummary = firstConf['failure-summary']; 84 var failureSummary = firstConf['failure-summary'];
83 BuildbotProgressIndicator.stepName = firstConf['step_name']; 85 BuildbotProgressIndicator.stepName = firstConf['step_name'];
84 var verbose = firstConf['verbose']; 86 var verbose = firstConf['verbose'];
85 var printTiming = firstConf['time']; 87 var printTiming = firstConf['time'];
86 var listTests = firstConf['list']; 88 var listTests = firstConf['list'];
87 89
88 if (!firstConf['append_logs']) { 90 if (!firstConf['append_logs']) {
89 var file = new File(TestUtils.flakyFileName()); 91 var file = new File(TestUtils.flakyFileName());
90 if (file.existsSync()) { 92 if (file.existsSync()) {
91 file.deleteSync(); 93 file.deleteSync();
(...skipping 96 matching lines...) Expand 10 before | Expand all | Expand 10 after
188 if (progressIndicator == 'diff') { 190 if (progressIndicator == 'diff') {
189 progressIndicator = 'compact'; 191 progressIndicator = 'compact';
190 formatter = new ColorFormatter(); 192 formatter = new ColorFormatter();
191 printFailures = false; 193 printFailures = false;
192 eventListener.add(new StatusFileUpdatePrinter()); 194 eventListener.add(new StatusFileUpdatePrinter());
193 } 195 }
194 eventListener.add(new SummaryPrinter()); 196 eventListener.add(new SummaryPrinter());
195 eventListener.add(new FlakyLogWriter()); 197 eventListener.add(new FlakyLogWriter());
196 if (printFailures) { 198 if (printFailures) {
197 // The buildbot has it's own failure summary since it needs to wrap it 199 // The buildbot has it's own failure summary since it needs to wrap it
198 // into'@@@'-annotated sections. 200 // into '@@@'-annotated sections.
199 var printFaiureSummary = 201 var printFailureSummary = progressIndicator != 'buildbot';
200 failureSummary && progressIndicator != 'buildbot'; 202 eventListener.add(new TestFailurePrinter(printFailureSummary, formatter));
201 eventListener.add(new TestFailurePrinter(printFaiureSummary, formatter));
202 } 203 }
203 eventListener.add(new ProgressIndicator.fromName(progressIndicator, 204 eventListener.add(new ProgressIndicator.fromName(progressIndicator,
204 startTime, 205 startTime,
205 formatter)); 206 formatter));
206 if (printTiming) { 207 if (printTiming) {
207 eventListener.add(new TimingPrinter(startTime)); 208 eventListener.add(new TimingPrinter(startTime));
208 } 209 }
209 eventListener.add(new SkippedCompilationsPrinter()); 210 eventListener.add(new SkippedCompilationsPrinter());
210 eventListener.add(new LeftOverTempDirPrinter()); 211 eventListener.add(new LeftOverTempDirPrinter());
211 } 212 }
(...skipping 20 matching lines...) Expand all
232 } 233 }
233 234
234 void main() { 235 void main() {
235 var optionsParser = new TestOptionsParser(); 236 var optionsParser = new TestOptionsParser();
236 var configurations = optionsParser.parse(new Options().arguments); 237 var configurations = optionsParser.parse(new Options().arguments);
237 if (configurations != null && configurations.length > 0) { 238 if (configurations != null && configurations.length > 0) {
238 testConfigurations(configurations); 239 testConfigurations(configurations);
239 } 240 }
240 } 241 }
241 242
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698