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

Unified Diff: tools/testing/dart/test_runner.dart

Issue 9315036: Change output of test.dart --list option, so it can be parsed automatically. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: more changes Created 8 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: tools/testing/dart/test_runner.dart
diff --git a/tools/testing/dart/test_runner.dart b/tools/testing/dart/test_runner.dart
index d60cf34bd963d08455e1ef3728745faeeb680a7b..d4a78e6c38704f18bfcaf99991f5dd8c95cc1075 100644
--- a/tools/testing/dart/test_runner.dart
+++ b/tools/testing/dart/test_runner.dart
@@ -495,7 +495,9 @@ class ProcessQueue {
new Process.start('/bin/rm', ['-rf', _temporaryDirectory]);
deletion.exitHandler = (int exitCode) {
if (exitCode == 0) {
- print('\nTemporary directory $_temporaryDirectory deleted.');
+ if (!_listTests) { // Output of --list option is used by scripts.
+ print('\nTemporary directory $_temporaryDirectory deleted.');
+ }
} else {
print('\nDeletion of temp dir $_temporaryDirectory failed.');
}
@@ -538,7 +540,11 @@ class ProcessQueue {
TestCase test = _tests.removeFirst();
if (_verbose) print(test.commandLine);
if (_listTests) {
- print(test.commandLine);
+ final String tab = '\t';
+ String outcomes =
+ Strings.join(new List.from(test.expectedOutcomes), ',');
+ print(test.displayName + tab + outcomes + tab + test.isNegative +
+ tab + Strings.join(test.arguments, tab));
return;
}
_progress.start(test);

Powered by Google App Engine
This is Rietveld 408576698