Skip to content

Commit

Permalink
test(replay): Remove outdated test constraint (#12270)
Browse files Browse the repository at this point in the history
Noticed this while looking into
#12244, console
should be instrumented just fine in the CDN bundle.
  • Loading branch information
mydea committed May 29, 2024
1 parent 005f40d commit 82f4432
Showing 1 changed file with 2 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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();
}

Expand Down Expand Up @@ -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();
}

Expand Down

0 comments on commit 82f4432

Please sign in to comment.