Skip to content

Commit fea706d

Browse files
Lms24Lms24
committed
skip tests for cdn bundles
1 parent 9f490f4 commit fea706d

36 files changed

Lines changed: 89 additions & 84 deletions

File tree

‎dev-packages/browser-integration-tests/suites/public-api/startSpan/streamed/test.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,13 @@ import {
1111
SEMANTIC_ATTRIBUTE_SENTRY_SOURCE,
1212
} from '@sentry/core';
1313
import { sentryTest } from '../../../../utils/fixtures';
14-
import { shouldSkipTracingTest } from '../../../../utils/helpers';
14+
import { shouldSkipTracingTest, testingCdnBundle } from '../../../../utils/helpers';
1515
import { waitForStreamedSpanEnvelope } from '../../../../utils/spanUtils';
1616

1717
sentryTest(
1818
'sends a streamed span envelope if spanStreamingIntegration is enabled',
1919
async ({ getLocalTestUrl, page }) => {
20-
sentryTest.skip(shouldSkipTracingTest());
20+
sentryTest.skip(shouldSkipTracingTest() || testingCdnBundle());
2121

2222
const spanEnvelopePromise = waitForStreamedSpanEnvelope(page);
2323

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/backgroundtab-pageload-streamed/test.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
import { expect } from '@playwright/test';
22
import { sentryTest } from '../../../../utils/fixtures';
3-
import { shouldSkipTracingTest } from '../../../../utils/helpers';
3+
import { shouldSkipTracingTest, testingCdnBundle } from '../../../../utils/helpers';
44
import { getSpanOp, waitForStreamedSpan } from '../../../../utils/spanUtils';
55

66
sentryTest('finishes streamed pageload span when the page goes background', async ({ getLocalTestUrl, page }) => {
7-
sentryTest.skip(shouldSkipTracingTest());
7+
sentryTest.skip(shouldSkipTracingTest() || testingCdnBundle());
88
const url = await getLocalTestUrl({ testDir: __dirname });
99
const pageloadSpanPromise = waitForStreamedSpan(page, span => getSpanOp(span) === 'pageload');
1010

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/http-timings-streamed/test.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { expect } from '@playwright/test';
22
import { sentryTest } from '../../../../utils/fixtures';
3-
import { shouldSkipTracingTest } from '../../../../utils/helpers';
3+
import { shouldSkipTracingTest, testingCdnBundle } from '../../../../utils/helpers';
44
import { getSpanOp, waitForStreamedSpans } from '../../../../utils/spanUtils';
55

66
/**
@@ -16,7 +16,7 @@ sentryTest(
1616
async ({ browserName, getLocalTestUrl, page }) => {
1717
const supportedBrowsers = ['chromium', 'firefox'];
1818

19-
sentryTest.skip(shouldSkipTracingTest() || !supportedBrowsers.includes(browserName));
19+
sentryTest.skip(shouldSkipTracingTest() || !supportedBrowsers.includes(browserName) || testingCdnBundle());
2020

2121
await page.route('http://sentry-test-site.example/*', async route => {
2222
const request = route.request();

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/interactions-streamed/test.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,13 @@ import {
1111
SEMANTIC_ATTRIBUTE_SENTRY_SOURCE,
1212
} from '@sentry/core';
1313
import { sentryTest } from '../../../../utils/fixtures';
14-
import { shouldSkipTracingTest } from '../../../../utils/helpers';
14+
import { shouldSkipTracingTest, testingCdnBundle } from '../../../../utils/helpers';
1515
import { getSpanOp, waitForStreamedSpan, waitForStreamedSpans } from '../../../../utils/spanUtils';
1616

1717
sentryTest('captures streamed interaction span tree. @firefox', async ({ browserName, getLocalTestUrl, page }) => {
1818
const supportedBrowsers = ['chromium', 'firefox'];
1919

20-
sentryTest.skip(shouldSkipTracingTest() || !supportedBrowsers.includes(browserName));
20+
sentryTest.skip(shouldSkipTracingTest() || !supportedBrowsers.includes(browserName) || testingCdnBundle());
2121
const url = await getLocalTestUrl({ testDir: __dirname });
2222

2323
const interactionSpansPromise = waitForStreamedSpans(page, spans =>

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/linked-traces-streamed/consistent-sampling/default/test.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
import { expect } from '@playwright/test';
22
import { extractTraceparentData, parseBaggageHeader, SEMANTIC_LINK_ATTRIBUTE_LINK_TYPE } from '@sentry/core';
33
import { sentryTest } from '../../../../../../utils/fixtures';
4-
import { shouldSkipTracingTest, waitForTracingHeadersOnUrl } from '../../../../../../utils/helpers';
4+
import { shouldSkipTracingTest, testingCdnBundle, waitForTracingHeadersOnUrl } from '../../../../../../utils/helpers';
55
import { getSpanOp, waitForStreamedSpanEnvelope } from '../../../../../../utils/spanUtils';
66

77
sentryTest.describe('When `consistentTraceSampling` is `true`', () => {
88
sentryTest('continues sampling decision from initial pageload span', async ({ getLocalTestUrl, page }) => {
9-
sentryTest.skip(shouldSkipTracingTest());
9+
sentryTest.skip(shouldSkipTracingTest() || testingCdnBundle());
1010

1111
const url = await getLocalTestUrl({ testDir: __dirname });
1212

@@ -80,7 +80,7 @@ sentryTest.describe('When `consistentTraceSampling` is `true`', () => {
8080
});
8181

8282
sentryTest('Propagates continued sampling decision to outgoing requests', async ({ page, getLocalTestUrl }) => {
83-
sentryTest.skip(shouldSkipTracingTest());
83+
sentryTest.skip(shouldSkipTracingTest() || testingCdnBundle());
8484

8585
const url = await getLocalTestUrl({ testDir: __dirname });
8686

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/linked-traces-streamed/consistent-sampling/meta-negative/test.ts‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import {
77
envelopeRequestParser,
88
hidePage,
99
shouldSkipTracingTest,
10+
testingCdnBundle,
1011
waitForClientReportRequest,
1112
waitForTracingHeadersOnUrl,
1213
} from '../../../../../../utils/helpers';
@@ -20,7 +21,7 @@ sentryTest.describe('When `consistentTraceSampling` is `true` and page contains
2021
sentryTest(
2122
'Continues negative sampling decision from meta tag across all traces and downstream propagations',
2223
async ({ getLocalTestUrl, page }) => {
23-
sentryTest.skip(shouldSkipTracingTest());
24+
sentryTest.skip(shouldSkipTracingTest() || testingCdnBundle());
2425

2526
const url = await getLocalTestUrl({ testDir: __dirname });
2627

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/linked-traces-streamed/consistent-sampling/meta-precedence/test.ts‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import {
66
envelopeRequestParser,
77
hidePage,
88
shouldSkipTracingTest,
9+
testingCdnBundle,
910
waitForClientReportRequest,
1011
waitForTracingHeadersOnUrl,
1112
} from '../../../../../../utils/helpers';
@@ -20,7 +21,7 @@ sentryTest.describe('When `consistentTraceSampling` is `true` and page contains
2021
sentryTest(
2122
'meta tag decision has precedence over sampling decision from previous trace in session storage',
2223
async ({ getLocalTestUrl, page }) => {
23-
sentryTest.skip(shouldSkipTracingTest());
24+
sentryTest.skip(shouldSkipTracingTest() || testingCdnBundle());
2425

2526
const url = await getLocalTestUrl({ testDir: __dirname });
2627

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/linked-traces-streamed/consistent-sampling/meta/test.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,15 @@ import {
55
SEMANTIC_ATTRIBUTE_SENTRY_PREVIOUS_TRACE_SAMPLE_RATE,
66
} from '@sentry/core';
77
import { sentryTest } from '../../../../../../utils/fixtures';
8-
import { shouldSkipTracingTest, waitForTracingHeadersOnUrl } from '../../../../../../utils/helpers';
8+
import { shouldSkipTracingTest, testingCdnBundle, waitForTracingHeadersOnUrl } from '../../../../../../utils/helpers';
99
import { getSpanOp, waitForStreamedSpanEnvelope } from '../../../../../../utils/spanUtils';
1010

1111
const metaTagSampleRand = 0.051121;
1212
const metaTagSampleRate = 0.2;
1313

1414
sentryTest.describe('When `consistentTraceSampling` is `true` and page contains <meta> tags', () => {
1515
sentryTest('Continues sampling decision across all traces from meta tag', async ({ getLocalTestUrl, page }) => {
16-
sentryTest.skip(shouldSkipTracingTest());
16+
sentryTest.skip(shouldSkipTracingTest() || testingCdnBundle());
1717

1818
const url = await getLocalTestUrl({ testDir: __dirname });
1919

@@ -96,7 +96,7 @@ sentryTest.describe('When `consistentTraceSampling` is `true` and page contains
9696
sentryTest(
9797
'Propagates continued <meta> tag sampling decision to outgoing requests',
9898
async ({ page, getLocalTestUrl }) => {
99-
sentryTest.skip(shouldSkipTracingTest());
99+
sentryTest.skip(shouldSkipTracingTest() || testingCdnBundle());
100100

101101
const url = await getLocalTestUrl({ testDir: __dirname });
102102

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/linked-traces-streamed/consistent-sampling/tracesSampler-precedence/test.ts‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import {
77
envelopeRequestParser,
88
hidePage,
99
shouldSkipTracingTest,
10+
testingCdnBundle,
1011
waitForClientReportRequest,
1112
} from '../../../../../../utils/helpers';
1213
import { getSpanOp, waitForStreamedSpanEnvelope } from '../../../../../../utils/spanUtils';
@@ -18,7 +19,7 @@ import { getSpanOp, waitForStreamedSpanEnvelope } from '../../../../../../utils/
1819
*/
1920
sentryTest.describe('When `consistentTraceSampling` is `true`', () => {
2021
sentryTest('explicit sampling decisions in `tracesSampler` have precedence', async ({ getLocalTestUrl, page }) => {
21-
sentryTest.skip(shouldSkipTracingTest());
22+
sentryTest.skip(shouldSkipTracingTest() || testingCdnBundle());
2223

2324
const url = await getLocalTestUrl({ testDir: __dirname });
2425

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/linked-traces-streamed/custom-trace/test.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
import { expect } from '@playwright/test';
22
import { SEMANTIC_LINK_ATTRIBUTE_LINK_TYPE } from '@sentry/core';
33
import { sentryTest } from '../../../../../utils/fixtures';
4-
import { shouldSkipTracingTest } from '../../../../../utils/helpers';
4+
import { shouldSkipTracingTest, testingCdnBundle } from '../../../../../utils/helpers';
55
import { getSpanOp, waitForStreamedSpan } from '../../../../../utils/spanUtils';
66

77
sentryTest('manually started custom traces are linked correctly in the chain', async ({ getLocalTestUrl, page }) => {
8-
sentryTest.skip(shouldSkipTracingTest());
8+
sentryTest.skip(shouldSkipTracingTest() || testingCdnBundle());
99

1010
const url = await getLocalTestUrl({ testDir: __dirname });
1111

0 commit comments

Comments
 (0)