Fix tests by initializing JSDom in the right places

Author: Luca Forstner

packages/nextjs/test/index.client.test.ts

@@ -4,6 +4,7 @@ import * as SentryReact from '@sentry/react';
 import { Integrations as TracingIntegrations } from '@sentry/tracing';
 import { Integration } from '@sentry/types';
 import { getGlobalObject, logger, SentryError } from '@sentry/utils';
+import { JSDOM } from 'jsdom';
 
 import { init, Integrations, nextRouterInstrumentation } from '../src/index.client';
 import { NextjsOptions } from '../src/utils/nextjsOptions';
@@ -16,6 +17,18 @@ const reactInit = jest.spyOn(SentryReact, 'init');
 const captureEvent = jest.spyOn(BaseClient.prototype, 'captureEvent');
 const logWarn = jest.spyOn(logger, 'warn');
 
+// Set up JSDom - needed for page load instrumentation
+const dom = new JSDOM(undefined, { url: 'https://example.com/' });
+Object.defineProperty(global, 'document', { value: dom.window.document, writable: true });
+
+const originalGlobalDocument = getGlobalObject<Window>().document;
+afterAll(() => {
+  // Clean up JSDom
+  Object.defineProperty(global, 'document', {
+    value: originalGlobalDocument,
+  });
+});
+
 describe('Client init()', () => {
   afterEach(() => {
     jest.clearAllMocks();

packages/nextjs/test/performance/client.test.ts

@@ -60,16 +60,12 @@ describe('client', () => {
         { url: pageProperties.url },
       );
 
-      Object.defineProperty(global, 'document', { value: dom.window.document, writable: true, configurable: true });
+      Object.defineProperty(global, 'document', { value: dom.window.document, writable: true });
     }
 
     afterEach(() => {
       // Clean up JSDom
-      Object.defineProperty(global, 'document', {
-        value: originalGlobalDocument,
-        writable: true,
-        configurable: true,
-      });
+      Object.defineProperty(global, 'document', { value: originalGlobalDocument });
     });
 
     it('waits for Router.ready()', () => {