When we have excess callbacks, throw instead of calling error() so that we

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 414 matching lines @@
     return guardAsync(
         () {
           ++_actualCalls;
           if (_shouldCallBack()) {
             return _callback(arg1, arg2);
           }
         },
         _after, _testNum);
   }
 
-  /** Returns false if we exceded the number of expected calls. */
+  /** Returns true if we have not exceeded the number of expected calls. */
   bool _checkCallCount() {
     if (_actualCalls > _expectedCalls) {
-      _testCase.error('Callback ${_id}called more times than expected '
-             '($_actualCalls > $_expectedCalls).', '');
+      throw new TestFailure('Callback ${_id}called more times than expected '
+                            '($_actualCalls > $_expectedCalls).');
       return false;

[Jennifer Messerly, 2013/02/27 22:42:55]

remove?

     }
     return true;
   }
 }
 
 /**
  * Indicate that [callback] is expected to be called a [count] number of times
  * (by default 1). The unittest framework will wait for the callback to run the
  * specified [count] times before it continues with the following test.  Using
  * [_expectAsync] will also ensure that errors that occur within [callback] are
@@ skipping 430 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 dynamic TestFunction();