fix more otel/node-experimental tests

fix missing eslint ignore after rebase :(

migration doc

revert next-env.d.ts change

once again fix things after rebasing :(

Author: Lms24

MIGRATION.md

@@ -205,6 +205,7 @@ In v8, the Span class is heavily reworked. The following properties & methods ar
 - `span.transaction`: Use `getRootSpan` utility function instead.
 - `span.spanRecorder`: Span recording will be handled internally by the SDK.
 - `span.status`: Use `.setStatus` to set or update and `spanToJSON()` to read the span status.
+- `span.op`: Use `startSpan` functions to set, `setAttribute()` to update and `spanToJSON` to read the span operation.
 - `transaction.setMetadata()`: Use attributes instead, or set data on the scope.
 - `transaction.metadata`: Use attributes instead, or set data on the scope.
 - `transaction.setContext()`: Set context on the surrounding scope instead.

packages/nextjs/test/integration/next-env.d.ts

@@ -1,6 +1,5 @@
 /// <reference types="next" />
 /// <reference types="next/image-types/global" />
-/// <reference types="next/navigation-types/compat/navigation" />
 
 // NOTE: This file should not be edited
 // see https://nextjs.org/docs/basic-features/typescript for more information.

packages/node-experimental/test/integration/transactions.test.ts

@@ -84,7 +84,7 @@ describe('Integration | Transactions', () => {
           },
           runtime: { name: 'node', version: expect.any(String) },
           trace: {
-            data: { 'otel.kind': 'INTERNAL' },
+            data: { 'otel.kind': 'INTERNAL', 'sentry.op': 'test op' },
             op: 'test op',
             span_id: expect.any(String),
             status: 'ok',
@@ -244,7 +244,7 @@ describe('Integration | Transactions', () => {
             },
           }),
           trace: {
-            data: { 'otel.kind': 'INTERNAL' },
+            data: { 'otel.kind': 'INTERNAL', 'sentry.op': 'test op' },
             op: 'test op',
             span_id: expect.any(String),
             status: 'ok',
@@ -286,7 +286,7 @@ describe('Integration | Transactions', () => {
             },
           }),
           trace: {
-            data: { 'otel.kind': 'INTERNAL' },
+            data: { 'otel.kind': 'INTERNAL', 'sentry.op': 'test op b' },
             op: 'test op b',
             span_id: expect.any(String),
             status: 'ok',
@@ -363,7 +363,7 @@ describe('Integration | Transactions', () => {
             attributes: {},
           }),
           trace: {
-            data: { 'otel.kind': 'INTERNAL' },
+            data: { 'otel.kind': 'INTERNAL', 'sentry.op': 'test op' },
             op: 'test op',
             span_id: expect.any(String),
             parent_span_id: parentSpanId,

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

@@ -79,6 +79,7 @@ describe('tracing', () => {
 
     // our span is at index 1 because the transaction itself is at index 0
     expect(spanToJSON(spans[1]).description).toEqual('GET http://dogs.are.great/');
+    // eslint-disable-next-line deprecation/deprecation
     expect(spans[1].op).toEqual('http.client');
     expect(spanToJSON(spans[1]).op).toEqual('http.client');
   });

packages/opentelemetry-node/test/spanprocessor.test.ts

@@ -560,6 +560,7 @@ describe('SentrySpanProcessor', () => {
             'http.url': 'http://example.com/my/route/123',
             'otel.kind': 'INTERNAL',
             url: 'http://example.com/my/route/123',
+            'sentry.op': 'http',
           });
 
           parentOtelSpan.end();

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

@@ -193,6 +193,7 @@ export function addPerformanceEntries(transaction: Transaction): void {
     const startTime = msToSec(entry.startTime);
     const duration = msToSec(entry.duration);
 
+    // eslint-disable-next-line deprecation/deprecation
     if (transaction.op === 'navigation' && transactionStartTime && timeOrigin + startTime < transactionStartTime) {
       return;
     }