fix tests

Author: nicohrubec

.size-limit.js

@@ -276,7 +276,7 @@ module.exports = [
     path: createCDNPath('bundle.tracing.replay.min.js'),
     gzip: false,
     brotli: false,
-    limit: '255 KB',
+    limit: '255.5 KB',
   },
   {
     name: 'CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed',

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

@@ -99,7 +99,7 @@ sentryTest('captures streamed interaction span tree. @firefox', async ({ browser
   });
 
   const loAFSpans = interactionSpanTree.filter(span => getSpanOp(span)?.startsWith('ui.long-animation-frame'));
-  expect(loAFSpans).toHaveLength(1);
+  expect(loAFSpans).toHaveLength(browserName === 'chromium' ? 1 : 0);
 
   const interactionSpan = interactionSpanTree.find(span => getSpanOp(span) === 'ui.interaction.click');
   expect(interactionSpan).toEqual({

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

@@ -15,7 +15,7 @@ import {
   waitForStreamedSpanEnvelope,
 } from '../../../../utils/spanUtils';
 
-sentryTest('starts a streamed navigation span on page navigation', async ({ getLocalTestUrl, page }) => {
+sentryTest('starts a streamed navigation span on page navigation', async ({ browserName, getLocalTestUrl, page }) => {
   sentryTest.skip(shouldSkipTracingTest());
 
   const pageloadSpanPromise = waitForStreamedSpan(page, span => getSpanOp(span) === 'pageload');
@@ -81,18 +81,20 @@ sentryTest('starts a streamed navigation span on page navigation', async ({ getL
         type: 'string',
         value: expect.any(String),
       },
-      'network.connection.effective_type': {
-        type: 'string',
-        value: expect.any(String),
-      },
       'device.processor_count': {
         type: expect.stringMatching(/^(integer)|(double)$/),
         value: expect.any(Number),
       },
-      'network.connection.rtt': {
-        type: expect.stringMatching(/^(integer)|(double)$/),
-        value: expect.any(Number),
-      },
+      ...(browserName !== 'webkit' && {
+        'network.connection.effective_type': {
+          type: 'string',
+          value: expect.any(String),
+        },
+        'network.connection.rtt': {
+          type: expect.stringMatching(/^(integer)|(double)$/),
+          value: expect.any(Number),
+        },
+      }),
       'sentry.idle_span_finish_reason': {
         type: 'string',
         value: 'idleTimeout',

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

@@ -16,7 +16,7 @@ import { getSpanOp, getSpansFromEnvelope, waitForStreamedSpanEnvelope } from '..
 
 sentryTest(
   'creates a pageload streamed span envelope with url as pageload span name source',
-  async ({ getLocalTestUrl, page }) => {
+  async ({ browserName, getLocalTestUrl, page }) => {
     sentryTest.skip(shouldSkipTracingTest());
 
     const spanEnvelopePromise = waitForStreamedSpanEnvelope(
@@ -74,11 +74,6 @@ sentryTest(
           type: 'string',
           value: expect.any(String),
         },
-        // formerly known as 'effectiveConnectionType'
-        'network.connection.effective_type': {
-          type: 'string',
-          value: expect.any(String),
-        },
         // formerly known as 'hardwareConcurrency'
         'device.processor_count': {
           type: expect.stringMatching(/^(integer)|(double)$/),
@@ -92,18 +87,25 @@ sentryTest(
           type: expect.stringMatching(/^(integer)|(double)$/),
           value: expect.any(Number),
         },
-        'network.connection.rtt': {
-          type: expect.stringMatching(/^(integer)|(double)$/),
-          value: expect.any(Number),
-        },
         'browser.web_vital.ttfb.request_time': {
           type: expect.stringMatching(/^(integer)|(double)$/),
           value: expect.any(Number),
         },
-        'browser.web_vital.ttfb.value': {
-          type: expect.stringMatching(/^(integer)|(double)$/),
-          value: expect.any(Number),
-        },
+        ...(browserName !== 'webkit' && {
+          // formerly known as 'effectiveConnectionType'
+          'network.connection.effective_type': {
+            type: 'string',
+            value: expect.any(String),
+          },
+          'network.connection.rtt': {
+            type: expect.stringMatching(/^(integer)|(double)$/),
+            value: expect.any(Number),
+          },
+          'browser.web_vital.ttfb.value': {
+            type: expect.stringMatching(/^(integer)|(double)$/),
+            value: expect.any(Number),
+          },
+        }),
         'sentry.idle_span_finish_reason': {
           type: 'string',
           value: 'idleTimeout',

dev-packages/browser-integration-tests/suites/tracing/request/fetch-streamed/test.ts

@@ -19,7 +19,11 @@ sentryTest('creates spans for fetch requests', async ({ getLocalTestUrl, page })
 
   const allSpans = await spansPromise;
   const pageloadSpan = allSpans.find(s => getSpanOp(s) === 'pageload');
-  const requestSpans = allSpans.filter(s => getSpanOp(s) === 'http.client');
+  const requestSpans = allSpans
+    .filter(s => getSpanOp(s) === 'http.client')
+    .sort((a, b) =>
+      (a.attributes!['http.url']!.value as string).localeCompare(b.attributes!['http.url']!.value as string),
+    );
 
   expect(requestSpans).toHaveLength(3);