Added color coding to test results in VM config.

pkg/unittest/lib/src/config.dart

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 part of unittest;
 
 // A custom failure handler for [expect] that routes expect failures
 // to the config.
 class _ExpectFailureHandler extends DefaultFailureHandler {
   Configuration _config;
@@ skipping 142 matching lines @@
       _testLogBuffer.add(reason);
       try {
         throw '';
       } catch (_, stack) {
         _testLogBuffer.add(stack);
       }
     }
   }
   
   /**
+   * Format a test result.
+   */
+  String formatResult(TestCase testCase) {
+    var result = new StringBuffer();
+    result.write(testCase.result.toUpperCase());
+    result.write(": ");
+    result.write(testCase.description);
+    result.write("\n");
+
+    if (testCase.message != '') {
+      result.write(_indent(testCase.message));
+      result.write("\n");
+    }
+
+    if (testCase.stackTrace != null && testCase.stackTrace != '') {
+      result.write(_indent(testCase.stackTrace));
+      result.write("\n");
+    }
+    return result.toString();
+  }
+
+  /**
    * Called with the result of all test cases. The default implementation prints
    * the result summary using the built-in [print] command. Browser tests
    * commonly override this to reformat the output.
    *
    * When [uncaughtError] is not null, it contains an error that occured outside
    * of tests (e.g. setting up the test).
    */
   void onSummary(int passed, int failed, int errors, List<TestCase> results,
       String uncaughtError) {
     // Print each test's result.
     for (final t in results) {
-      var resultString = "${t.result}".toUpperCase();
-      print('$resultString: ${t.description}');
-
-      if (t.message != '') {
-        print(_indent(t.message));
-      }
-
-      if (t.stackTrace != null && t.stackTrace != '') {
-        print(_indent(t.stackTrace));
-      }
+      print(formatResult(t));

[Siggi Cherem (dart-lang), 2013/05/07 20:55:01]

just to make sure, this is not affecting compact_vm_config, right? (we override
onSummary there, I believe).

     }
 
     // Show the summary.
     print('');
 
     if (passed == 0 && failed == 0 && errors == 0 && uncaughtError == null) {
       print('No tests found.');
       // This is considered a failure too.
     } else if (failed == 0 && errors == 0 && uncaughtError == null) {
       print('All $passed tests passed.');
@@ skipping 33 matching lines @@
   // Currently e.message works in dartium, but not in dartc.
   void handleExternalError(e, String message, [String stack = '']) =>
       _reportTestError('$message\nCaught $e', stack);
 
   _postMessage(String message) {
     // In dart2js browser tests, the JavaScript-based test controller
     // intercepts calls to print and listens for "secret" messages.
     print(message);
   }
 }