Use isTurbopackDevMode for nextjs-16 dev mode skips

isDevMode matches both TEST_ENV=development (turbopack) and
TEST_ENV=development-webpack, which silently drops test coverage
for the webpack dev variant where the Turbopack 404 issue doesn't
apply. Add isTurbopackDevMode (exact match on 'development') and
use it for all Turbopack-specific skips.

Also skip the dynamic page test in server-components.test.ts
(/nested-layout/[dynamic]) which has the same Turbopack 404 flake.

Author: andreiborza

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

@@ -1 +1,2 @@
 export const isDevMode = !!process.env.TEST_ENV && process.env.TEST_ENV.includes('development');
+export const isTurbopackDevMode = process.env.TEST_ENV === 'development';

dev-packages/e2e-tests/test-applications/nextjs-16/tests/route-handler.test.ts

@@ -1,9 +1,9 @@
 import test, { expect } from '@playwright/test';
 import { waitForError, waitForTransaction } from '@sentry-internal/test-utils';
-import { isDevMode } from './isDevMode';
+import { isTurbopackDevMode } from './isDevMode';
 
 test('Should create a transaction for node route handlers', async ({ request }) => {
-  test.skip(isDevMode, 'Turbopack intermittently returns 404 for dynamic routes in dev mode');
+  test.skip(isTurbopackDevMode, 'Turbopack intermittently returns 404 for dynamic routes in dev mode');
 
   const routehandlerTransactionPromise = waitForTransaction('nextjs-16', async transactionEvent => {
     return transactionEvent?.transaction === 'GET /route-handler/[xoxo]/node';
@@ -40,7 +40,7 @@ test('Should create a transaction for edge route handlers', async ({ request })
 test('Should report an error with a parameterized transaction name for a throwing route handler', async ({
   request,
 }) => {
-  test.skip(isDevMode, 'Turbopack intermittently returns 404 for dynamic routes in dev mode');
+  test.skip(isTurbopackDevMode, 'Turbopack intermittently returns 404 for dynamic routes in dev mode');
 
   const errorEventPromise = waitForError('nextjs-16', errorEvent => {
     return errorEvent?.exception?.values?.some(value => value.value === 'route-handler-error') ?? false;
@@ -72,7 +72,7 @@ test('Should report an error with a parameterized transaction name for a throwin
 test('Should set a parameterized transaction name on a captureMessage event in a route handler', async ({
   request,
 }) => {
-  test.skip(isDevMode, 'Turbopack intermittently returns 404 for dynamic routes in dev mode');
+  test.skip(isTurbopackDevMode, 'Turbopack intermittently returns 404 for dynamic routes in dev mode');
 
   const messageEventPromise = waitForError('nextjs-16', event => {
     return event?.message === 'route-handler-message';
@@ -105,7 +105,7 @@ test('Should set a parameterized transaction name on a captureMessage event in a
 test('Should set a parameterized transaction name on a captureException event in a route handler', async ({
   request,
 }) => {
-  test.skip(isDevMode, 'Turbopack intermittently returns 404 for dynamic routes in dev mode');
+  test.skip(isTurbopackDevMode, 'Turbopack intermittently returns 404 for dynamic routes in dev mode');
 
   const errorEventPromise = waitForError('nextjs-16', errorEvent => {
     return errorEvent?.exception?.values?.some(value => value.value === 'route-handler-capture-exception') ?? false;

dev-packages/e2e-tests/test-applications/nextjs-16/tests/server-components.test.ts

@@ -1,11 +1,9 @@
 import { expect, test } from '@playwright/test';
 import { waitForTransaction } from '@sentry-internal/test-utils';
-import { isDevMode } from './isDevMode';
+import { isTurbopackDevMode } from './isDevMode';
 
 test('Sends a transaction for a request to app router with URL', async ({ page }) => {
-  // Turbopack dev mode intermittently 404s on nested dynamic routes, and the SDK
-  // drops 404 server transactions, so the waitForTransaction times out.
-  test.skip(isDevMode, 'Turbopack intermittently returns 404 for nested dynamic routes in dev mode');
+  test.skip(isTurbopackDevMode, 'Turbopack intermittently returns 404 for nested dynamic routes in dev mode');
 
   const serverComponentTransactionPromise = waitForTransaction('nextjs-16', transactionEvent => {
     return (
@@ -79,6 +77,8 @@ test('Will create a transaction with spans for every server component and metada
 test('Will create a transaction with spans for every server component and metadata generation functions when visiting a dynamic page', async ({
   page,
 }) => {
+  test.skip(isTurbopackDevMode, 'Turbopack intermittently returns 404 for dynamic routes in dev mode');
+
   const serverTransactionEventPromise = waitForTransaction('nextjs-16', async transactionEvent => {
     return transactionEvent?.transaction === 'GET /nested-layout/[dynamic]';
   });