test(node): Remove axios in favor of using fetch

Author: mydea

dev-packages/node-integration-tests/package.json

@@ -39,7 +39,6 @@
     "ai": "^4.0.6",
     "amqplib": "^0.10.7",
     "apollo-server": "^3.11.1",
-    "axios": "^1.7.7",
     "body-parser": "^1.20.3",
     "connect": "^3.7.0",
     "cors": "^2.8.5",

dev-packages/node-integration-tests/suites/tracing/meta-tags/test.ts

@@ -12,15 +12,14 @@ describe('getTraceMetaTags', () => {
 
     const runner = createRunner(__dirname, 'server.js').start();
 
-    const response = await runner.makeRequest('get', '/test', {
+    const response = await runner.makeRequest<{ response: string }>('get', '/test', {
       headers: {
         'sentry-trace': `${traceId}-${parentSpanId}-1`,
         baggage: 'sentry-environment=production,sentry-sample_rand=0.42',
       },
     });
 
-    // @ts-ignore - response is defined, types just don't reflect it
-    const html = response?.response as unknown as string;
+    const html = response?.response;
 
     expect(html).toMatch(/<meta name="sentry-trace" content="cd7ee7a6fe3ebe7ab9c3271559bc203c-[a-z0-9]{16}-1"\/>/);
     expect(html).toContain('<meta name="baggage" content="sentry-environment=production,sentry-sample_rand=0.42"/>');
@@ -29,12 +28,11 @@ describe('getTraceMetaTags', () => {
   test('injects <meta> tags with new trace if no incoming headers', async () => {
     const runner = createRunner(__dirname, 'server.js').start();
 
-    const response = await runner.makeRequest('get', '/test');
+    const response = await runner.makeRequest<{ response: string }>('get', '/test');
 
-    // @ts-ignore - response is defined, types just don't reflect it
-    const html = response?.response as unknown as string;
+    const html = response?.response;
 
-    const traceId = html.match(/<meta name="sentry-trace" content="([a-z0-9]{32})-[a-z0-9]{16}-1"\/>/)?.[1];
+    const traceId = html?.match(/<meta name="sentry-trace" content="([a-z0-9]{32})-[a-z0-9]{16}-1"\/>/)?.[1];
     expect(traceId).not.toBeUndefined();
 
     expect(html).toContain('<meta name="baggage"');
@@ -44,12 +42,11 @@ describe('getTraceMetaTags', () => {
   test('injects <meta> tags with negative sampling decision if tracesSampleRate is 0', async () => {
     const runner = createRunner(__dirname, 'server-tracesSampleRate-zero.js').start();
 
-    const response = await runner.makeRequest('get', '/test');
+    const response = await runner.makeRequest<{ response: string }>('get', '/test');
 
-    // @ts-ignore - response is defined, types just don't reflect it
-    const html = response?.response as unknown as string;
+    const html = response?.response;
 
-    const traceId = html.match(/<meta name="sentry-trace" content="([a-z0-9]{32})-[a-z0-9]{16}-0"\/>/)?.[1];
+    const traceId = html?.match(/<meta name="sentry-trace" content="([a-z0-9]{32})-[a-z0-9]{16}-0"\/>/)?.[1];
     expect(traceId).not.toBeUndefined();
 
     expect(html).toContain('<meta name="baggage"');
@@ -63,15 +60,14 @@ describe('getTraceMetaTags', () => {
 
     const runner = createRunner(__dirname, 'server-sdk-disabled.js').start();
 
-    const response = await runner.makeRequest('get', '/test', {
+    const response = await runner.makeRequest<{ response: string }>('get', '/test', {
       headers: {
         'sentry-trace': `${traceId}-${parentSpanId}-1`,
         baggage: 'sentry-environment=production',
       },
     });
 
-    // @ts-ignore - response is defined, types just don't reflect it
-    const html = response?.response as unknown as string;
+    const html = response?.response;
 
     expect(html).not.toContain('"sentry-trace"');
     expect(html).not.toContain('"baggage"');

dev-packages/node-integration-tests/utils/runner.ts

@@ -12,7 +12,6 @@ import type {
   TransactionEvent,
 } from '@sentry/core';
 import { normalize } from '@sentry/core';
-import axios from 'axios';
 import { execSync, spawn, spawnSync } from 'child_process';
 import { existsSync, readFileSync, unlinkSync, writeFileSync } from 'fs';
 import { join } from 'path';
@@ -532,7 +531,7 @@ export function createRunner(...paths: string[]) {
         makeRequest: async function <T>(
           method: 'get' | 'post',
           path: string,
-          options: { headers?: Record<string, string>; data?: unknown; expectError?: boolean } = {},
+          options: { headers?: Record<string, string>; data?: Record<string, unknown>; expectError?: boolean } = {},
         ): Promise<T | undefined> {
           try {
             await waitFor(() => scenarioServerPort !== undefined, 10_000, 'Timed out waiting for server port');
@@ -548,16 +547,31 @@ export function createRunner(...paths: string[]) {
 
           if (process.env.DEBUG) log('making request', method, url, headers, data);
 
+          if (data) {
+            headers['Content-Type'] = 'application/json';
+          }
+
           try {
-            const res =
-              method === 'post' ? await axios.post(url, data, { headers }) : await axios.get(url, { headers });
+            const res = await fetch(url, { headers, method, body: data ? JSON.stringify(data) : undefined });
+
+            if (!res.ok) {
+              if (!expectError) {
+                complete(new Error(`Expected request to "${path}" to succeed, but got a ${res.status} response`));
+              }
+
+              return;
+            }
 
             if (expectError) {
               complete(new Error(`Expected request to "${path}" to fail, but got a ${res.status} response`));
               return;
             }
 
-            return res.data;
+            if (res.headers.get('content-type')?.includes('application/json')) {
+              return await res.json();
+            }
+
+            return (await res.text()) as T;
           } catch (e) {
             if (expectError) {
               return;

yarn.lock

@@ -10168,7 +10168,7 @@ aws-ssl-profiles@^1.1.1:
   resolved "https://registry.yarnpkg.com/aws-ssl-profiles/-/aws-ssl-profiles-1.1.2.tgz#157dd77e9f19b1d123678e93f120e6f193022641"
   integrity sha512-NZKeq9AfyQvEeNlN0zSYAaWrmBffJh3IELMZfRpJVWgrpEbtEpnjvzqBPf+mxoI287JohRDoa+/nsfqqiZmF6g==
 
-axios@1.8.2, axios@^1.0.0, axios@^1.7.7:
+axios@1.8.2, axios@^1.0.0:
   version "1.8.2"
   resolved "https://registry.yarnpkg.com/axios/-/axios-1.8.2.tgz#fabe06e241dfe83071d4edfbcaa7b1c3a40f7979"
   integrity sha512-ls4GYBm5aig9vWx8AWDSGLpnpDQRtWAfrjU+EuytuODrFBkqesN2RkOQCBzrA1RQNHw1SmRMSDDDSwzNAYQ6Rg==