Cleaning up unittest configurations

pkg/unittest/test/unittest_test.dart

 // Copyright (c) 2012, 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.
 
 // TODO(gram):
 // Unfortunately I can't seem to test anything that involves timeouts, e.g.
 // insufficient callbacks, because the timeout is controlled externally
 // (test.dart?), and we would need to use a shorter timeout for the inner tests
 // so the outer timeout doesn't fire. So I removed all such tests.
 // I'd like to revisit this at some point.
@@ skipping 43 matching lines @@
 
 class TestConfiguration extends Configuration {
 
   // Some test state that is captured
   int count = 0; // a count of callbacks
   String setup = ''; // the name of the test group setup function, if any
   String teardown = ''; // the name of the test group teardown function, if any
 
   // The port to communicate with the parent isolate
   SendPort _port;
+  String _result;
 
   TestConfiguration(this._port);
 
-  void onDone(int passed, int failed, int errors, List<TestCase> results,
+  void onSummary(int passed, int failed, int errors, List<TestCase> results,
       String uncaughtError) {
-    var result = buildStatusString(passed, failed, errors, results,
+    _result = buildStatusString(passed, failed, errors, results,
         count: count, setup: setup, teardown: teardown,
         uncaughtError: uncaughtError);
-    _port.send(result);
+  }
+
+  void onDone(bool success) {
+    _port.send(_result);
   }
 }
 runTest() {
   port.receive((testName, sendport) {
     configure(_testconfig = new TestConfiguration(sendport));
     if (testName == 'single correct test') {
       test(testName, () => expect(2 + 3, equals(5)));
     } else if (testName == 'single failing test') {
       test(testName, () => expect(2 + 2, equals(5)));
     } else if (testName == 'exception test') {
@@ skipping 120 matching lines @@
     buildStatusString(0, 1, 0, tests[10], message: 'Caught error!'),
     buildStatusString(1, 0, 1, 'testOne', message: 'Callback called after already being marked as done (1).:testTwo:'),
     buildStatusString(2, 1, 0, 'testOne::testTwo:Expected: false but: was <true>.:testThree')
   ];
 
   actual = [];
 
   nextTest(0);
 }