Index: test/mjsunit/es6/debug-promises-uncaught-all.js |
diff --git a/test/mjsunit/es6/debug-promises-uncaught-all.js b/test/mjsunit/es6/debug-promises-uncaught-all.js |
index 714e7da9c58e8c4ef407dcf980fc90edcdbc7d22..7ee8cb2b68066baa0dd93ca3fce848e05982e66f 100644 |
--- a/test/mjsunit/es6/debug-promises-uncaught-all.js |
+++ b/test/mjsunit/es6/debug-promises-uncaught-all.js |
@@ -6,7 +6,10 @@ |
// Test debug events when we listen to all exceptions and |
// there is a catch handler for the exception thrown in a Promise. |
-// We expect an Exception debug event with a promise to be triggered. |
+// Expectation: |
+// - the normal Exception debug event is triggered. |
+// - the PendingExceptionInPromise debug event is triggered afterwards, |
+// with the same exception object. |
Debug = debug.Debug; |
@@ -22,24 +25,28 @@ var p = new Promise(function(resolve, reject) { |
var q = p.chain( |
function() { |
log.push("throw"); |
- throw new Error("uncaught"); // event |
+ throw new Error("uncaught"); |
}); |
function listener(event, exec_state, event_data, data) { |
try { |
// Ignore exceptions during startup in stress runs. |
- if (step >= 1) return; |
+ if (step > 1) return; |
assertEquals(["resolve", "end main", "throw"], log); |
if (event == Debug.DebugEvent.Exception) { |
assertEquals(0, step); |
- assertEquals("uncaught", event_data.exception().message); |
+ exception = event_data.exception(); |
+ assertEquals(undefined, event_data.promise()); |
+ } else if (event == Debug.DebugEvent.PendingExceptionInPromise) { |
+ assertEquals(1, step); |
+ assertEquals(exception, event_data.exception()); |
+ assertEquals("uncaught", exception.message); |
assertTrue(event_data.promise() instanceof Promise); |
- assertEquals(q, event_data.promise()); |
assertTrue(event_data.uncaught()); |
- // Assert that the debug event is triggered at the throw site. |
- assertTrue(exec_state.frame(0).sourceLineText().indexOf("// event") > 0); |
- step++; |
+ } else { |
+ return; |
} |
+ step++; |
} catch (e) { |
// Signal a failure with exit code 1. This is necessary since the |
// debugger swallows exceptions and we expect the chained function |