Undo change to global.location

Author: lforst

packages/nextjs/src/performance/client.ts

@@ -111,7 +111,7 @@ export function nextRouterInstrumentation(
   if (startTransactionOnPageLoad) {
     const { route, traceParentData, baggage, params } = extractNextDataTagInformation();
 
-    prevTransactionName = route || global.document.location.pathname;
+    prevTransactionName = route || global.location.pathname;
     const source = route ? 'route' : 'url';
 
     activeTransaction = startTransactionCb({

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

@@ -20,13 +20,14 @@ 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 });
+Object.defineProperty(global, 'location', { value: dom.window.document.location, writable: true });
 
 const originalGlobalDocument = getGlobalObject<Window>().document;
+const originalGlobalLocation = getGlobalObject<Window>().location;
 afterAll(() => {
   // Clean up JSDom
-  Object.defineProperty(global, 'document', {
-    value: originalGlobalDocument,
-  });
+  Object.defineProperty(global, 'document', { value: originalGlobalDocument });
+  Object.defineProperty(global, 'location', { value: originalGlobalLocation });
 });
 
 describe('Client init()', () => {

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

@@ -31,6 +31,7 @@ describe('client', () => {
 
   describe('nextRouterInstrumentation', () => {
     const originalGlobalDocument = getGlobalObject<Window>().document;
+    const originalGlobalLocation = getGlobalObject<Window>().location;
 
     function setUpNextPage(pageProperties: {
       url: string;
@@ -61,11 +62,13 @@ describe('client', () => {
       );
 
       Object.defineProperty(global, 'document', { value: dom.window.document, writable: true });
+      Object.defineProperty(global, 'location', { value: dom.window.document.location, writable: true });
     }
 
     afterEach(() => {
       // Clean up JSDom
       Object.defineProperty(global, 'document', { value: originalGlobalDocument });
+      Object.defineProperty(global, 'location', { value: originalGlobalLocation });
     });
 
     it('waits for Router.ready()', () => {