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

Unified Diff: tests/lib_strong/async/catch_errors13_test.dart

Issue 2771453003: Format all tests. (Closed)
Patch Set: Format files Created 3 years, 8 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
Index: tests/lib_strong/async/catch_errors13_test.dart
diff --git a/tests/lib_strong/async/catch_errors13_test.dart b/tests/lib_strong/async/catch_errors13_test.dart
index 1723f87d1e264b450ff52ad2bf832b96f63746ff..7c4d0e0f45e8509c9965ea1163edbdf245bc2a20 100644
--- a/tests/lib_strong/async/catch_errors13_test.dart
+++ b/tests/lib_strong/async/catch_errors13_test.dart
@@ -15,7 +15,7 @@ main() {
// Work around bug that makes scheduleMicrotask use Timers. By invoking
// `scheduleMicrotask` here we make sure that asynchronous non-timer events
// are executed before any Timer events.
- scheduleMicrotask(() { });
+ scheduleMicrotask(() {});
// Test that errors are caught by nested `catchErrors`. Also uses
// `scheduleMicrotask` in the body of a Timer.
@@ -25,7 +25,9 @@ main() {
events.add("catch error entry");
catchErrors(() {
events.add("catch error entry2");
- Timer.run(() { throw "timer error"; });
+ Timer.run(() {
+ throw "timer error";
+ });
// We want this timer to run after the timer below. When the machine is
// slow we saw that the timer below was scheduled more than 50ms after
@@ -36,16 +38,17 @@ main() {
// This way we could reduce the timeout (to 10ms now), while still
// allowing for more time, in case the machine is slow.
void runDelayed() {
- new Timer(const Duration(milliseconds: 10),
- () {
- if (outerTimerRan) {
- scheduleMicrotask(() { throw "scheduleMicrotask"; });
- throw "delayed error";
- } else if (outerTimerCounterDelayCount < 100) {
- outerTimerCounterDelayCount++;
- runDelayed();
- }
- });
+ new Timer(const Duration(milliseconds: 10), () {
+ if (outerTimerRan) {
+ scheduleMicrotask(() {
+ throw "scheduleMicrotask";
+ });
+ throw "delayed error";
+ } else if (outerTimerCounterDelayCount < 100) {
+ outerTimerCounterDelayCount++;
+ runDelayed();
+ }
+ });
}
runDelayed();
@@ -62,27 +65,27 @@ main() {
});
throw "inner throw";
}).listen((x) {
- events.add(x);
- if (x == "inner done throw") done.complete(true);
- },
- onDone: () { Expect.fail("Unexpected callback"); });
+ events.add(x);
+ if (x == "inner done throw") done.complete(true);
+ }, onDone: () {
+ Expect.fail("Unexpected callback");
+ });
done.future.whenComplete(() {
// Give callbacks time to run.
Timer.run(() {
Expect.listEquals([
- "catch error entry",
- "catch error entry2",
- "after inner",
- "main exit",
- "inner throw",
- "timer error",
- "timer outer",
- "delayed error",
- "scheduleMicrotask",
- "inner done throw"
- ],
- events);
+ "catch error entry",
+ "catch error entry2",
+ "after inner",
+ "main exit",
+ "inner throw",
+ "timer error",
+ "timer outer",
+ "delayed error",
+ "scheduleMicrotask",
+ "inner done throw"
+ ], events);
asyncEnd();
});
});

Powered by Google App Engine
This is Rietveld 408576698