tests: test the actual transaction sending for page loads

Author: logaretm

dev-packages/e2e-tests/test-applications/nextjs-15/app/isr-test/[product]/page.tsx

@@ -1,3 +1,6 @@
+export const revalidate = 60; // ISR: revalidate every 60 seconds
+export const dynamicParams = true; // Allow dynamic params beyond generateStaticParams
+
 export async function generateStaticParams(): Promise<Array<{ product: string }>> {
   return [{ product: 'laptop' }, { product: 'phone' }, { product: 'tablet' }];
 }

dev-packages/e2e-tests/test-applications/nextjs-15/app/isr-test/static/page.tsx

@@ -1,3 +1,6 @@
+export const revalidate = 60; // ISR: revalidate every 60 seconds
+export const dynamicParams = true;
+
 export async function generateStaticParams(): Promise<never[]> {
   return [];
 }

dev-packages/e2e-tests/test-applications/nextjs-15/instrumentation-client.ts

@@ -2,10 +2,11 @@ import * as Sentry from '@sentry/nextjs';
 
 Sentry.init({
   environment: 'qa', // dynamic sampling bias to keep transactions
-  dsn: process.env.NEXT_PUBLIC_E2E_TEST_DSN,
+  dsn: 'https://username@domain/123',
   tunnel: `http://localhost:3031/`, // proxy server
   tracesSampleRate: 1.0,
   sendDefaultPii: true,
+  debug: true,
 });
 
 export const onRouterTransitionStart = Sentry.captureRouterTransitionStart;

dev-packages/e2e-tests/test-applications/nextjs-15/tests/isr-routes.test.ts

@@ -41,37 +41,34 @@ test('should remove meta tags for different ISR dynamic route values', async ({
   await expect(page.locator('meta[name="baggage"]')).toHaveCount(0);
 });
 
-test('should create unique transactions for ISR pages (not using stale trace IDs)', async ({ page }) => {
-  // First navigation - capture the trace ID
-  const firstTransactionPromise = waitForTransaction('nextjs-15', async transactionEvent => {
-    return transactionEvent.transaction === '/isr-test/:product' && transactionEvent.contexts?.trace?.op === 'pageload';
-  });
-
-  await page.goto('/isr-test/laptop');
-  const firstTransaction = await firstTransactionPromise;
-  const firstTraceId = firstTransaction.contexts?.trace?.trace_id;
-
-  expect(firstTraceId).toBeDefined();
-  expect(firstTraceId).toMatch(/[a-f0-9]{32}/);
-
-  // Second navigation to the same ISR route with different param
-  const secondTransactionPromise = waitForTransaction('nextjs-15', async transactionEvent => {
-    return !!(
-      transactionEvent.transaction === '/isr-test/:product' &&
-      transactionEvent.contexts?.trace?.op === 'pageload' &&
-      transactionEvent.request?.url?.includes('/isr-test/phone')
-    );
-  });
-
-  await page.goto('/isr-test/phone');
-  const secondTransaction = await secondTransactionPromise;
-  const secondTraceId = secondTransaction.contexts?.trace?.trace_id;
-
-  expect(secondTraceId).toBeDefined();
-  expect(secondTraceId).toMatch(/[a-f0-9]{32}/);
-
-  // Verify that each page load gets a NEW trace ID (not reusing cached/stale ones)
-  expect(firstTraceId).not.toBe(secondTraceId);
+test('should create unique transactions for ISR pages on each visit', async ({ page }) => {
+  const traceIds: string[] = [];
+
+  // Load the same ISR page 5 times to ensure cached HTML meta tags are consistently removed
+  for (let i = 0; i < 5; i++) {
+    const transactionPromise = waitForTransaction('nextjs-15', async transactionEvent => {
+      return !!(
+        transactionEvent.transaction === '/isr-test/:product' && transactionEvent.contexts?.trace?.op === 'pageload'
+      );
+    });
+
+    if (i === 0) {
+      await page.goto('/isr-test/laptop');
+    } else {
+      await page.reload();
+    }
+
+    const transaction = await transactionPromise;
+    const traceId = transaction.contexts?.trace?.trace_id;
+
+    expect(traceId).toBeDefined();
+    expect(traceId).toMatch(/[a-f0-9]{32}/);
+    traceIds.push(traceId!);
+  }
+
+  // Verify all 5 page loads have unique trace IDs (no reuse of cached/stale meta tags)
+  const uniqueTraceIds = new Set(traceIds);
+  expect(uniqueTraceIds.size).toBe(5);
 });
 
 test('ISR route should be identified correctly in the route manifest', async ({ page }) => {

dev-packages/e2e-tests/test-applications/nextjs-16/app/isr-test/[product]/page.tsx

@@ -1,3 +1,6 @@
+export const revalidate = 60; // ISR: revalidate every 60 seconds
+export const dynamicParams = true; // Allow dynamic params beyond generateStaticParams
+
 export async function generateStaticParams(): Promise<Array<{ product: string }>> {
   return [{ product: 'laptop' }, { product: 'phone' }, { product: 'tablet' }];
 }

dev-packages/e2e-tests/test-applications/nextjs-16/app/isr-test/static/page.tsx

@@ -1,3 +1,6 @@
+export const revalidate = 60; // ISR: revalidate every 60 seconds
+export const dynamicParams = true;
+
 export async function generateStaticParams(): Promise<never[]> {
   return [];
 }

dev-packages/e2e-tests/test-applications/nextjs-16/tests/isr-routes.test.ts

@@ -41,37 +41,34 @@ test('should remove meta tags for different ISR dynamic route values', async ({
   await expect(page.locator('meta[name="baggage"]')).toHaveCount(0);
 });
 
-test('should create unique transactions for ISR pages (not using stale trace IDs)', async ({ page }) => {
-  // First navigation - capture the trace ID
-  const firstTransactionPromise = waitForTransaction('nextjs-16', async transactionEvent => {
-    return transactionEvent.transaction === '/isr-test/:product' && transactionEvent.contexts?.trace?.op === 'pageload';
-  });
-
-  await page.goto('/isr-test/laptop');
-  const firstTransaction = await firstTransactionPromise;
-  const firstTraceId = firstTransaction.contexts?.trace?.trace_id;
-
-  expect(firstTraceId).toBeDefined();
-  expect(firstTraceId).toMatch(/[a-f0-9]{32}/);
-
-  // Second navigation to the same ISR route with different param
-  const secondTransactionPromise = waitForTransaction('nextjs-16', async transactionEvent => {
-    return !!(
-      transactionEvent.transaction === '/isr-test/:product' &&
-      transactionEvent.contexts?.trace?.op === 'pageload' &&
-      transactionEvent.request?.url?.includes('/isr-test/phone')
-    );
-  });
-
-  await page.goto('/isr-test/phone');
-  const secondTransaction = await secondTransactionPromise;
-  const secondTraceId = secondTransaction.contexts?.trace?.trace_id;
-
-  expect(secondTraceId).toBeDefined();
-  expect(secondTraceId).toMatch(/[a-f0-9]{32}/);
-
-  // Verify that each page load gets a NEW trace ID (not reusing cached/stale ones)
-  expect(firstTraceId).not.toBe(secondTraceId);
+test('should create unique transactions for ISR pages on each visit', async ({ page }) => {
+  const traceIds: string[] = [];
+
+  // Load the same ISR page 5 times to ensure cached HTML meta tags are consistently removed
+  for (let i = 0; i < 5; i++) {
+    const transactionPromise = waitForTransaction('nextjs-16', async transactionEvent => {
+      return !!(
+        transactionEvent.transaction === '/isr-test/:product' && transactionEvent.contexts?.trace?.op === 'pageload'
+      );
+    });
+
+    if (i === 0) {
+      await page.goto('/isr-test/laptop');
+    } else {
+      await page.reload();
+    }
+
+    const transaction = await transactionPromise;
+    const traceId = transaction.contexts?.trace?.trace_id;
+
+    expect(traceId).toBeDefined();
+    expect(traceId).toMatch(/[a-f0-9]{32}/);
+    traceIds.push(traceId!);
+  }
+
+  // Verify all 5 page loads have unique trace IDs (no reuse of cached/stale meta tags)
+  const uniqueTraceIds = new Set(traceIds);
+  expect(uniqueTraceIds.size).toBe(5);
 });
 
 test('ISR route should be identified correctly in the route manifest', async ({ page }) => {