Delete tests that have become redundant

Author: Luca Forstner

packages/nextjs/test/edge/edgeWrapperUtils.test.ts

@@ -1,5 +1,4 @@
 import * as coreSdk from '@sentry/core';
-import { SEMANTIC_ATTRIBUTE_SENTRY_SOURCE } from '@sentry/core';
 
 import { withEdgeWrapping } from '../../src/common/utils/edgeWrapperUtils';
 
@@ -61,38 +60,6 @@ describe('withEdgeWrapping', () => {
     expect(captureExceptionSpy).toHaveBeenCalledTimes(1);
   });
 
-  it('should return a function that calls trace', async () => {
-    const startSpanSpy = jest.spyOn(coreSdk, 'startSpan');
-
-    const request = new Request('https://sentry.io/');
-    const origFunction = jest.fn(_req => new Response());
-
-    const wrappedFunction = withEdgeWrapping(origFunction, {
-      spanDescription: 'some label',
-      mechanismFunctionName: 'some name',
-      spanOp: 'some op',
-    });
-
-    await wrappedFunction(request);
-
-    expect(startSpanSpy).toHaveBeenCalledTimes(1);
-    expect(startSpanSpy).toHaveBeenCalledWith(
-      expect.objectContaining({
-        attributes: {
-          [SEMANTIC_ATTRIBUTE_SENTRY_SOURCE]: 'route',
-          [coreSdk.SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.function.nextjs.withEdgeWrapping',
-        },
-        name: 'some label',
-        op: 'some op',
-      }),
-      expect.any(Function),
-    );
-
-    expect(coreSdk.getIsolationScope().getScopeData().sdkProcessingMetadata).toEqual({
-      request: { headers: {} },
-    });
-  });
-
   it("should return a function that doesn't crash when req isn't passed", async () => {
     const origFunctionReturnValue = new Response();
     const origFunction = jest.fn(() => origFunctionReturnValue);

packages/nextjs/test/edge/withSentryAPI.test.ts

@@ -1,6 +1,3 @@
-import * as coreSdk from '@sentry/core';
-import { SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN, SEMANTIC_ATTRIBUTE_SENTRY_SOURCE } from '@sentry/core';
-
 import { wrapApiHandlerWithSentry } from '../../src/edge';
 
 const origRequest = global.Request;
@@ -31,53 +28,16 @@ afterAll(() => {
   global.Response = origResponse;
 });
 
-const startSpanSpy = jest.spyOn(coreSdk, 'startSpan');
-
 afterEach(() => {
   jest.clearAllMocks();
 });
 
 describe('wrapApiHandlerWithSentry', () => {
-  it('should return a function that calls trace', async () => {
-    const request = new Request('https://sentry.io/');
-    const origFunction = jest.fn(_req => new Response());
-
-    const wrappedFunction = wrapApiHandlerWithSentry(origFunction, '/user/[userId]/post/[postId]');
-
-    await wrappedFunction(request);
-
-    expect(startSpanSpy).toHaveBeenCalledTimes(1);
-    expect(startSpanSpy).toHaveBeenCalledWith(
-      expect.objectContaining({
-        attributes: {
-          [SEMANTIC_ATTRIBUTE_SENTRY_SOURCE]: 'route',
-          [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.function.nextjs.withEdgeWrapping',
-        },
-        name: 'POST /user/[userId]/post/[postId]',
-        op: 'http.server',
-      }),
-      expect.any(Function),
-    );
-  });
-
-  it('should return a function that calls trace without throwing when no request is passed', async () => {
+  it('should return a function that does not throw when no request is passed', async () => {
     const origFunction = jest.fn(() => new Response());
 
     const wrappedFunction = wrapApiHandlerWithSentry(origFunction, '/user/[userId]/post/[postId]');
 
     await wrappedFunction();
-
-    expect(startSpanSpy).toHaveBeenCalledTimes(1);
-    expect(startSpanSpy).toHaveBeenCalledWith(
-      expect.objectContaining({
-        attributes: {
-          [SEMANTIC_ATTRIBUTE_SENTRY_SOURCE]: 'route',
-          [coreSdk.SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.function.nextjs.withEdgeWrapping',
-        },
-        name: 'handler (/user/[userId]/post/[postId])',
-        op: 'http.server',
-      }),
-      expect.any(Function),
-    );
   });
 });