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

Unified Diff: tests/isolate/handle_error3_test.dart

Issue 169703002: Add an error listener on isolates. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Rewrite. Now catch all in IsolateContext.eval Created 6 years, 7 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/isolate/handle_error3_test.dart
diff --git a/tests/isolate/handle_error3_test.dart b/tests/isolate/handle_error3_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..2c5fa0c6646e06c12e40fa276c454fb0cc0a5139
--- /dev/null
+++ b/tests/isolate/handle_error3_test.dart
@@ -0,0 +1,112 @@
+// Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+library handle_error_test;
+
+import "dart:isolate";
+import "dart:async";
+import "package:async_helper/async_helper.dart";
+import "package:expect/expect.dart";
+
+isomain1(replyPort) {
+ RawReceivePort port = new RawReceivePort();
+ port.handler = (v) {
+ switch (v) {
+ case 0:
+ replyPort.send(42);
+ break;
+ case 1:
+ throw new ArgumentError("whoops");
+ case 2:
+ throw new RangeError.value(37);
+ case 3:
+ port.close();
+ }
+ };
+ replyPort.send(port.sendPort);
+}
+
+Future spawn(entry) {
+ ReceivePort reply = new ReceivePort();
+ Future isolate = Isolate.spawn(entry, reply.sendPort, paused: true);
+ return isolate.then((Isolate isolate) {
+ isolate.setErrorsFatal(false);
+ isolate.resume(isolate.pauseCapability);
+ Future result = reply.first.then((sendPort) {
+ return [isolate, sendPort];
+ });
+ return result;
+ });
+}
+
+main(){
+ asyncStart();
+ asyncStart();
+ RawReceivePort reply = new RawReceivePort(null);
+ RawReceivePort reply2 = new RawReceivePort(null);
+ // Start paused so we have time to set up the error handler.
floitsch 2014/05/15 12:15:25 ditto (as in handle_error2)
+ Future iso1 = spawn(isomain1);
+ Future iso2 = spawn(isomain1);
+ Future.wait([iso1, iso2]).then((l) {
+ var isolate1 = l[0][0];
+ var sendPort1 = l[0][1];
+ var isolate2 = l[1][0];
+ var sendPort2 = l[1][1];
+ // Capture errors from one isolate as stream.
+ Stream errors = isolate1.errors; // Broadcast stream, never a done message.
+ int state = 1;
+ var subscription;
+ subscription = errors.listen(null, onError: (error, stack) {
+ switch (state) {
+ case 1:
+ Expect.equals(new ArgumentError("whoops").toString(), "$error");
+ state++;
+ break;
+ case 2:
+ Expect.equals(new RangeError.value(37).toString(), "$error");
+ state++;
+ reply.close();
+ subscription.cancel();
+ asyncEnd();
+ break;
+ default:
+ throw "Bad state for error: $state: $error";
+ }
+ });
+ // Capture errors from other isolate as raw messages.
+ RawReceivePort errorPort2 = new RawReceivePort();
+ int state2 = 1;
+ errorPort2.handler = (message) {
+ String error = message[0];
+ String stack = message[1];
+ switch (state2) {
+ case 1:
+ Expect.equals(new ArgumentError("whoops").toString(), "$error");
+ state2++;
+ break;
+ case 2:
+ Expect.equals(new RangeError.value(37).toString(), "$error");
+ state2++;
+ reply.close();
+ isolate2.removeErrorListener(errorPort2.sendPort);
+ errorPort2.close();
+ asyncEnd();
+ break;
+ default:
+ throw "Bad state-2 for error: $state: $error";
+ }
+ };
+ isolate2.addErrorListener(errorPort2.sendPort);
+
+ sendPort1.send(0);
+ sendPort2.send(0);
+ sendPort1.send(1);
+ sendPort2.send(1);
+ sendPort1.send(2);
+ sendPort2.send(2);
+ sendPort1.send(3);
+ sendPort2.send(3);
+ });
+}
+

Powered by Google App Engine
This is Rietveld 408576698