Skip to content

Commit 82f4432

Browse files
authored
test(replay): Remove outdated test constraint (#12270)
Noticed this while looking into #12244, console should be instrumented just fine in the CDN bundle.
1 parent 005f40d commit 82f4432

File tree

1 file changed

+2
-6
lines changed
  • dev-packages/browser-integration-tests/suites/replay/captureConsoleLog

1 file changed

+2
-6
lines changed

dev-packages/browser-integration-tests/suites/replay/captureConsoleLog/test.ts

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,7 @@ import { sentryTest } from '../../../utils/fixtures';
44
import { getCustomRecordingEvents, shouldSkipReplayTest, waitForReplayRequest } from '../../../utils/replayHelpers';
55

66
sentryTest('should capture console messages in replay', async ({ getLocalTestPath, page, forceFlushReplay }) => {
7-
// console integration is not used in bundles/loader
8-
const bundle = process.env.PW_BUNDLE || '';
9-
if (shouldSkipReplayTest() || bundle.startsWith('bundle_') || bundle.startsWith('loader_')) {
7+
if (shouldSkipReplayTest()) {
108
sentryTest.skip();
119
}
1210

@@ -57,9 +55,7 @@ sentryTest('should capture console messages in replay', async ({ getLocalTestPat
5755
});
5856

5957
sentryTest('should capture very large console logs', async ({ getLocalTestPath, page, forceFlushReplay }) => {
60-
// console integration is not used in bundles/loader
61-
const bundle = process.env.PW_BUNDLE || '';
62-
if (shouldSkipReplayTest() || bundle.startsWith('bundle_') || bundle.startsWith('loader_')) {
58+
if (shouldSkipReplayTest()) {
6359
sentryTest.skip();
6460
}
6561

0 commit comments

Comments
 (0)