Make Completers asynchronous.

pkg/scheduled_test/test/substitute_future_test.dart

Index: pkg/scheduled_test/test/substitute_future_test.dart
diff --git a/pkg/scheduled_test/test/substitute_future_test.dart b/pkg/scheduled_test/test/substitute_future_test.dart
index 7202c86092abe9efbf0a32bf9109204e75ffdb0f..5cffb4b6b62ee494a1071529b935ee6d08898573 100644
--- a/pkg/scheduled_test/test/substitute_future_test.dart
+++ b/pkg/scheduled_test/test/substitute_future_test.dart
@@ -148,7 +148,9 @@ void main() {
     var completer = new Completer();
     var future = new SubstituteFuture(completer.future);
     completer.complete('success');
-    expect(() => future.substitute(new Future.value()),
-        throwsStateError);
+    future.then(expectAsync1((_) {
+      expect(() => future.substitute(new Future.value()),
+          throwsStateError);
+    }));

[nweiz, 2013/05/07 00:11:52]

[expectAsync*] is usually less expressive than [expect]/[completes]. In this
case, you could write:

    expect(future.then((_) => future.substitute(new Future.value())),
        throwsStateError);

since [throwsA] is Future-aware by default.

Also, add a comment here explaining why this needs to be async.

[floitsch, 2013/05/13 11:19:25]

done in CL 15128002.

   });
 }