From 82f44327e435430201a43dda1d7ae8434e3261f9 Mon Sep 17 00:00:00 2001 From: Francesco Novy Date: Wed, 29 May 2024 09:42:06 +0200 Subject: [PATCH] test(replay): Remove outdated test constraint (#12270) Noticed this while looking into https://github.com/getsentry/sentry-javascript/issues/12244, console should be instrumented just fine in the CDN bundle. --- .../suites/replay/captureConsoleLog/test.ts | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/dev-packages/browser-integration-tests/suites/replay/captureConsoleLog/test.ts b/dev-packages/browser-integration-tests/suites/replay/captureConsoleLog/test.ts index d773cd553c05..8ea7d10bd158 100644 --- a/dev-packages/browser-integration-tests/suites/replay/captureConsoleLog/test.ts +++ b/dev-packages/browser-integration-tests/suites/replay/captureConsoleLog/test.ts @@ -4,9 +4,7 @@ import { sentryTest } from '../../../utils/fixtures'; import { getCustomRecordingEvents, shouldSkipReplayTest, waitForReplayRequest } from '../../../utils/replayHelpers'; sentryTest('should capture console messages in replay', async ({ getLocalTestPath, page, forceFlushReplay }) => { - // console integration is not used in bundles/loader - const bundle = process.env.PW_BUNDLE || ''; - if (shouldSkipReplayTest() || bundle.startsWith('bundle_') || bundle.startsWith('loader_')) { + if (shouldSkipReplayTest()) { sentryTest.skip(); } @@ -57,9 +55,7 @@ sentryTest('should capture console messages in replay', async ({ getLocalTestPat }); sentryTest('should capture very large console logs', async ({ getLocalTestPath, page, forceFlushReplay }) => { - // console integration is not used in bundles/loader - const bundle = process.env.PW_BUNDLE || ''; - if (shouldSkipReplayTest() || bundle.startsWith('bundle_') || bundle.startsWith('loader_')) { + if (shouldSkipReplayTest()) { sentryTest.skip(); }