fix tests

Author: DominikB2014

packages/tracing-internal/test/browser/metrics/index.test.ts

@@ -1,24 +1,26 @@
-import { GLOBAL_OBJ } from '@sentry/utils';
+import { GLOBAL_OBJ, parseUrl } from '@sentry/utils';
 import { Transaction } from '../../../src';
 import type { ResourceEntry } from '../../../src/browser/metrics';
 import { _addMeasureSpans, _addResourceSpans } from '../../../src/browser/metrics';
 import { WINDOW } from '../../../src/browser/types';
 
 const mockWindowLocation = {
   ancestorOrigins: {},
-  href: "https://github.com/getsentry/sentry-javascript/pull/10205/files",
-  origin: "https://github.com",
+  href: "https://example.com/path/to/something",
+  origin: "https://example.com",
   protocol: "https:",
-  host: "github.com",
-  hostname: "github.com",
+  host: "example.com",
+  hostname: "example.com",
   port: "",
-  pathname: "/getsentry/sentry-javascript/pull/10205/files",
+  pathname: "/path/to/something",
   search: "",
   hash: ""
 } as Window['location'];
 
 WINDOW.location = mockWindowLocation;
 
+const resourceEntryName = 'https://example.com/assets/to/css';
+
 describe('_addMeasureSpans', () => {
   // eslint-disable-next-line deprecation/deprecation
   const transaction = new Transaction({ op: 'pageload', name: '/' });
@@ -73,7 +75,7 @@ describe('_addResourceSpans', () => {
       decodedBodySize: 256,
       renderBlockingStatus: 'non-blocking',
     };
-    _addResourceSpans(transaction, entry, '/assets/to/me', 123, 456, 100);
+    _addResourceSpans(transaction, entry, resourceEntryName, 123, 456, 100);
 
     // eslint-disable-next-line @typescript-eslint/unbound-method, deprecation/deprecation
     expect(transaction.startChild).toHaveBeenCalledTimes(0);
@@ -87,7 +89,7 @@ describe('_addResourceSpans', () => {
       decodedBodySize: 256,
       renderBlockingStatus: 'non-blocking',
     };
-    _addResourceSpans(transaction, entry, '/assets/to/me', 123, 456, 100);
+    _addResourceSpans(transaction, entry, 'https://example.com/assets/to/me', 123, 456, 100);
 
     // eslint-disable-next-line @typescript-eslint/unbound-method, deprecation/deprecation
     expect(transaction.startChild).toHaveBeenCalledTimes(0);
@@ -106,7 +108,7 @@ describe('_addResourceSpans', () => {
     const startTime = 23;
     const duration = 356;
 
-    _addResourceSpans(transaction, entry, '/assets/to/css', startTime, duration, timeOrigin);
+    _addResourceSpans(transaction, entry, resourceEntryName, startTime, duration, timeOrigin);
 
     // eslint-disable-next-line @typescript-eslint/unbound-method, deprecation/deprecation
     expect(transaction.startChild).toHaveBeenCalledTimes(1);
@@ -117,9 +119,9 @@ describe('_addResourceSpans', () => {
         ['http.response_content_length']: entry.encodedBodySize,
         ['http.response_transfer_size']: entry.transferSize,
         ['resource.render_blocking_status']: entry.renderBlockingStatus,
-        ['url.scheme']: WINDOW.location.protocol,
-        ['server.address']: WINDOW.location.host,
-        ['url.same_origin']: false,
+        ['url.scheme']: 'https:',
+        ['server.address']: 'example.com',
+        ['url.same_origin']: true,
       },
       description: '/assets/to/css',
       endTimestamp: timeOrigin + startTime + duration,
@@ -157,7 +159,7 @@ describe('_addResourceSpans', () => {
       const entry: ResourceEntry = {
         initiatorType,
       };
-      _addResourceSpans(transaction, entry, '/assets/to/me', 123, 234, 465);
+      _addResourceSpans(transaction, entry, 'https://example.com/assets/to/me', 123, 234, 465);
 
       // eslint-disable-next-line @typescript-eslint/unbound-method, deprecation/deprecation
       expect(transaction.startChild).toHaveBeenLastCalledWith(
@@ -177,7 +179,7 @@ describe('_addResourceSpans', () => {
       renderBlockingStatus: 'non-blocking',
     };
 
-    _addResourceSpans(transaction, entry, '/assets/to/css', 100, 23, 345);
+    _addResourceSpans(transaction, entry, resourceEntryName, 100, 23, 345);
 
     // eslint-disable-next-line @typescript-eslint/unbound-method, deprecation/deprecation
     expect(transaction.startChild).toHaveBeenCalledTimes(1);
@@ -189,6 +191,7 @@ describe('_addResourceSpans', () => {
           ['http.response_content_length']: entry.encodedBodySize,
           ['http.response_transfer_size']: entry.transferSize,
           ['resource.render_blocking_status']: entry.renderBlockingStatus,
+          ['resource.render_blocking_status']: entry.renderBlockingStatus,
         },
       }),
     );
@@ -202,7 +205,7 @@ describe('_addResourceSpans', () => {
       decodedBodySize: 2147483647,
     };
 
-    _addResourceSpans(transaction, entry, '/assets/to/css', 100, 23, 345);
+    _addResourceSpans(transaction, entry, resourceEntryName, 100, 23, 345);
 
     // eslint-disable-next-line @typescript-eslint/unbound-method, deprecation/deprecation
     expect(transaction.startChild).toHaveBeenCalledTimes(1);
@@ -224,7 +227,7 @@ describe('_addResourceSpans', () => {
       decodedBodySize: null,
     } as unknown as ResourceEntry;
 
-    _addResourceSpans(transaction, entry, '/assets/to/css', 100, 23, 345);
+    _addResourceSpans(transaction, entry, resourceEntryName, 100, 23, 345);
 
     // eslint-disable-next-line @typescript-eslint/unbound-method, deprecation/deprecation
     expect(transaction.startChild).toHaveBeenCalledTimes(1);