Declare global prototype in JSDOMEnvironment to fix issue #8347 (#8352)

* Declare global prototype in JSDOMEnvironment to fix issue #8347

* Change type to check isWin and remove set global to null

* Add description of PR to the changelog

* Set global to `null` in teardown process and uncheck is window for global prototype

CHANGELOG.md

@@ -17,6 +17,7 @@
 - `[jest-core]` Make `detectOpenHandles` imply `runInBand` ([#8283](https://github.com/facebook/jest/pull/8283))
 - `[jest-haste-map]` Fix the `mapper` option which was incorrectly ignored ([#8299](https://github.com/facebook/jest/pull/8299))
 - `[jest-jasmine2]` Fix describe return value warning being shown if the describe function throws ([#8335](https://github.com/facebook/jest/pull/8335))
+- `[jest-environment-jsdom]` Re-declare global prototype of JSDOMEnvironment ([#8352](https://github.com/facebook/jest/pull/8352))
 
 ### Chore & Maintenance
 

packages/jest-environment-jsdom/src/index.ts

@@ -15,24 +15,17 @@ import {JSDOM, VirtualConsole} from 'jsdom';
 
 // The `Window` interface does not have an `Error.stackTraceLimit` property, but
 // `JSDOMEnvironment` assumes it is there.
-interface Win extends Window {
-  Error: {
-    stackTraceLimit: number;
+type Win = Window &
+  Global.Global & {
+    Error: {
+      stackTraceLimit: number;
+    };
   };
-}
-
-function isWin(globals: Win | Global.Global): globals is Win {
-  return (globals as Win).document !== undefined;
-}
-
-// A lot of the globals expected by other APIs are `NodeJS.Global` and not
-// `Window`, so we need to cast here and there
 
 class JSDOMEnvironment implements JestEnvironment {
   dom: JSDOM | null;
   fakeTimers: FakeTimers<number> | null;
-  // @ts-ignore
-  global: Global.Global | Win | null;
+  global: Win;
   errorEventListener: ((event: Event & {error: Error}) => void) | null;
   moduleMocker: ModuleMocker | null;
 
@@ -109,16 +102,15 @@ class JSDOMEnvironment implements JestEnvironment {
       this.fakeTimers.dispose();
     }
     if (this.global) {
-      if (this.errorEventListener && isWin(this.global)) {
+      if (this.errorEventListener) {
         this.global.removeEventListener('error', this.errorEventListener);
       }
       // Dispose "document" to prevent "load" event from triggering.
       Object.defineProperty(this.global, 'document', {value: null});
-      if (isWin(this.global)) {
-        this.global.close();
-      }
+      this.global.close();
     }
     this.errorEventListener = null;
+    // @ts-ignore
     this.global = null;
     this.dom = null;
     this.fakeTimers = null;