tests: update tests

Author: logaretm

packages/nextjs/src/client/routing/isrRoutingTracing.ts

@@ -1,8 +1,8 @@
 import type { RouteManifest } from '../../config/manifest/types';
 import { maybeParameterizeRoute } from './parameterization';
-import { GLOBAL_OBJ } from '@sentry/core';
+import { WINDOW } from '@sentry/react';
 
-const globalWithInjectedValues = GLOBAL_OBJ as typeof GLOBAL_OBJ & {
+const globalWithInjectedValues = WINDOW as typeof WINDOW & {
   _sentryRouteManifest: string | RouteManifest;
 };
 
@@ -41,14 +41,14 @@ function isIsrSsgRoute(pathname: string): boolean {
  * This prevents the browser tracing integration from using stale/cached trace IDs.
  */
 export function removeIsrSsgTraceMetaTags(): void {
-  if (typeof document === 'undefined') {
+  if (!WINDOW.document || !isIsrSsgRoute(WINDOW.location.pathname)) {
     return;
   }
 
   // Helper function to remove a meta tag
   const removeMetaTag = (metaName: string) => {
     try {
-      const meta = document.querySelector(`meta[name="${metaName}"]`);
+      const meta = WINDOW.document.querySelector(`meta[name="${metaName}"]`);
       if (meta) {
         meta.remove();
       }
@@ -57,10 +57,6 @@ export function removeIsrSsgTraceMetaTags(): void {
     }
   };
 
-  if (!isIsrSsgRoute(window.location.pathname)) {
-    return;
-  }
-
   // Remove the meta tags so browserTracingIntegration won't pick them up
   removeMetaTag('sentry-trace');
   removeMetaTag('baggage');

packages/nextjs/test/config/manifest/suites/base-path/base-path.test.ts

@@ -19,6 +19,7 @@ describe('basePath', () => {
           hasOptionalPrefix: false,
         },
       ],
+      isrRoutes: [],
     });
   });
 

packages/nextjs/test/config/manifest/suites/catchall-at-root/catchall-at-root.test.ts

@@ -16,6 +16,7 @@ describe('catchall', () => {
           hasOptionalPrefix: false,
         },
       ],
+      isrRoutes: [],
     });
   });
 

packages/nextjs/test/config/manifest/suites/catchall/catchall.test.ts

@@ -16,6 +16,7 @@ describe('catchall', () => {
           hasOptionalPrefix: false,
         },
       ],
+      isrRoutes: [],
     });
   });
 

packages/nextjs/test/config/manifest/suites/dynamic/dynamic.test.ts

@@ -34,6 +34,7 @@ describe('dynamic', () => {
           hasOptionalPrefix: false,
         },
       ],
+      isrRoutes: [],
     });
   });
 

packages/nextjs/test/config/manifest/suites/file-extensions/file-extensions.test.ts

@@ -16,6 +16,7 @@ describe('file-extensions', () => {
         { path: '/typescript' },
       ],
       dynamicRoutes: [],
+      isrRoutes: [],
     });
   });
 });

packages/nextjs/test/config/manifest/suites/route-groups/route-groups.test.ts

@@ -26,6 +26,7 @@ describe('route-groups', () => {
             hasOptionalPrefix: false,
           },
         ],
+        isrRoutes: [],
       });
     });
 
@@ -59,6 +60,7 @@ describe('route-groups', () => {
             hasOptionalPrefix: false,
           },
         ],
+        isrRoutes: [],
       });
     });
 

packages/nextjs/test/config/manifest/suites/static/static.test.ts

@@ -8,6 +8,7 @@ describe('static', () => {
     expect(manifest).toEqual({
       staticRoutes: [{ path: '/' }, { path: '/some/nested' }, { path: '/user' }, { path: '/users' }],
       dynamicRoutes: [],
+      isrRoutes: [],
     });
   });
 });