fix unit tests

Author: mydea

packages/node-core/test/integrations/http.test.ts

@@ -0,0 +1,34 @@
+import { describe, expect, it } from 'vitest';
+import { isStaticAssetRequest } from '../../src/integrations/http/incoming-requests';
+
+describe('httpIntegration', () => {
+  describe('isStaticAssetRequest', () => {
+    it.each([
+      ['/favicon.ico', true],
+      ['/apple-touch-icon.png', true],
+      ['/static/style.css', true],
+      ['/assets/app.js', true],
+      ['/fonts/font.woff2', true],
+      ['/images/logo.svg', true],
+      ['/img/photo.jpeg', true],
+      ['/img/photo.jpg', true],
+      ['/img/photo.jpg?v=123', true],
+      ['/img/photo.webp', true],
+      ['/font/font.ttf', true],
+      ['/robots.txt', true],
+      ['/sitemap.xml', true],
+      ['/manifest.json', true],
+      ['/browserconfig.xml', true],
+      // non-static routes
+      ['/api/users', false],
+      ['/some-json.json', false],
+      ['/some-xml.xml', false],
+      ['/some-txt.txt', false],
+      ['/users', false],
+      ['/graphql', false],
+      ['/', false],
+    ])('returns %s -> %s', (urlPath, expected) => {
+      expect(isStaticAssetRequest(urlPath)).toBe(expected);
+    });
+  });
+});

packages/node/src/integrations/http.ts

@@ -275,22 +275,3 @@ function getConfigWithDefaults(options: Partial<HttpOptions> = {}): HttpInstrume
   return instrumentationConfig;
 }
 
-/**
- * Check if a request is for a common static asset that should be ignored by default.
- *
- * Only exported for tests.
- */
-export function isStaticAssetRequest(urlPath: string): boolean {
-  const path = stripUrlQueryAndFragment(urlPath);
-  // Common static file extensions
-  if (path.match(/\.(ico|png|jpg|jpeg|gif|svg|css|js|woff|woff2|ttf|eot|webp|avif)$/)) {
-    return true;
-  }
-
-  // Common metadata files
-  if (path.match(/^\/(robots\.txt|sitemap\.xml|manifest\.json|browserconfig\.xml)$/)) {
-    return true;
-  }
-
-  return false;
-}

packages/node/test/integrations/http.test.ts

@@ -1,5 +1,5 @@
 import { describe, expect, it } from 'vitest';
-import { _shouldInstrumentSpans, isStaticAssetRequest } from '../../src/integrations/http';
+import { _shouldInstrumentSpans } from '../../src/integrations/http';
 import { conditionalTest } from '../helpers/conditional';
 
 describe('httpIntegration', () => {
@@ -27,34 +27,4 @@ describe('httpIntegration', () => {
       expect(actual).toBe(true);
     });
   });
-
-  describe('isStaticAssetRequest', () => {
-    it.each([
-      ['/favicon.ico', true],
-      ['/apple-touch-icon.png', true],
-      ['/static/style.css', true],
-      ['/assets/app.js', true],
-      ['/fonts/font.woff2', true],
-      ['/images/logo.svg', true],
-      ['/img/photo.jpeg', true],
-      ['/img/photo.jpg', true],
-      ['/img/photo.jpg?v=123', true],
-      ['/img/photo.webp', true],
-      ['/font/font.ttf', true],
-      ['/robots.txt', true],
-      ['/sitemap.xml', true],
-      ['/manifest.json', true],
-      ['/browserconfig.xml', true],
-      // non-static routes
-      ['/api/users', false],
-      ['/some-json.json', false],
-      ['/some-xml.xml', false],
-      ['/some-txt.txt', false],
-      ['/users', false],
-      ['/graphql', false],
-      ['/', false],
-    ])('returns %s -> %s', (urlPath, expected) => {
-      expect(isStaticAssetRequest(urlPath)).toBe(expected);
-    });
-  });
 });