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

Side by Side Diff: tools/test-runtime.dart

Issue 12302016: Refactoring of ProgressIndicator (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 9 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 | « tools/test.dart ('k') | tools/testing/dart/test_progress.dart » ('j') | 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) 2012, the Dart project authors. Please see the AUTHORS file 2 // Copyright (c) 2012, 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 // TODO(ager): Get rid of this version of test.dart when we don't have 6 // TODO(ager): Get rid of this version of test.dart when we don't have
7 // to worry about the special runtime checkout anymore. 7 // to worry about the special runtime checkout anymore.
8 // This file is identical to test.dart with test suites in the 8 // This file is identical to test.dart with test suites in the
9 // directories samples, client, compiler, and utils removed. 9 // directories samples, client, compiler, and utils removed.
10 10
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 } 92 }
93 } 93 }
94 } 94 }
95 95
96 void allTestsFinished() { 96 void allTestsFinished() {
97 DebugLogger.close(); 97 DebugLogger.close();
98 } 98 }
99 99
100 var maxBrowserProcesses = maxProcesses; 100 var maxBrowserProcesses = maxProcesses;
101 101
102 var eventListener = [];
103 if (progressIndicator != 'silent') {
104 var printFailures = true;
105 var formatter = new Formatter();
106 if (progressIndicator == 'color') {
107 progressIndicator = 'compact';
108 formatter = new ColorFormatter();
109 }
110 if (progressIndicator == 'diff') {
111 progressIndicator = 'compact';
112 formatter = new ColorFormatter();
113 printFailures = false;
114 eventListener.add(new StatusFileUpdatePrinter());
115 }
116 eventListener.add(new SummaryPrinter());
117 eventListener.add(new FlakyLogWriter());
118 if (printFailures) {
119 eventListener.add(new TestFailurePrinter(formatter));
120 }
121 eventListener.add(new ProgressIndicator.fromName(progressIndicator,
122 startTime,
123 formatter));
124 if (printTiming) {
125 eventListener.add(new TimingPrinter(startTime));
126 }
127 eventListener.add(new SkippedCompilationsPrinter());
128 eventListener.add(new LeftOverTempDirPrinter());
129 }
130 eventListener.add(new ExitCodeSetter());
131
102 // Start process queue. 132 // Start process queue.
103 new ProcessQueue( 133 new ProcessQueue(
104 maxProcesses, 134 maxProcesses,
105 maxBrowserProcesses, 135 maxBrowserProcesses,
106 progressIndicator,
107 startTime, 136 startTime,
108 printTiming,
109 testSuites, 137 testSuites,
138 eventListener,
110 allTestsFinished, 139 allTestsFinished,
111 verbose, 140 verbose,
112 listTests); 141 listTests);
113 } 142 }
OLDNEW
« no previous file with comments | « tools/test.dart ('k') | tools/testing/dart/test_progress.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698