TBR: Revrt r21819 until I can figure out why ity is breaking the scheduled_test package.

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 50 matching lines @@
     _result = buildStatusString(passed, failed, errors, results,
         count: count, setup: setup, teardown: teardown,
         uncaughtError: uncaughtError);
   }
 
   void onDone(bool success) {
     _port.send(_result);
   }
 }
 
-makeDelayedSetup(index, s) => () {
-  return new Future.delayed(new Duration(milliseconds:1), () {
-    s.write('l$index U ');
-  });
-};
-
-makeDelayedTeardown(index, s) => () {
-  return new Future.delayed(new Duration(milliseconds:1), () {
-    s.write('l$index D ');
-  });
-};
-
-makeImmediateSetup(index, s) => () {
-  s.write('l$index U ');
-};
-
-makeImmediateTeardown(index, s) => () {
-  s.write('l$index D ');
-};
-
 runTest() {
   port.receive((String testName, sendport) {
     var testConfig = new TestConfiguration(sendport);
     unittestConfiguration = testConfig;
 
     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 211 matching lines @@
       });
       test('foo6', () {
       });
     } else if (testName == 'testCases immutable') {
       test(testName, () {
         expect(() => testCases.clear(), throwsUnsupportedError);
         expect(() => testCases.removeLast(), throwsUnsupportedError);
       });
     } else if (testName == 'runTests without tests') {
       runTests();
-    } else if (testName == 'nested groups setup/teardown') {
-      StringBuffer s = new StringBuffer();
-      group('level 1', () {
-        setUp(makeDelayedSetup(1, s));
-        group('level 2', () {
-          setUp(makeImmediateSetup(2, s));
-          tearDown(makeDelayedTeardown(2, s));
-          group('level 3', () {
-            group('level 4', () {
-              setUp(makeDelayedSetup(4, s));
-              tearDown(makeImmediateTeardown(4, s));
-              group('level 5', () {
-                setUp(makeImmediateSetup(5, s));
-                group('level 6', () {
-                  tearDown(makeDelayedTeardown(6, s));
-                  test('inner', () {});
-                });
-              });
-            });
-          });
-        });
-      });
-      test('after nest', () {
-        expect(s.toString(), "l1 U l2 U l4 U l5 U l6 D l4 D l2 D ");
-      });
     }
   });
 }
 
 main() {
   var tests = {
     'single correct test': buildStatusString(1, 0, 0, 'single correct test'),
     'single failing test': buildStatusString(0, 1, 0, 'single failing test',
         message: 'Expected: <5> but: was <4>.'),
     'exception test': buildStatusString(0, 1, 0, 'exception test',
@@ skipping 36 matching lines @@
         'foo5'),
     'test returning future using Timer': buildStatusString(2, 4, 0,
         'successful::'
         'fail1:Expected: <false> but: was <true>.:'
         'error1:Callback called more times than expected (1).:'
         'fail2:failure:'
         'error2:Callback called more times than expected (1).:'
         'foo6'),
     'testCases immutable':
         buildStatusString(1, 0, 0, 'testCases immutable'),
-    'runTests without tests': buildStatusString(0, 0, 0, null),
-    'nested groups setup/teardown':
-        buildStatusString(2, 0, 0,
-            'level 1 level 2 level 3 level 4 level 5 level 6 inner::after nest')
+    'runTests without tests': buildStatusString(0, 0, 0, null)
   };
 
   tests.forEach((String name, String expected) {
     test(name, () => spawnFunction(runTest)
         .call(name)
         .then((String msg) => expect(msg.trim(), equals(expected))));
     });
 }