Dartest CL - Please review

client/testing/unittest/shared.dart

 // Copyright (c) 2011, 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.
 
 /**
  * Description text of the current test group. If multiple groups are nested,
  * this will contain all of their text concatenated.
  */
 String _currentGroup = '';
 
 /** Tests executed in this suite. */
 List<TestCase> _tests;
 
+/** 
+ * Callback used to run tests. Entrypoints can replace this with their own
+ * if they want. 
+ */
+Function _testRunner;
+
 /** Whether this is run within dartium layout tests. */
 bool _isLayoutTest = false;
 
 /** Current test being executed. */
 int _currentTest = 0;
 
 /** Total number of callbacks that have been executed in the current test. */
 int _callbacksCalled = 0;
 
 final _UNINITIALIZED = 0;
@@ skipping 107 matching lines @@
   } else if (_callbacksCalled > testCase.callbacks) {
     final expected = testCase.callbacks;
     testCase.error(
         'More calls to callbackDone() than expected. '
         + 'Actual: ${_callbacksCalled}, expected: ${expected}', '');
     _state = _UNCAUGHT_ERROR;
   } else if ((_callbacksCalled == testCase.callbacks) &&
       (_state != _RUNNING_TEST)) {
     testCase.pass();
     _currentTest++;
-    _nextBatch();
+    _testRunner();
   }
 }
 
 void forLayoutTests() {
   _isLayoutTest = true;
 }
 
 /** Runs all queued tests, one at a time. */
 _runTests() {
   _platformStartTests();
 
   _platformDefer(() {
     assert (_currentTest == 0);
-    _nextBatch();
+    _testRunner();
   });
 }
 
 /** Runs a single test. */
 _runTest(TestCase testCase) {
   try {
     _callbacksCalled = 0;
     _state = _RUNNING_TEST;
 
     testCase.test();
@@ skipping 64 matching lines @@
 
 /**
  * Lazily initializes the test library if not already initialized.
  */
 _ensureInitialized() {
   if (_state != _UNINITIALIZED) return;
 
   _tests = <TestCase>[];
   _currentGroup = '';
   _state = _READY;
-
+  _testRunner = _nextBatch;
+  
   _platformInitialize();
 
   // Immediately queue the suite up. It will run after a timeout (i.e. after
   // main() has returned).
   _platformDefer(_runTests);
 }
 
 /**
  * Wraps an value and provides an "==" operator that can be used to verify that
  * the value matches a given expectation.
@@ skipping 69 matching lines @@
   /** Error or failure message. */
   String message  = '';
 
   /**
    * One of [_PASS], [_FAIL], or [_ERROR] or [null] if the test hasn't run yet.
    */
   String result;
 
   /** Stack trace associated with this test, or null if it succeeded. */
   String stackTrace;
+  
+  Date startTime;
+  
+  Duration runningTime;
 
   TestCase(this.id, this.description, this.test, this.callbacks);
-
+  
   bool get isComplete() => result != null;
 
   void pass() {
     result = _PASS;
   }
 
   void fail(String message, String stackTrace) {
     result = _FAIL;
     this.message = message;
     this.stackTrace = stackTrace;
   }
 
   void error(String message, String stackTrace) {
     result = _ERROR;
     this.message = message;
     this.stackTrace = stackTrace;
   }
 }
 
 typedef void TestFunction();