test(sveltekit): Switch to explicit vitest imports (#12960)

As per https://vitest.dev/config/#globals

> By default, vitest does not provide global APIs for explicitness

I think we should follow vitest defaults here and explicitly import in
the APIs that we need. This refactors our SvelteKit SDK tests to do so.

This change also removes `environment: 'jsdom'` from the sveltekit
global vite config in favour of explicitly adding jsdom environment via
the `@vitest-environment` pragma to the specific test file that needs
it. This should means that our server tests are not polluted with jsdom
globals, and that future writers have to explicitly opt-in to the
behaviour.

Author: AbhiPrasad

packages/sveltekit/test/client/browserTracingIntegration.test.ts

@@ -1,7 +1,12 @@
+/**
+ * @vitest-environment jsdom
+ */
+
 /* eslint-disable @typescript-eslint/unbound-method */
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+
 import type { Span } from '@sentry/types';
 import { writable } from 'svelte/store';
-import { vi } from 'vitest';
 
 import { navigating, page } from '$app/stores';
 

packages/sveltekit/test/client/fetch.test.ts

@@ -1,3 +1,9 @@
+/**
+ * @vitest-environment jsdom
+ */
+
+import { beforeEach, describe, expect, it } from 'vitest';
+
 import { init } from '../../src/client/index';
 
 describe('instruments fetch', () => {

packages/sveltekit/test/client/handleError.test.ts

@@ -1,6 +1,7 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+
 import * as SentrySvelte from '@sentry/svelte';
 import type { HandleClientError, NavigationEvent } from '@sveltejs/kit';
-import { vi } from 'vitest';
 
 import { handleErrorWithSentry } from '../../src/client/handleError';
 

packages/sveltekit/test/client/load.test.ts

@@ -1,7 +1,8 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+
 import * as SentrySvelte from '@sentry/svelte';
 import type { Load } from '@sveltejs/kit';
 import { redirect } from '@sveltejs/kit';
-import { vi } from 'vitest';
 
 import { SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN, SEMANTIC_ATTRIBUTE_SENTRY_SOURCE } from '@sentry/core';
 import { wrapLoadWithSentry } from '../../src/client/load';

packages/sveltekit/test/client/sdk.test.ts

@@ -1,7 +1,8 @@
+import { afterEach, describe, expect, it, vi } from 'vitest';
+
 import type { BrowserClient } from '@sentry/svelte';
 import * as SentrySvelte from '@sentry/svelte';
 import { SDK_VERSION, getClient, getCurrentScope, getGlobalScope, getIsolationScope } from '@sentry/svelte';
-import { vi } from 'vitest';
 
 import { init } from '../../src/client';
 

packages/sveltekit/test/common/utils.test.ts

@@ -1,3 +1,5 @@
+import { describe, expect, it } from 'vitest';
+
 import { isHttpError, isRedirect } from '../../src/common/utils';
 
 describe('isRedirect', () => {

packages/sveltekit/test/index.test.ts

@@ -1,3 +1,5 @@
+import { describe, expect, it } from 'vitest';
+
 import * as SentryClient from '../src/client';
 import * as SentryServer from '../src/server';
 

packages/sveltekit/test/server/handle.test.ts

@@ -1,3 +1,5 @@
+import { beforeEach, describe, expect, it } from 'vitest';
+
 import {
   SEMANTIC_ATTRIBUTE_SENTRY_SOURCE,
   getRootSpan,

packages/sveltekit/test/server/handleError.test.ts

@@ -1,6 +1,7 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+
 import * as SentryNode from '@sentry/node';
 import type { HandleServerError, RequestEvent } from '@sveltejs/kit';
-import { vi } from 'vitest';
 
 import { handleErrorWithSentry } from '../../src/server/handleError';
 

packages/sveltekit/test/server/load.test.ts

@@ -1,3 +1,5 @@
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+
 import {
   SEMANTIC_ATTRIBUTE_SENTRY_OP,
   SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN,
@@ -8,7 +10,6 @@ import * as SentryNode from '@sentry/node';
 import type { Event } from '@sentry/types';
 import type { Load, ServerLoad } from '@sveltejs/kit';
 import { error, redirect } from '@sveltejs/kit';
-import { vi } from 'vitest';
 
 import { wrapLoadWithSentry, wrapServerLoadWithSentry } from '../../src/server/load';
 import { getDefaultNodeClientOptions } from '../utils';