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

Unified Diff: test/subscription_stream_test.dart

Issue 1215293002: Remove warnings in tests, and even fix analyzer hints. (Closed) Base URL: https://github.com/dart-lang/async@master
Patch Set: Created 5 years, 6 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 | « test/stream_queue_test.dart ('k') | test/utils.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/subscription_stream_test.dart
diff --git a/test/subscription_stream_test.dart b/test/subscription_stream_test.dart
index da77593e5918071088041b2c106d97dbbde4e1f2..03f0dddec203723465beb5679edbed44f787adc9 100644
--- a/test/subscription_stream_test.dart
+++ b/test/subscription_stream_test.dart
@@ -83,10 +83,10 @@ main() {
test("- subscriptionStream: no", () async {
var done = new Completer();
var events = [];
- var subscription = subscriptionStream.listen(events.add,
- onError: events.add,
- onDone: done.complete,
- cancelOnError: false);
+ subscriptionStream.listen(events.add,
+ onError: events.add,
+ onDone: done.complete,
+ cancelOnError: false);
var expected = [1, 2, "To err is divine!"];
if (sourceCancels) {
var timeout = done.future.timeout(const Duration(milliseconds: 5),
« no previous file with comments | « test/stream_queue_test.dart ('k') | test/utils.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698