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

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

Issue 9150027: Add configuration information to test.dart output. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: 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
« no previous file with comments | « no previous file | tools/testing/dart/test_runner.dart » ('j') | tools/testing/dart/test_runner.dart » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/testing/dart/test_progress.dart
diff --git a/tools/testing/dart/test_progress.dart b/tools/testing/dart/test_progress.dart
index d2fbfce308f18a92541d019ce81527b52412cc77..e7ce73425db74c2b23a5219104d083df168a72e2 100644
--- a/tools/testing/dart/test_progress.dart
+++ b/tools/testing/dart/test_progress.dart
@@ -69,7 +69,8 @@ class ProgressIndicator {
for (int i = 0; i < 20 && i < _tests.length; i++) {
var name = _tests[i].displayName;
var duration = _tests[i].output.time;
- print('${duration} - $name');
+ var configuration = _tests[i].configurationString;
+ print('${duration} - $configuration $name');
}
}
}
@@ -112,7 +113,7 @@ class ProgressIndicator {
void _printFailureOutput(TestCase test) {
List<String> output = new List<String>();
output.add('');
- output.add('FAILED: ${test.displayName}');
+ output.add('FAILED: ${test.configurationString} ${test.displayName}');
StringBuffer expected = new StringBuffer();
expected.add('Expected: ');
for (var expectation in test.expectedOutcomes) {
@@ -279,7 +280,7 @@ class LineProgressIndicator extends ProgressIndicator {
if (test.output.unexpectedOutput) {
status = 'fail';
}
- print('Done ${test.displayName}: $status');
+ print('Done ${test.configurationString} ${test.displayName}: $status');
}
}
@@ -289,7 +290,7 @@ class VerboseProgressIndicator extends ProgressIndicator {
: super(startTime, printTiming);
void _printStartProgress(TestCase test) {
- print('Starting ${test.displayName}...');
+ print('Starting ${test.configurationString} ${test.displayName}...');
}
void _printDoneProgress(TestCase test) {
@@ -297,7 +298,7 @@ class VerboseProgressIndicator extends ProgressIndicator {
if (test.output.unexpectedOutput) {
status = 'fail';
}
- print('Done ${test.displayName}: $status');
+ print('Done ${test.configurationString} ${test.displayName}: $status');
}
}
@@ -327,7 +328,7 @@ class BuildbotProgressIndicator extends ProgressIndicator {
status = 'fail';
}
var percent = ((_completedTests() / _foundTests) * 100).toInt().toString();
- print('Done ${test.displayName}: $status');
+ print('Done ${test.configurationString} ${test.displayName}: $status');
print('@@@STEP_CLEAR@@@');
print('@@@STEP_TEXT@ $percent% +$_passedTests -$_failedTests @@@');
}
« no previous file with comments | « no previous file | tools/testing/dart/test_runner.dart » ('j') | tools/testing/dart/test_runner.dart » ('J')

Powered by Google App Engine
This is Rietveld 408576698