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

Unified Diff: tests/lib/async/future_test.dart

Issue 2788103002: Fix test that assumed an async function returned a Future<not-dynamic>. (Closed)
Patch Set: Created 3 years, 9 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/lib/async/future_test.dart
diff --git a/tests/lib/async/future_test.dart b/tests/lib/async/future_test.dart
index efe66e25c327ffc2c83d7ba7281a801b74a3853c..4c46e3f79c94230c15da2e6a59c30e340e724c79 100644
--- a/tests/lib/async/future_test.dart
+++ b/tests/lib/async/future_test.dart
@@ -994,9 +994,9 @@ void testTypes() {
new Future<int>.delayed(Duration.ZERO, () => value));
testType(
"Future.microtask($value)", new Future<int>.microtask(() => value));
- testType("Future.sync($value)", new Future<int>.sync(() => value)); // //# 01: ok
- testType("Future.sync(future($value))", // //# 01: continued
- new Future<int>.sync(() async => new Future<int>.value(value))); //# 01: continued
+ testType("Future.sync($value)", new Future<int>.sync(() => value)); //# 01: ok
+ testType("Future.sync(future($value))", // //# 01: continued
+ new Future<int>.sync(() => new Future<int>.value(value))); //# 01: continued
testType("Future.value($value)", new Future<int>.value(value));
}
testType("Completer.future", new Completer<int>().future);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698