clean up dev tests

Author: chargome

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

@@ -1,11 +1,9 @@
 import { expect, test } from '@playwright/test';
 import fs from 'fs';
+import { isDevMode } from './isDevMode';
 
 test('should not print warning for async params', async ({ page }) => {
-  test.skip(
-    process.env.TEST_ENV !== 'development' && process.env.TEST_ENV !== 'dev-turbopack',
-    'should be skipped for non-dev mode',
-  );
+  test.skip(!isDevMode, 'should be skipped for non-dev mode');
   await page.goto('/');
 
   // If the server exits with code 1, the test will fail (see instrumentation.ts)

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

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

dev-packages/e2e-tests/test-applications/nextjs-16/tests/nested-rsc-error.test.ts

@@ -1,20 +1,9 @@
 import { expect, test } from '@playwright/test';
 import { waitForError, waitForTransaction } from '@sentry-internal/test-utils';
 
-const packageJson = require('../package.json');
-
 test('Should capture errors from nested server components when `Sentry.captureRequestError` is added to the `onRequestError` hook', async ({
   page,
 }) => {
-  const [, minor, patch, canary] = packageJson.dependencies.next.split('.');
-
-  test.skip(
-    minor === '0' &&
-      patch.startsWith('0-') &&
-      ((patch.includes('canary') && Number(canary) < 63) || patch.includes('rc')),
-    'Next.js version does not expose these errors',
-  );
-
   const errorEventPromise = waitForError('nextjs-16', errorEvent => {
     return !!errorEvent?.exception?.values?.some(value => value.value === 'I am technically uncatchable');
   });

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

@@ -1,11 +1,9 @@
 import { expect, test } from '@playwright/test';
 import { waitForTransaction } from '@sentry-internal/test-utils';
+import { isDevMode } from './isDevMode';
 
 test('Prefetch client spans should have a http.request.prefetch attribute', async ({ page }) => {
-  test.skip(
-    process.env.TEST_ENV === 'development' || process.env.TEST_ENV === 'dev-turbopack',
-    "Prefetch requests don't have the prefetch header in dev mode",
-  );
+  test.skip(isDevMode, "Prefetch requests don't have the prefetch header in dev mode");
 
   const pageloadTransactionPromise = waitForTransaction('nextjs-16', async transactionEvent => {
     return transactionEvent?.transaction === '/prefetching';

dev-packages/e2e-tests/test-applications/nextjs-16/tests/streaming-rsc-error.test.ts

@@ -1,20 +1,9 @@
 import { expect, test } from '@playwright/test';
 import { waitForError, waitForTransaction } from '@sentry-internal/test-utils';
 
-const packageJson = require('../package.json');
-
 test('Should capture errors for crashing streaming promises in server components when `Sentry.captureRequestError` is added to the `onRequestError` hook', async ({
   page,
 }) => {
-  const [, minor, patch, canary] = packageJson.dependencies.next.split('.');
-
-  test.skip(
-    minor === '0' &&
-      patch.startsWith('0-') &&
-      ((patch.includes('canary') && Number(canary) < 63) || patch.includes('rc')),
-    'Next.js version does not expose these errors',
-  );
-
   const errorEventPromise = waitForError('nextjs-16', errorEvent => {
     return !!errorEvent?.exception?.values?.some(value => value.value === 'I am a data streaming error');
   });