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

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

Issue 9015005: Add dartc junit tests to test.dart. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 9 years 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 903b90a494ec87783f1052f5366011670f9ee9d9..6cbccee45e91ae271aa09d20847b3b9a5840ebce 100644
--- a/tools/testing/dart/test_runner.dart
+++ b/tools/testing/dart/test_runner.dart
@@ -128,6 +128,7 @@ class TestOutput {
bool get unexpectedOutput() => !testCase.expectedOutcomes.contains(result);
bool get hasCrashed() {
+ print('exit code: $exitCode');
Bill Hesse 2011/12/29 18:32:24 Remove this line.
Bill Hesse 2011/12/30 16:04:33 Done.
if (new Platform().operatingSystem() == 'windows') {
// The VM uses std::abort to terminate on asserts.
// std::abort terminates with exit code 3 on Windows.
@@ -472,7 +473,8 @@ class ProcessQueue {
oldCallback(test_arg);
};
test.completedHandler = wrapper;
- if (test.configuration['component'] == 'dartc') {
+ if (test.configuration['component'] == 'dartc' &&
+ test.displayName != 'dartc/junit_tests') {
_ensureDartcBatchRunnersStarted(test.executablePath);
_getDartcBatchRunnerProcess().startTest(test);
} else {

Powered by Google App Engine
This is Rietveld 408576698