Fix issue 2933 - if an exception is thrown before we start the first test, but after the testcase i…

pkg/unittest/unittest.dart

Index: pkg/unittest/unittest.dart
===================================================================
--- pkg/unittest/unittest.dart	(revision 11994)
+++ pkg/unittest/unittest.dart	(working copy)
@@ -662,13 +662,15 @@
 void _reportTestError(String msg, String trace) {
  if (_currentTest < _tests.length) {
     final testCase = _tests[_currentTest];
-    testCase.error(msg, trace);
-    if (testCase.callbackFunctionsOutstanding > 0) {
-      _nextTestCase();
+    if (testCase.running) {

[Siggi Cherem (dart-lang), 2012/09/07 17:13:23]

seems that this means that also tests that fail after they are done running
(e.g. when an async test creates more callbacks than expected and some callbacks
happen afterwards), then we'll also mark the error as an uncaught top level
error. Is that what we want there too? or do we need to have a different state
for before/after running?

[gram, 2012/09/07 17:40:53]

I thought about that. I have mixed feelings about this. In a way, 'uncaught
error' seems like a valid way of handling this, and can possibly lead to us
simplifying the code later on, as right now we have some extra checks to deal
with the fact that an error might happen on a test that is not the currently
running test. As it is, we recently made the change where a later failure will
flip the test from pass/fail to error.

OTOH I also feel that this side effect is not really desirable in that the fix
should really address just the issue at hand. As such I have come up with an
alternative that is more specific to this issue. PTAL.

+      testCase.error(msg, trace);
+      if (testCase.callbackFunctionsOutstanding > 0) {
+        _nextTestCase();
+      }
+      return;
     }
-  } else {
-    _uncaughtErrorMessage = "$msg: $trace";
   }
+  _uncaughtErrorMessage = "$msg: $trace";
 }
 
 /** Runs [callback] at the end of the event loop. */