Cleaning up unittest configurations

pkg/unittest/lib/unittest.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.
 
 /**
  * A library for writing dart unit tests.
  *
  * To import this library, use the pub package manager.
  * Create a pubspec.yaml file in your project and add
  * a dependency on unittest with the following lines:
@@ skipping 804 matching lines @@
         testCase.callbackFunctionsOutstanding > 0) return;
     _currentTest++;
   }
 
   _completeTests();
 }
 
 /** Publish results on the page and notify controller. */
 _completeTests() {
   if (!_initialized) return;
-  int testsPassed_ = 0;
-  int testsFailed_ = 0;
-  int testsErrors_ = 0;
+  int passed = 0;
+  int failed = 0;
+  int errors = 0;
 
   for (TestCase t in _tests) {
     switch (t.result) {
-      case PASS:  testsPassed_++; break;
-      case FAIL:  testsFailed_++; break;
-      case ERROR: testsErrors_++; break;
+      case PASS:  passed++; break;
+      case FAIL:  failed++; break;
+      case ERROR: errors++; break;
     }
   }
-  _config.onDone(testsPassed_, testsFailed_, testsErrors_, _tests,
-      _uncaughtErrorMessage);
+  _config.onSummary(passed, failed, errors, _tests, _uncaughtErrorMessage);
+  _config.onDone(passed > 0 && failed == 0 && errors == 0 &&
+      _uncaughtErrorMessage == null);
   _initialized = false;
 }
 
 String _fullSpec(String spec) {
   if (spec == null) return '$_currentGroup';
   return _currentGroup != '' ? '$_currentGroup$groupSep$spec' : spec;
 }
 
 void fail(String message) {
   throw new ExpectException(message);
@@ skipping 52 matching lines @@
 }
 
 /** Enable a test by ID. */
 void enableTest(int testId) => _setTestEnabledState(testId, true);
 
 /** Disable a test by ID. */
 void disableTest(int testId) => _setTestEnabledState(testId, false);
 
 /** Signature for a test function. */
 typedef void TestFunction();