Unit test improvements:

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 59 matching lines @@
       String uncaughtError) {
     _result = buildStatusString(passed, failed, errors, results,
         count: count, setup: setup, teardown: teardown,
         uncaughtError: uncaughtError);
   }
 
   void onDone(bool success) {
     _port.send(_result);
   }
 }
+
+foo() {
+  ++_testconfig.count;
+}
+
 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') {
       test(testName, () { throw new Exception('Fail.'); });
     } else if (testName == 'group name test') {
@@ skipping 15 matching lines @@
       });
     } else if (testName == 'setup and teardown test') {
       group('a', () {
         setUp(() { _testconfig.setup = 'setup'; });
         tearDown(() { _testconfig.teardown = 'teardown'; });
         test(testName, () {});
       });
     } else if (testName == 'correct callback test') {
       test(testName,
         () =>_defer(expectAsync0((){ ++_testconfig.count;})));
-    } else if (testName == 'excess callback test') {
+    } else if (testName == 'excess callback test with callback name') {
       test(testName, () {
-        var _callback = expectAsync0((){ ++_testconfig.count;});
+        var _callback = expectAsync0(foo);
         _defer(_callback);
         _defer(_callback);
       });
     } else if (testName == 'completion test') {
       test(testName, () {
              var _callback;
              _callback = expectAsyncUntil0(() {
                if (++_testconfig.count < 10) {
                  _defer(_callback);
                }
@@ skipping 126 matching lines @@
 main() {
   tests = [
     'single correct test',
     'single failing test',
     'exception test',
     'group name test',
     'setup test',
     'teardown test',
     'setup and teardown test',
     'correct callback test',
-    'excess callback test',
+    'excess callback test with callback name',
     'completion test',
     'async exception test',
     'late exception test',
     'middle exception test',
     'async setup/teardown test'
   ];
 
   expected = [
     buildStatusString(1, 0, 0, tests[0]),
     buildStatusString(0, 1, 0, tests[1],
         message: 'Expected: <5> but: was <4>.'),
     buildStatusString(0, 1, 0, tests[2], message: 'Caught Exception: Fail.'),
     buildStatusString(2, 0, 0, 'a a::a b b'),
     buildStatusString(1, 0, 0, 'a ${tests[4]}', count: 0, setup: 'setup'),
     buildStatusString(1, 0, 0, 'a ${tests[5]}', count: 0, setup: '',
         teardown: 'teardown'),
     buildStatusString(1, 0, 0, 'a ${tests[6]}', count: 0,
         setup: 'setup', teardown: 'teardown'),
     buildStatusString(1, 0, 0, tests[7], count: 1),
     buildStatusString(0, 0, 1, tests[8], count: 1,
-        message: 'Callback called more times than expected (2 > 1).'),
+        message: 'Callback foo called more times than expected (2 > 1).'),
     buildStatusString(1, 0, 0, tests[9], count: 10),
     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'),
     buildStatusString(2, 0, 3, 
         'good setup/good teardown foo1::'
         'good setup/bad teardown foo2:good setup/bad teardown '
         'foo2: Test teardown failed: Failed to complete tearDown:'
         'bad setup/good teardown foo3:bad setup/good teardown '
         'foo3: Test setup failed: Failed to complete setUp:'
         'bad setup/bad teardown foo4:bad setup/bad teardown '
         'foo4: Test teardown failed: Failed to complete tearDown:'
         'post groups')
   ];
 
   actual = [];
 
   nextTest(0);
 }