fix(core): fix#1153, ZoneTask.toString should always be a string (#1166)

Author: JiaLiPassion

lib/zone.ts

@@ -1240,7 +1240,7 @@ const Zone: ZoneType = (function(global: any) {
 
 publictoString(){
 if(this.data&&typeofthis.data.handleId!=='undefined'){
-returnthis.data.handleId;
+returnthis.data.handleId.toString();
 }else{
 returnObject.prototype.toString.call(this);
 }

test/common/toString.spec.ts

@@ -47,3 +47,43 @@ describe('global function patch', () => {
 }));
 });
 });
+
+describe('ZoneTask',()=>{
+it('should return handleId.toString if handleId is available',()=>{
+letmacroTask1: any=undefined;
+letmacroTask2: any=undefined;
+letmicroTask: any=undefined;
+constzone=Zone.current.fork({
+name: 'timer',
+onScheduleTask: (delegate: ZoneDelegate,curr: Zone,target: Zone,task: Task)=>{
+if(task.type==='macroTask'){
+if(!macroTask1){
+macroTask1=task;
+}else{
+macroTask2=task;
+}
+}elseif(task.type==='microTask'){
+microTask=task;
+}
+returntask;
+}
+});
+zone.run(()=>{
+constid1=setTimeout(()=>{});
+clearTimeout(id1);
+constid2=setTimeout(()=>{});
+clearTimeout(id2);
+Promise.resolve().then(()=>{});
+constmacroTask1Str=macroTask1.toString();
+constmacroTask2Str=macroTask2.toString();
+expect(typeofmacroTask1Str).toEqual('string');
+expect(macroTask1Str).toEqual(id1.toString());
+expect(typeofmacroTask2Str).toEqual('string');
+expect(macroTask2Str).toEqual(id2.toString());
+if(macroTask1.data&&typeofmacroTask1.data.handleId==='number'){
+expect(macroTask1Str).not.toEqual(macroTask2Str);
+}
+expect(typeofmicroTask.toString()).toEqual('string');
+});
+});
+});