Skip to content

Commit e905ec4

Browse files
committed
skip tests for non-tracing bundles
1 parent b078eea commit e905ec4

File tree

1 file changed

+15
-1
lines changed
  • dev-packages/browser-integration-tests/suites/profiling/legacyMode

1 file changed

+15
-1
lines changed

dev-packages/browser-integration-tests/suites/profiling/legacyMode/test.ts

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,18 @@
11
import { expect } from '@playwright/test';
22
import type { Event, Profile } from '@sentry/core';
33
import { sentryTest } from '../../../utils/fixtures';
4-
import { properEnvelopeRequestParser, waitForTransactionRequestOnUrl } from '../../../utils/helpers';
4+
import {
5+
properEnvelopeRequestParser,
6+
shouldSkipTracingTest,
7+
waitForTransactionRequestOnUrl,
8+
} from '../../../utils/helpers';
59

610
sentryTest('does not send profile envelope when document-policy is not set', async ({ page, getLocalTestUrl }) => {
11+
if (shouldSkipTracingTest()) {
12+
// Profiling only works when tracing is enabled
13+
sentryTest.skip();
14+
}
15+
716
const url = await getLocalTestUrl({ testDir: __dirname });
817

918
const req = await waitForTransactionRequestOnUrl(page, url);
@@ -16,6 +25,11 @@ sentryTest('does not send profile envelope when document-policy is not set', asy
1625
});
1726

1827
sentryTest('sends profile envelope in legacy mode', async ({ page, getLocalTestUrl }) => {
28+
if (shouldSkipTracingTest()) {
29+
// Profiling only works when tracing is enabled
30+
sentryTest.skip();
31+
}
32+
1933
const url = await getLocalTestUrl({ testDir: __dirname, responseHeaders: { 'Document-Policy': 'js-profiling' } });
2034

2135
const req = await waitForTransactionRequestOnUrl(page, url);

0 commit comments

Comments
 (0)