fix(promise): include stack trace in an unhandlerd promise (#463)

mhevery · web-flow · commit 737f8d834a95 · 2016-09-20T11:41:12.000-07:00
diff --git a/lib/zone.ts b/lib/zone.ts
@@ -867,7 +867,7 @@ const Zone: ZoneType = (function(global: any) {
       if (this.data && typeof this.data.handleId !== 'undefined') {
         return this.data.handleId;
       } else {
-        return this.toString();
+        return Object.prototype.toString.call(this);
       }
     }
   }
@@ -994,7 +994,7 @@ const Zone: ZoneType = (function(global: any) {
         if (queue.length == 0 && state == REJECTED) {
           promise[symbolState] = REJECTED_NO_CATCH;
           try {
-            throw new Error("Uncaught (in promise): " +  value);
+            throw new Error("Uncaught (in promise): " +  value + (value && value.stack ? '\n' + value.stack : ''));
           } catch (e) {
             const error: UncaughtPromiseError = e;
             error.rejection = value;
diff --git a/test/common/Promise.spec.ts b/test/common/Promise.spec.ts
@@ -233,6 +233,7 @@ describe('Promise', ifEnvSupports('Promise', function () {
         var promiseError: Error = null;
         var zone: Zone = null;
         var task: Task = null;
+        var error: Error = null;
         queueZone.fork({
           name: 'promise-error',
           onHandleError: (delegate: ZoneDelegate, current: Zone, target: Zone,
@@ -244,13 +245,19 @@ describe('Promise', ifEnvSupports('Promise', function () {
         }).run(() => {
           zone = Zone.current;
           task = Zone.currentTask;
-          Promise.reject('rejectedErrorShouldBeHandled');
+          error = new Error('rejectedErrorShouldBeHandled');
+          try {
+            // throw so that the stack trace is captured
+            throw error;
+          } catch (e) {}
+          Promise.reject(error);
           expect(promiseError).toBe(null);
         });
         setTimeout(() => null);
         setTimeout(() => {
-          expect(promiseError.message).toBe('Uncaught (in promise): rejectedErrorShouldBeHandled');
-          expect(promiseError['rejection']).toBe('rejectedErrorShouldBeHandled');
+          expect(promiseError.message).toBe('Uncaught (in promise): ' + error +
+              (error.stack ? '\n' + error.stack : ''));
+          expect(promiseError['rejection']).toBe(error);
           expect(promiseError['zone']).toBe(zone);
           expect(promiseError['task']).toBe(task);
           done();
diff --git a/test/zone-spec/fake-async-test.spec.ts b/test/zone-spec/fake-async-test.spec.ts
@@ -59,7 +59,7 @@ describe('FakeAsyncTestZoneSpec', () => {
         fakeAsyncTestZone.run(() => {
           Promise.resolve(null).then((_) => { throw new Error('async'); });
           expect(() => { testZoneSpec.flushMicrotasks(); })
-            .toThrowError('Uncaught (in promise): Error: async');
+            .toThrowError(/Uncaught \(in promise\): Error: async/);
         });
       });
 

Original file line number	Diff line number	Diff line change
`@@ -867,7 +867,7 @@ const Zone: ZoneType = (function(global: any) {`
`867`	`867`	`if (this.data && typeof this.data.handleId !== 'undefined') {`
`868`	`868`	`return this.data.handleId;`
`869`	`869`	`} else {`
`870`		`- return this.toString();`
	`870`	`+ return Object.prototype.toString.call(this);`
`871`	`871`	`}`
`872`	`872`	`}`
`873`	`873`	`}`
`@@ -994,7 +994,7 @@ const Zone: ZoneType = (function(global: any) {`
`994`	`994`	`if (queue.length == 0 && state == REJECTED) {`
`995`	`995`	`promise[symbolState] = REJECTED_NO_CATCH;`
`996`	`996`	`try {`
`997`		`- throw new Error("Uncaught (in promise): " + value);`
	`997`	`+ throw new Error("Uncaught (in promise): " + value + (value && value.stack ? '\n' + value.stack : ''));`
`998`	`998`	`} catch (e) {`
`999`	`999`	`const error: UncaughtPromiseError = e;`
`1000`	`1000`	`error.rejection = value;`