fixup! ref(cloudflare)!: Remove wrapRequestHandler from main entrypoint

Author: JPeer264

packages/sveltekit/test/worker/cloudflare.test.ts

@@ -1,9 +1,15 @@
-import { beforeEach } from 'node:test';
 import * as SentryCloudflare from '@sentry/cloudflare';
+import { wrapRequestHandler } from '@sentry/cloudflare/request';
+import type * as SentryCloudflareRequest from '@sentry/cloudflare/request';
 import type { Carrier, GLOBAL_OBJ } from '@sentry/core';
-import { describe, expect, it, vi } from 'vitest';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { initCloudflareSentryHandle } from '../../src/worker';
 
+vi.mock('@sentry/cloudflare/request', async importOriginal => {
+  const actual = await importOriginal<typeof SentryCloudflareRequest>();
+  return { ...actual, wrapRequestHandler: vi.fn(actual.wrapRequestHandler) };
+});
+
 const globalWithSentry = globalThis as typeof GLOBAL_OBJ & Carrier;
 
 function getHandlerInput() {
@@ -19,6 +25,7 @@ function getHandlerInput() {
 describe('initCloudflareSentryHandle', () => {
   beforeEach(() => {
     delete globalWithSentry.__SENTRY__;
+    vi.mocked(wrapRequestHandler).mockClear();
   });
 
   it('sets the async context strategy when called', () => {
@@ -36,15 +43,15 @@ describe('initCloudflareSentryHandle', () => {
     const { options, event, resolve, request, context } = getHandlerInput();
 
     // @ts-expect-error - resolving an empty object is enough for this test
-    vi.spyOn(SentryCloudflare, 'wrapRequestHandler').mockImplementationOnce((_, cb) => cb());
+    vi.mocked(wrapRequestHandler).mockImplementationOnce((_, cb) => cb());
 
     const handle = initCloudflareSentryHandle(options);
 
     // @ts-expect-error - only passing a partial event object
     await handle({ event, resolve });
 
-    expect(SentryCloudflare.wrapRequestHandler).toHaveBeenCalledTimes(1);
-    expect(SentryCloudflare.wrapRequestHandler).toHaveBeenCalledWith(
+    expect(wrapRequestHandler).toHaveBeenCalledTimes(1);
+    expect(wrapRequestHandler).toHaveBeenCalledWith(
       { options: expect.objectContaining({ dsn: options.dsn }), request, context, captureErrors: false },
       expect.any(Function),
     );
@@ -57,7 +64,7 @@ describe('initCloudflareSentryHandle', () => {
     const locals = {};
 
     // @ts-expect-error - resolving an empty object is enough for this test
-    vi.spyOn(SentryCloudflare, 'wrapRequestHandler').mockImplementationOnce((_, cb) => cb());
+    vi.mocked(wrapRequestHandler).mockImplementationOnce((_, cb) => cb());
 
     const handle = initCloudflareSentryHandle(options);
 
@@ -74,14 +81,14 @@ describe('initCloudflareSentryHandle', () => {
     delete event.platform;
 
     // @ts-expect-error - resolving an empty object is enough for this test
-    vi.spyOn(SentryCloudflare, 'wrapRequestHandler').mockImplementationOnce((_, cb) => cb());
+    vi.mocked(wrapRequestHandler).mockImplementationOnce((_, cb) => cb());
 
     const handle = initCloudflareSentryHandle(options);
 
     // @ts-expect-error - only passing a partial event object
     await handle({ event, resolve });
 
-    expect(SentryCloudflare.wrapRequestHandler).not.toHaveBeenCalled();
+    expect(wrapRequestHandler).not.toHaveBeenCalled();
     expect(resolve).toHaveBeenCalledTimes(1);
   });
 });