skip integration tests for cdn bundles

Author: Lms24

dev-packages/browser-integration-tests/suites/span-first/backgroundtab-pageload/test.ts

@@ -1,12 +1,14 @@
 import { expect } from '@playwright/test';
 import { sentryTest } from '../../../utils/fixtures';
-import { shouldSkipTracingTest } from '../../../utils/helpers';
+import { shouldSkipTracingTest, testingCdnBundle } from '../../../utils/helpers';
 import { getSpanOp, waitForV2Spans } from '../../../utils/spanFirstUtils';
 
 sentryTest('ends pageload span when the page goes to background', async ({ getLocalTestUrl, page }) => {
-  if (shouldSkipTracingTest()) {
+  // for now, spanStreamingIntegration is only exported in the NPM package, so we skip the test for bundles.
+  if (shouldSkipTracingTest() || testingCdnBundle()) {
     sentryTest.skip();
   }
+
   const url = await getLocalTestUrl({ testDir: __dirname });
 
   const spanPromise = waitForV2Spans(page, spans => !!spans.find(span => getSpanOp(span) === 'pageload'));

dev-packages/browser-integration-tests/suites/span-first/error/test.ts

@@ -5,19 +5,17 @@ import {
   envelopeRequestParser,
   runScriptInSandbox,
   shouldSkipTracingTest,
+  testingCdnBundle,
   waitForErrorRequest,
 } from '../../../utils/helpers';
 import { getSpanOp, waitForV2Spans } from '../../../utils/spanFirstUtils';
 
 sentryTest(
   'puts the pageload span name onto an error event caught during pageload',
   async ({ getLocalTestUrl, page, browserName }) => {
-    if (browserName === 'webkit') {
-      // This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
-      sentryTest.skip();
-    }
-
-    if (shouldSkipTracingTest()) {
+    // for now, spanStreamingIntegration is only exported in the NPM package, so we skip the test for bundles.
+    // This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
+    if (shouldSkipTracingTest() || testingCdnBundle() || browserName === 'webkit') {
       sentryTest.skip();
     }
 

dev-packages/browser-integration-tests/suites/span-first/linked-traces/test.ts

@@ -1,11 +1,12 @@
 import { expect } from '@playwright/test';
 import { SEMANTIC_LINK_ATTRIBUTE_LINK_TYPE } from '@sentry/core';
 import { sentryTest } from '../../../utils/fixtures';
-import { shouldSkipTracingTest } from '../../../utils/helpers';
+import { shouldSkipTracingTest, testingCdnBundle } from '../../../utils/helpers';
 import { getSpanOp, waitForV2Spans } from '../../../utils/spanFirstUtils';
 
 sentryTest("navigation spans link back to previous trace's root span", async ({ getLocalTestUrl, page }) => {
-  if (shouldSkipTracingTest()) {
+  // for now, spanStreamingIntegration is only exported in the NPM package, so we skip the test for bundles.
+  if (shouldSkipTracingTest() || testingCdnBundle()) {
     sentryTest.skip();
   }
 
@@ -77,7 +78,8 @@ sentryTest("navigation spans link back to previous trace's root span", async ({
 });
 
 sentryTest("doesn't link between hard page reloads by default", async ({ getLocalTestUrl, page }) => {
-  if (shouldSkipTracingTest()) {
+  // for now, spanStreamingIntegration is only exported in the NPM package, so we skip the test for bundles.
+  if (shouldSkipTracingTest() || testingCdnBundle()) {
     sentryTest.skip();
   }
 

dev-packages/browser-integration-tests/suites/span-first/pageload/test.ts

@@ -1,10 +1,11 @@
 import { expect } from '@playwright/test';
 import { sentryTest } from '../../../utils/fixtures';
-import { shouldSkipTracingTest } from '../../../utils/helpers';
+import { shouldSkipTracingTest, testingCdnBundle } from '../../../utils/helpers';
 import { getSpanOp, waitForSpanV2Envelope } from '../../../utils/spanFirstUtils';
 
 sentryTest('sends a span v2 envelope for the pageload', async ({ getLocalTestUrl, page }) => {
-  if (shouldSkipTracingTest()) {
+  // for now, spanStreamingIntegration is only exported in the NPM package, so we skip the test for bundles.
+  if (shouldSkipTracingTest() || testingCdnBundle()) {
     sentryTest.skip();
   }
 

dev-packages/browser-integration-tests/suites/span-first/web-vitals/web-vitals-ttfb/test.ts

@@ -1,10 +1,11 @@
 import { expect } from '@playwright/test';
 import { sentryTest } from '../../../../utils/fixtures';
-import { shouldSkipTracingTest } from '../../../../utils/helpers';
+import { shouldSkipTracingTest, testingCdnBundle } from '../../../../utils/helpers';
 import { getSpanOp, waitForV2Spans } from '../../../../utils/spanFirstUtils';
 
 sentryTest('captures TTFB web vital', async ({ getLocalTestUrl, page }) => {
-  if (shouldSkipTracingTest()) {
+  // for now, spanStreamingIntegration is only exported in the NPM package, so we skip the test for bundles.
+  if (shouldSkipTracingTest() || testingCdnBundle()) {
     sentryTest.skip();
   }
   const pageloadSpansPromise = waitForV2Spans(page, spans => !!spans.find(span => getSpanOp(span) === 'pageload'));

dev-packages/browser-integration-tests/utils/helpers.ts

@@ -314,6 +314,14 @@ export function shouldSkipTracingTest(): boolean {
   return bundle != null && !bundle.includes('tracing') && !bundle.includes('esm') && !bundle.includes('cjs');
 }
 
+/**
+ * @returns `true` if we are testing a CDN bundle
+ */
+export function testingCdnBundle(): boolean {
+  const bundle = process.env.PW_BUNDLE;
+  return bundle != null;
+}
+
 /**
  * Today we always run feedback tests, but this can be used to guard this if we ever need to.
  */