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

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

Issue 26998002: Remove pixel/layout tests in tests/utils + special casing in the testing scripts for pixel/layout t… (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 2 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/testing/dart/browser_test.dart ('k') | 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) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, 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:async"; 7 import "dart:async";
8 import "dart:io"; 8 import "dart:io";
9 import "dart:io" as io; 9 import "dart:io" as io;
10 import "http_server.dart" as http_server; 10 import "http_server.dart" as http_server;
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
100 if (!commandOutput.diagnostics.isEmpty) { 100 if (!commandOutput.diagnostics.isEmpty) {
101 String prefix = 'diagnostics:'; 101 String prefix = 'diagnostics:';
102 for (var s in commandOutput.diagnostics) { 102 for (var s in commandOutput.diagnostics) {
103 output.add('$prefix ${s}'); 103 output.add('$prefix ${s}');
104 prefix = ' '; 104 prefix = ' ';
105 } 105 }
106 } 106 }
107 if (!commandOutput.stdout.isEmpty) { 107 if (!commandOutput.stdout.isEmpty) {
108 output.add(''); 108 output.add('');
109 output.add('stdout:'); 109 output.add('stdout:');
110 if (command.isPixelTest) { 110 output.addAll(getLinesWithoutCarriageReturn(commandOutput.stdout));
111 output.add('DRT pixel test failed! stdout is not printed because it '
112 'contains binary data!');
113 } else {
114 output.addAll(getLinesWithoutCarriageReturn(commandOutput.stdout));
115 }
116 } 111 }
117 if (!commandOutput.stderr.isEmpty) { 112 if (!commandOutput.stderr.isEmpty) {
118 output.add(''); 113 output.add('');
119 output.add('stderr:'); 114 output.add('stderr:');
120 output.addAll(getLinesWithoutCarriageReturn(commandOutput.stderr)); 115 output.addAll(getLinesWithoutCarriageReturn(commandOutput.stderr));
121 } 116 }
122 } 117 }
123 } 118 }
124 if (test is BrowserTestCase) { 119 if (test is BrowserTestCase) {
125 // Additional command for rerunning the steps locally after the fact. 120 // Additional command for rerunning the steps locally after the fact.
(...skipping 431 matching lines...) Expand 10 before | Expand all | Expand 10 after
557 return new VerboseProgressIndicator(startTime); 552 return new VerboseProgressIndicator(startTime);
558 case 'status': 553 case 'status':
559 return new ProgressIndicator(startTime); 554 return new ProgressIndicator(startTime);
560 case 'buildbot': 555 case 'buildbot':
561 return new BuildbotProgressIndicator(startTime); 556 return new BuildbotProgressIndicator(startTime);
562 default: 557 default:
563 assert(false); 558 assert(false);
564 break; 559 break;
565 } 560 }
566 } 561 }
OLDNEW
« no previous file with comments | « tools/testing/dart/browser_test.dart ('k') | tools/testing/dart/test_runner.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698