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

Side by Side Diff: tests/lib/async/catch_errors28_test.dart

Issue 15764003: Add Zone support for Timers. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Fix bug and add tests. Created 7 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file.
4
5 import "package:expect/expect.dart";
6 import 'dart:async';
7 import 'dart:isolate';
8
9 main() {
10 // We keep a ReceivePort open until all tests are done. This way the VM will
11 // time out if the callbacks are not invoked.
Lasse Reichstein Nielsen 2013/05/29 08:04:59 The VM won't time out. It will hang, and the test
floitsch 2013/05/29 15:21:29 Done.
12 var port = new ReceivePort();
13 var events = [];
14 // Test that periodic Timers are handled correctly by `catchErrors`.
15 catchErrors(() {
16 int counter = 0;
17 new Timer.periodic(const Duration(milliseconds: 50), (timer) {
18 if (counter++ == 5) timer.cancel();
Lasse Reichstein Nielsen 2013/05/29 08:04:59 Please move counter++ to a line of its own. Onl
floitsch 2013/05/29 15:21:29 Done.
19 events.add(counter);
20 });
21 }).listen((x) {
22 events.add(x);
23 },
24 onDone: () {
25 Expect.listEquals([
26 "main exit",
27 1, 2, 3, 4, 5, 6,
28 ],
29 events);
30 port.close();
31 });
32 events.add("main exit");
33 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698