Clean up the test srapper code so that it does not use reportTestError, and made that private.

lib/unittest/unittest.dart

Index: lib/unittest/unittest.dart
===================================================================
--- lib/unittest/unittest.dart	(revision 10299)
+++ lib/unittest/unittest.dart	(working copy)
@@ -582,6 +582,8 @@
     _testSetup = null;
     _testTeardown = null;
     body();
+  } catch(var e, var trace) {

[Siggi Cherem (dart-lang), 2012/08/06 21:42:49]

I wonder if somewhere we relied on the fact that his was letting the exceptions
pass through ?

+    _reportTestError(e.toString(), trace == null ? '' : trace.toString());
   } finally {
     // Now that the group is over, restore the previous one.
     _currentGroup = parentGroup;
@@ -660,7 +662,7 @@
  * Utility function that can be used to notify the test framework that an
  *  error was caught outside of this library.

[Siggi Cherem (dart-lang), 2012/08/06 21:42:49]

remove extra space:
*  error
 =>
* error

  */
-void reportTestError(String msg, String trace) {
+void _reportTestError(String msg, String trace) {
  if (_currentTest < _tests.length) {
     final testCase = _tests[_currentTest];
     testCase.error(msg, trace);