fix(zone.js): fix several test cases which trigger done() multiple times (#45025)

JiaLiPassion · alxhub · commit d5565ccdb457 · 2022-02-09T10:18:56.000-08:00
Several test cases trigger the `done()` multiple times and cause warning from jasmine. This commit fixes these test cases. PR Close #45025
diff --git a/packages/zone.js/test/browser/XMLHttpRequest.spec.ts b/packages/zone.js/test/browser/XMLHttpRequest.spec.ts
@@ -336,14 +336,16 @@ describe('XMLHttpRequest', function() {
          const req = new XMLHttpRequest();
          req.open('get', '/', true);
          req.send();
-         req.addEventListener('readystatechange', function(ev) {
+         const listener = function(ev: any) {
            if (req.readyState >= 2) {
              expect(() => {
                req.abort();
              }).not.toThrow();
+             req.removeEventListener('readystatechange', listener);
              done();
            }
-         });
+         };
+         req.addEventListener('readystatechange', listener);
        });
      });
 
diff --git a/packages/zone.js/test/browser/browser.spec.ts b/packages/zone.js/test/browser/browser.spec.ts
@@ -2907,6 +2907,7 @@ describe('Zone', function() {
 
               expect(entries.length).toBe(1);
               expect(entries[0].target).toBe(div);
+              observer.disconnect();
               done();
             });
 
diff --git a/packages/zone.js/test/extra/cordova.spec.ts b/packages/zone.js/test/extra/cordova.spec.ts
@@ -19,7 +19,6 @@ describe('cordova test', () => {
       cordova.exec(
           () => {
             expect(Zone.current.name).toEqual('cordova');
-            done();
           },
           () => {
             fail('should not fail');
