Fix an issue where test timeouts would cause no more tests to execute.

pkg/unittest/lib/unittest.dart

 // Copyright (c) 2013, 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.
 
 /**
  * Support for writing Dart unit tests.
  *
  * For information on installing and importing this library, see the
  * [unittest package on pub.dartlang.org]
  * (http://pub.dartlang.org/packages/unittest).
@@ skipping 732 matching lines @@
     _completeTests();
   } else {
     final testCase = testCases[_currentTestCaseIndex];
     var f = _guardAsync(testCase._run, null, testCase);
     Timer timer;
     final Duration timeout = unittestConfiguration.timeout;
     if (timeout != null) {
       try {
         timer = new Timer(timeout, () {
           testCase.error("Test timed out after ${timeout.inSeconds} seconds.");
+          Timer.run(_nextTestCase);

[Siggi Cherem (dart-lang), 2013/10/07 21:39:45]

since this is already in a timer callback, it seems ok to just invoke
_nextTestCase()

(remove the extra Timer.run?)

[devoncarew, 2013/10/07 21:45:33]

Done.

         });
       } on UnsupportedError catch (e) {
         if (e.message != "Timer greater than 0.") rethrow;
         // Support running on d8 and jsshell which don't support timers.
       }
     }
     f.whenComplete(() {
       if (timer != null) timer.cancel();
       var now = new DateTime.now().millisecondsSinceEpoch;
       if ((now - _lastBreath) >= BREATH_INTERVAL) {
@@ skipping 88 matching lines @@
 
 /**
  * A flag that controls whether we hide unittest and core library details in
  * exception stacks.
  *
  * Useful to disable when debugging unittest or matcher customizations.
  */
 bool formatStacks = true;
 
 /**
- * A flag that controls whether we try to filter out irrelevant frames from 
+ * A flag that controls whether we try to filter out irrelevant frames from
  * the stack trace. Requires formatStacks to be set.
  */
 bool filterStacks = true;
 
 /**
- * Returns a Trace object from a StackTrace object or a String, or the 
+ * Returns a Trace object from a StackTrace object or a String, or the
  * unchanged input if formatStacks is false;
  */
 Trace _getTrace(stack) {
   Trace trace;
   if (stack == null || !formatStacks) return null;
   if (stack is String) {
     trace = new Trace.parse(stack);
   } else if (stack is StackTrace) {
     trace = new Trace.from(stack);
   } else {
     throw new Exception('Invalid stack type ${stack.runtimeType} for $stack.');
   }
 
   if (!filterStacks) return trace;
 
   // Format the stack trace by removing everything above TestCase._runTest,
   // which is usually going to be irrelevant. Also fold together unittest and
   // core library calls so only the function the user called is visible.
   return new Trace(trace.frames.takeWhile((frame) {
     return frame.package != 'unittest' || frame.member != 'TestCase._runTest';
   })).terse.foldFrames((frame) => frame.package == 'unittest' || frame.isCore);
 }