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

Side by Side Diff: tools/testing/dart/test_progress.dart

Issue 12379041: Quote all arguments when printing a Command (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 | « no previous file | tools/testing/dart/test_runner.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 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library test_progress; 5 library test_progress;
6 6
7 import "dart:io"; 7 import "dart:io";
8 import "dart:io" as io; 8 import "dart:io" as io;
9 import "http_server.dart" as http_server; 9 import "http_server.dart" as http_server;
10 import "status_file_parser.dart"; 10 import "status_file_parser.dart";
(...skipping 192 matching lines...) Expand 10 before | Expand all | Expand 10 after
203 if (test is BrowserTestCase) { 203 if (test is BrowserTestCase) {
204 // Additional command for rerunning the steps locally after the fact. 204 // Additional command for rerunning the steps locally after the fact.
205 var command = 205 var command =
206 test.configuration["_servers_"].httpServerCommandline(); 206 test.configuration["_servers_"].httpServerCommandline();
207 output.add('To retest, run: $command'); 207 output.add('To retest, run: $command');
208 } 208 }
209 for (Command c in test.commands) { 209 for (Command c in test.commands) {
210 output.add(''); 210 output.add('');
211 String message = (c == test.commands.last 211 String message = (c == test.commands.last
212 ? "Command line" : "Compilation command"); 212 ? "Command line" : "Compilation command");
213 output.add('$message: ${c.commandLine}'); 213 output.add('$message: $c');
214 } 214 }
215 return output; 215 return output;
216 } 216 }
217 217
218 void _printFailureSummary() { 218 void _printFailureSummary() {
219 for (String line in _failureSummary) { 219 for (String line in _failureSummary) {
220 print(line); 220 print(line);
221 } 221 }
222 print(''); 222 print('');
223 } 223 }
(...skipping 267 matching lines...) Expand 10 before | Expand all | Expand 10 after
491 print(''); 491 print('');
492 print('$config:'); 492 print('$config:');
493 statuses.sort((a, b) => a.compareTo(b)); 493 statuses.sort((a, b) => a.compareTo(b));
494 for (String status in statuses) { 494 for (String status in statuses) {
495 print(' $status'); 495 print(' $status');
496 } 496 }
497 }); 497 });
498 _printStatus(); 498 _printStatus();
499 } 499 }
500 } 500 }
OLDNEW
« no previous file with comments | « no previous file | tools/testing/dart/test_runner.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698