Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(241)

Unified Diff: test/stream_group_test.dart

Issue 2369953004: Don't expect a `null` return from subscription.cancel. (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « pubspec.yaml ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/stream_group_test.dart
diff --git a/test/stream_group_test.dart b/test/stream_group_test.dart
index 5078dad474e3fef9da9c61b13f2cf0fdcd62ef79..76b69b95e1f59136756063b134624677af783665 100644
--- a/test/stream_group_test.dart
+++ b/test/stream_group_test.dart
@@ -507,7 +507,7 @@ void regardlessOfType(StreamGroup<String> newStreamGroup()) {
await flushMicrotasks();
controller.add("second");
- expect(streamGroup.remove(controller.stream), isNull);
+ expect(streamGroup.remove(controller.stream), completion(null));
expect(streamGroup.close(), completes);
});
@@ -548,7 +548,7 @@ void regardlessOfType(StreamGroup<String> newStreamGroup()) {
expect(streamGroup.stream.toList(), completion(isEmpty));
controller.add("first");
- expect(streamGroup.remove(controller.stream), isNull);
+ expect(streamGroup.remove(controller.stream), completion(null));
controller.add("second");
expect(streamGroup.close(), completes);
« no previous file with comments | « pubspec.yaml ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698