feat(solidjs): Avoid leaking vi mocks across tests

Author: andreiborza

packages/astro/test/client/sdk.test.ts

@@ -8,14 +8,14 @@ import {
 } from '@sentry/browser';
 import * as SentryBrowser from '@sentry/browser';
 import { SDK_VERSION, getClient } from '@sentry/browser';
-import { vi } from 'vitest';
 
+import { vi } from 'vitest';
 import { init } from '../../../astro/src/client/sdk';
 
-const browserInit = vi.spyOn(SentryBrowser, 'init');
-
 describe('Sentry client SDK', () => {
   describe('init', () => {
+    const browserInit = vi.spyOn(SentryBrowser, 'init');
+
     afterEach(() => {
       vi.clearAllMocks();
 

packages/solidjs/test/sdk.test.ts

@@ -4,9 +4,9 @@ import * as SentryBrowser from '@sentry/browser';
 import { vi } from 'vitest';
 import { init as solidInit } from '../src/sdk';
 
-const browserInit = vi.spyOn(SentryBrowser, 'init');
-
 describe('Initialize SolidJS SDk', () => {
+  const browserInit = vi.spyOn(SentryBrowser, 'init');
+
   beforeEach(() => {
     vi.clearAllMocks();
   });

packages/svelte/test/sdk.test.ts

@@ -7,15 +7,9 @@ import { detectAndReportSvelteKit, init as svelteInit, isSvelteKitApp } from '..
 
 let passedEventProcessor: EventProcessor | undefined;
 
-const browserInit = vi.spyOn(SentryBrowser, 'init');
-const addEventProcessor = vi
-  .spyOn(SentryBrowser, 'addEventProcessor')
-  .mockImplementation((eventProcessor: EventProcessor) => {
-    passedEventProcessor = eventProcessor;
-    return () => {};
-  });
-
 describe('Initialize Svelte SDk', () => {
+  const browserInit = vi.spyOn(SentryBrowser, 'init');
+
   beforeEach(() => {
     vi.clearAllMocks();
   });
@@ -73,6 +67,12 @@ describe('Initialize Svelte SDk', () => {
 });
 
 describe('detectAndReportSvelteKit()', () => {
+  const addEventProcessor = vi
+    .spyOn(SentryBrowser, 'addEventProcessor')
+    .mockImplementation((eventProcessor: EventProcessor) => {
+      passedEventProcessor = eventProcessor;
+      return () => {};
+    });
   const originalHtmlBody = document.body.innerHTML;
   beforeEach(() => {
     vi.clearAllMocks();