CI

Author: Luca Forstner

packages/tracing-internal/test/browser/request.test.ts

@@ -1,15 +1,15 @@
 /* eslint-disable deprecation/deprecation */
 import * as sentryCore from '@sentry/core';
+import type { HandlerDataFetch } from '@sentry/types';
 import * as utils from '@sentry/utils';
 import { SENTRY_XHR_DATA_KEY } from '@sentry/utils';
 
 import type { Transaction } from '../../../tracing/src';
 import { addExtensionMethods, Span, spanStatusfromHttpCode } from '../../../tracing/src';
 import { getDefaultBrowserClientOptions } from '../../../tracing/test/testutils';
-import type { FetchData, XHRData } from '../../src/browser/request';
+import type { XHRData } from '../../src/browser/request';
 import {
   extractNetworkProtocol,
-  fetchCallback,
   instrumentOutgoingRequests,
   shouldAttachHeaders,
   xhrCallback,
@@ -76,7 +76,7 @@ describe('callbacks', () => {
   });
 
   describe('fetchCallback()', () => {
-    let fetchHandlerData: FetchData;
+    let fetchHandlerData: HandlerDataFetch;
 
     const fetchSpan = {
       data: {
@@ -109,7 +109,7 @@ describe('callbacks', () => {
     ])(
       'span creation/header attachment interaction - shouldCreateSpan: %s, shouldAttachHeaders: %s',
       (shouldCreateSpanReturnValue, shouldAttachHeadersReturnValue, expectedSpan, expectedHeaderKeys) => {
-        fetchCallback(
+        sentryCore.instrumentFetchRequest(
           fetchHandlerData,
           () => shouldCreateSpanReturnValue,
           () => shouldAttachHeadersReturnValue,
@@ -126,10 +126,11 @@ describe('callbacks', () => {
     );
 
     it('adds neither fetch request spans nor fetch request headers if there is no fetch data in handler data', () => {
+      // @ts-expect-error asdfasdfa
       delete fetchHandlerData.fetchData;
       const spans = {};
 
-      fetchCallback(fetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
+      sentryCore.instrumentFetchRequest(fetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
 
       expect(spans).toEqual({});
 
@@ -141,7 +142,7 @@ describe('callbacks', () => {
       hasTracingEnabled.mockReturnValueOnce(false);
       const spans = {};
 
-      fetchCallback(fetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
+      sentryCore.instrumentFetchRequest(fetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
 
       expect(spans).toEqual({});
 
@@ -153,7 +154,7 @@ describe('callbacks', () => {
       const spans = {};
 
       // triggered by request being sent
-      fetchCallback(fetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
+      sentryCore.instrumentFetchRequest(fetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
 
       const newSpan = transaction.spanRecorder?.spans[1] as Span;
 
@@ -174,7 +175,7 @@ describe('callbacks', () => {
       };
 
       // triggered by response coming back
-      fetchCallback(postRequestFetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
+      sentryCore.instrumentFetchRequest(postRequestFetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
 
       expect(newSpan.endTimestamp).toBeDefined();
     });
@@ -183,7 +184,7 @@ describe('callbacks', () => {
       const spans: Record<string, Span> = {};
 
       // triggered by request being sent
-      fetchCallback(fetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
+      sentryCore.instrumentFetchRequest(fetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
 
       const newSpan = transaction.spanRecorder?.spans[1] as Span;
 
@@ -196,7 +197,7 @@ describe('callbacks', () => {
       };
 
       // triggered by response coming back
-      fetchCallback(postRequestFetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
+      sentryCore.instrumentFetchRequest(postRequestFetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
 
       expect(newSpan.status).toBe(spanStatusfromHttpCode(404));
     });
@@ -211,7 +212,7 @@ describe('callbacks', () => {
       };
 
       // in that case, the response coming back will be ignored
-      fetchCallback(postRequestFetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, {});
+      sentryCore.instrumentFetchRequest(postRequestFetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, {});
 
       const newSpan = transaction.spanRecorder?.spans[1];
 
@@ -221,7 +222,7 @@ describe('callbacks', () => {
     it('uses active span to generate sentry-trace header', () => {
       const spans: Record<string, Span> = {};
       // triggered by request being sent
-      fetchCallback(fetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
+      sentryCore.instrumentFetchRequest(fetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
 
       const activeSpan = transaction.spanRecorder?.spans[1] as Span;
 
@@ -232,7 +233,7 @@ describe('callbacks', () => {
       };
 
       // triggered by response coming back
-      fetchCallback(postRequestFetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
+      sentryCore.instrumentFetchRequest(postRequestFetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
 
       const headers = (fetchHandlerData.args[1].headers as Record<string, string>) || {};
       expect(headers['sentry-trace']).toEqual(`${activeSpan.traceId}-${activeSpan.spanId}-1`);
@@ -242,7 +243,7 @@ describe('callbacks', () => {
       const spans: Record<string, Span> = {};
 
       // triggered by request being sent
-      fetchCallback(fetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
+      sentryCore.instrumentFetchRequest(fetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
 
       const newSpan = transaction.spanRecorder?.spans[1] as Span;
 
@@ -252,10 +253,10 @@ describe('callbacks', () => {
         ...fetchHandlerData,
         endTimestamp,
         response: { status: 404, headers: { get: () => 123 } },
-      };
+      } as unknown as HandlerDataFetch;
 
       // triggered by response coming back
-      fetchCallback(postRequestFetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
+      sentryCore.instrumentFetchRequest(postRequestFetchHandlerData, alwaysCreateSpan, alwaysAttachHeaders, spans);
 
       const finishedSpan = transaction.spanRecorder?.spans[1] as Span;