Skip to content

Commit a0df026

Browse files
committed
skip webpack dev tests
1 parent b1e0687 commit a0df026

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

dev-packages/e2e-tests/test-applications/nextjs-app-dir/tests/client-errors.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ const packageJson = require('../package.json');
66
test('Sends a client-side exception to Sentry', async ({ page }) => {
77
const nextjsVersion = packageJson.dependencies.next;
88
const nextjsMajor = Number(nextjsVersion.split('.')[0]);
9-
const isDevMode = process.env.TEST_ENV === 'development';
9+
const isDevMode = process.env.TEST_ENV.includes('development');
1010

1111
await page.goto('/');
1212

dev-packages/e2e-tests/test-applications/nextjs-app-dir/tests/devErrorSymbolification.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import { expect, test } from '@playwright/test';
22
import { waitForError } from '@sentry-internal/test-utils';
33

44
test('should have symbolicated dev errors', async ({ page }) => {
5-
test.skip(process.env.TEST_ENV !== 'development', 'should be skipped for non-dev mode');
5+
test.skip(!process.env.TEST_ENV.includes('development'), 'should be skipped for non-dev mode');
66

77
await page.goto('/');
88

dev-packages/e2e-tests/test-applications/nextjs-app-dir/tests/transactions.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ const packageJson = require('../package.json');
66
test('Sends a pageload transaction', async ({ page }) => {
77
const nextjsVersion = packageJson.dependencies.next;
88
const nextjsMajor = Number(nextjsVersion.split('.')[0]);
9-
const isDevMode = process.env.TEST_ENV === 'development';
9+
const isDevMode = process.env.TEST_ENV.includes('development');
1010

1111
const pageloadTransactionEventPromise = waitForTransaction('nextjs-app-dir', transactionEvent => {
1212
return transactionEvent?.contexts?.trace?.op === 'pageload' && transactionEvent?.transaction === '/';
@@ -79,7 +79,7 @@ test('Should send a wrapped server action as a child of a nextjs transaction', a
7979
const nextjsVersion = packageJson.dependencies.next;
8080
const nextjsMajor = Number(nextjsVersion.split('.')[0]);
8181
test.skip(!isNaN(nextjsMajor) && nextjsMajor < 14, 'only applies to nextjs apps >= version 14');
82-
test.skip(process.env.TEST_ENV === 'development', 'this magically only works in production');
82+
test.skip(process.env.TEST_ENV.includes('development'), 'this magically only works in production');
8383

8484
const nextjsPostTransactionPromise = waitForTransaction('nextjs-app-dir', async transactionEvent => {
8585
return (

0 commit comments

Comments
 (0)