linter and cleanup

getsentry · Jun 23, 2023 · 568bc7c · 568bc7c
1 parent 236cb9e
commit 568bc7c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/packages/sveltekit/test/client/load.test.ts b/packages/sveltekit/test/client/load.test.ts
@@ -1,6 +1,5 @@
 import { addTracingExtensions, Scope } from '@sentry/svelte';
 import { baggageHeaderToDynamicSamplingContext } from '@sentry/utils';
-import * as utils from '@sentry/utils';
 import type { Load } from '@sveltejs/kit';
 import { redirect } from '@sveltejs/kit';
 import { vi } from 'vitest';
@@ -28,9 +27,9 @@ vi.mock('@sentry/svelte', async () => {
   };
 });
 
-vi.spyOn(utils, 'getDomElement').mockImplementation(() => {
+vi.mock('../../src/client/vendor/lookUpCache', () => {
   return {
-    textContent: 'test',
+    isRequestCached: () => false,
   };
 });
 
@@ -440,7 +439,6 @@ describe('wrapLoadWithSentry', () => {
     ['is undefined', undefined],
     ["doesn't have a `getClientById` method", {}],
   ])("doesn't instrument fetch if the client %s", async (_, client) => {
-    // @ts-expect-error: we're mocking the client
     mockedGetClient.mockImplementationOnce(() => client);
 
     async function load(_event: Parameters<Load>[0]): Promise<ReturnType<Load>> {

diff --git a/packages/sveltekit/test/vitest.setup.ts b/packages/sveltekit/test/vitest.setup.ts
@@ -13,7 +13,6 @@ export function setup() {
 }
 
 if (!globalThis.fetch) {
-  // @ts-ignore dfsf
+  // @ts-ignore - Needed for vitest to work with SvelteKit fetch instrumentation
   globalThis.Request = class Request {};
 }
-console.log(globalThis.fetch, globalThis.Response, globalThis.Request);