adjust integration test

Author: Lms24

dev-packages/browser-integration-tests/suites/tracing/metrics/pageload-resource-spans/test.ts

@@ -4,7 +4,7 @@ import { type Event, SEMANTIC_ATTRIBUTE_SENTRY_OP, SEMANTIC_ATTRIBUTE_SENTRY_ORI
 import { sentryTest } from '../../../../utils/fixtures';
 import { getFirstSentryEnvelopeRequest, shouldSkipTracingTest } from '../../../../utils/helpers';
 
-sentryTest('should add resource spans to pageload transaction', async ({ getLocalTestUrl, page, browserName }) => {
+sentryTest('adds resource spans to pageload transaction', async ({ getLocalTestUrl, page, browserName }) => {
   if (shouldSkipTracingTest()) {
     sentryTest.skip();
   }
@@ -74,6 +74,21 @@ sentryTest('should add resource spans to pageload transaction', async ({ getLoca
       'http.decoded_response_content_length': expect.any(Number),
       'http.response_content_length': expect.any(Number),
       'http.response_transfer_size': expect.any(Number),
+      'http.request.connect_end': expect.any(Number),
+      'http.request.connect_start': expect.any(Number),
+      'http.request.domain_lookup_end': expect.any(Number),
+      'http.request.domain_lookup_start': expect.any(Number),
+      'http.request.fetch_start': expect.any(Number),
+      'http.request.redirect_end': expect.any(Number),
+      'http.request.redirect_start': expect.any(Number),
+      'http.request.request_start': expect.any(Number),
+      'http.request.secure_connection_start': expect.any(Number),
+      'http.request.worker_start': expect.any(Number),
+      'http.response.end': expect.any(Number),
+      'http.response.final_response_headers_start': expect.any(Number),
+      'http.response.first_interim_response_start': expect.any(Number),
+      'http.response.start': expect.any(Number),
+      'http.response.status_code': expect.any(Number),
       'network.protocol.name': '',
       'network.protocol.version': 'unknown',
       [SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'resource.img',
@@ -101,6 +116,21 @@ sentryTest('should add resource spans to pageload transaction', async ({ getLoca
       'http.decoded_response_content_length': expect.any(Number),
       'http.response_content_length': expect.any(Number),
       'http.response_transfer_size': expect.any(Number),
+      'http.request.connect_end': expect.any(Number),
+      'http.request.connect_start': expect.any(Number),
+      'http.request.domain_lookup_end': expect.any(Number),
+      'http.request.domain_lookup_start': expect.any(Number),
+      'http.request.fetch_start': expect.any(Number),
+      'http.request.redirect_end': expect.any(Number),
+      'http.request.redirect_start': expect.any(Number),
+      'http.request.request_start': expect.any(Number),
+      'http.request.secure_connection_start': expect.any(Number),
+      'http.request.worker_start': expect.any(Number),
+      'http.response.end': expect.any(Number),
+      'http.response.final_response_headers_start': expect.any(Number),
+      'http.response.first_interim_response_start': expect.any(Number),
+      'http.response.start': expect.any(Number),
+      'http.response.status_code': expect.any(Number),
       'network.protocol.name': '',
       'network.protocol.version': 'unknown',
       [SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'resource.link',
@@ -128,6 +158,21 @@ sentryTest('should add resource spans to pageload transaction', async ({ getLoca
       'http.decoded_response_content_length': expect.any(Number),
       'http.response_content_length': expect.any(Number),
       'http.response_transfer_size': expect.any(Number),
+      'http.request.connect_end': expect.any(Number),
+      'http.request.connect_start': expect.any(Number),
+      'http.request.domain_lookup_end': expect.any(Number),
+      'http.request.domain_lookup_start': expect.any(Number),
+      'http.request.fetch_start': expect.any(Number),
+      'http.request.redirect_end': expect.any(Number),
+      'http.request.redirect_start': expect.any(Number),
+      'http.request.request_start': expect.any(Number),
+      'http.request.secure_connection_start': expect.any(Number),
+      'http.request.worker_start': expect.any(Number),
+      'http.response.end': expect.any(Number),
+      'http.response.final_response_headers_start': expect.any(Number),
+      'http.response.first_interim_response_start': expect.any(Number),
+      'http.response.start': expect.any(Number),
+      'http.response.status_code': expect.any(Number),
       'network.protocol.name': '',
       'network.protocol.version': 'unknown',
       'sentry.op': 'resource.script',

packages/browser-utils/src/metrics/browserMetrics.ts

@@ -808,7 +808,10 @@ type ExperimentalResourceTimingProperty =
   | 'firstInterimResponseStart'
   | 'finalResponseHeadersStart'
   | 'renderBlockingStatus'
-  | 'deliveryType';
+  | 'deliveryType'
+  // for some reason, TS during build things this is not a property of PerformanceResourceTiming
+  // while it actually is. Hence, we're adding it here
+  | 'responseStatus';
 
 /**
  * Use this to set any attributes we can take directly form the PerformanceResourceTiming entry.