use native AbortController (#11753)

apollographql · Apr 4, 2024 · 062394e · 062394e
1 parent 4900d5a
commit 062394e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/config/FixJSDOMEnvironment.js b/config/FixJSDOMEnvironment.js
@@ -14,7 +14,7 @@ class FixJSDOMEnvironment extends JSDOMEnvironment {
     // and setting AbortController breaks PersistedQueryLink tests, which may
     // indicate a memory leak
     // this.global.fetch = fetch;
-    // this.global.AbortController = AbortController;
+    this.global.AbortController = AbortController;
   }
 }
 

diff --git a/src/link/persisted-queries/__tests__/persisted-queries.test.ts b/src/link/persisted-queries/__tests__/persisted-queries.test.ts
@@ -573,7 +573,11 @@ describe("failure path", () => {
           variables,
         }).subscribe({ complete })
       );
-
+      // fetch-mock holds a history of all options it has been called with
+      // that includes the `signal` option, which (with the native `AbortController`)
+      // has a reference to the `Request` instance, which will somehow reference our
+      // hash object
+      fetchMock.resetHistory();
       await expect(hashRefs[0]).toBeGarbageCollected();
     }
   );