fix(error): fix #706, handleError when onHasTask throw error (#709)

JiaLiPassion · mhevery · commit f514c11f0362 · 2017-04-10T11:35:00.000+02:00
diff --git a/lib/zone.ts b/lib/zone.ts
@@ -1091,6 +1091,7 @@ const Zone: ZoneType = (function(global: any) {
             this._hasTaskZS.onHasTask(
                 this._hasTaskDlgt, this._hasTaskCurrZone, targetZone, isEmpty);
       } catch (err) {
+        this.handleError(targetZone, err);
       }
     }
 
diff --git a/test/common/zone.spec.ts b/test/common/zone.spec.ts
@@ -394,6 +394,25 @@ describe('Zone', function() {
         JSON.stringify(macro);
       }).not.toThrow();
     });
+
+    it('should call onHandleError callback when zoneSpec onHasTask throw error', () => {
+      const spy = jasmine.createSpy('error');
+      const hasTaskZone = Zone.current.fork({
+        name: 'hasTask',
+        onHasTask: (delegate: ZoneDelegate, currentZone: Zone, targetZone: Zone,
+                    hasTasState: HasTaskState) => {
+          throw new Error('onHasTask Error');
+        },
+        onHandleError:
+            (delegate: ZoneDelegate, currentZone: Zone, targetZone: Zone, error: Error) => {
+              spy(error.message);
+              return delegate.handleError(targetZone, error);
+            }
+      });
+
+      const microTask = hasTaskZone.scheduleMicroTask('test', () => {}, null, () => {});
+      expect(spy).toHaveBeenCalledWith('onHasTask Error');
+    });
   });
 });
 

Original file line number	Diff line number	Diff line change
`@@ -1091,6 +1091,7 @@ const Zone: ZoneType = (function(global: any) {`
`1091`	`1091`	`this._hasTaskZS.onHasTask(`
`1092`	`1092`	`this._hasTaskDlgt, this._hasTaskCurrZone, targetZone, isEmpty);`
`1093`	`1093`	`} catch (err) {`
	`1094`	`+ this.handleError(targetZone, err);`
`1094`	`1095`	`}`
`1095`	`1096`	`}`
`1096`	`1097`