fix: Implement proper window.location mocking for Jest with TypeScript

Uses Partial<Window> type assertion to allow delete operation while maintaining type safety. This avoids both TypeScript's 'delete must be optional' error and JSDOM's 'Cannot redefine property' error.

Author: felixweinberger

client/src/utils/__tests__/configUtils.test.ts

@@ -10,12 +10,27 @@ import { InspectorConfig } from "../../lib/configurationTypes";
 
 // Helper to mock window.location
 function mockWindowLocation(href: string) {
-  // Use Object.defineProperty with a getter to avoid type issues
-  Object.defineProperty(window, "location", {
-    value: new URL(href),
-    writable: true,
-    configurable: true,
-  });
+  // Delete using Partial<Window> to satisfy TypeScript
+  delete (window as Partial<Window>).location;
+  
+  // Create a complete location mock
+  const url = new URL(href);
+  window.location = {
+    href,
+    hostname: url.hostname,
+    pathname: url.pathname,
+    protocol: url.protocol,
+    search: url.search,
+    hash: url.hash,
+    host: url.host,
+    origin: url.origin,
+    port: url.port,
+    reload: jest.fn(),
+    replace: jest.fn(),
+    assign: jest.fn(),
+    toString: jest.fn(() => href),
+    ancestorOrigins: {} as DOMStringList,
+  } as Location;
 }
 
 describe("configUtils", () => {
@@ -97,11 +112,8 @@ describe("configUtils", () => {
     });
 
     afterEach(() => {
-      Object.defineProperty(window, "location", {
-        value: originalLocation,
-        writable: true,
-        configurable: true,
-      });
+      delete (window as Partial<Window>).location;
+      window.location = originalLocation;
     });
 
     test("returns transport type from URL query parameter", () => {
@@ -145,11 +157,8 @@ describe("configUtils", () => {
     });
 
     afterEach(() => {
-      Object.defineProperty(window, "location", {
-        value: originalLocation,
-        writable: true,
-        configurable: true,
-      });
+      delete (window as Partial<Window>).location;
+      window.location = originalLocation;
     });
 
     test("returns serverUrl from query parameter", () => {
@@ -192,11 +201,8 @@ describe("configUtils", () => {
     });
 
     afterEach(() => {
-      Object.defineProperty(window, "location", {
-        value: originalLocation,
-        writable: true,
-        configurable: true,
-      });
+      delete (window as Partial<Window>).location;
+      window.location = originalLocation;
     });
 
     test("returns serverCommand from query parameter", () => {
@@ -229,11 +235,8 @@ describe("configUtils", () => {
     });
 
     afterEach(() => {
-      Object.defineProperty(window, "location", {
-        value: originalLocation,
-        writable: true,
-        configurable: true,
-      });
+      delete (window as Partial<Window>).location;
+      window.location = originalLocation;
     });
 
     test("returns serverArgs from query parameter", () => {