fix: Use type-safe window.location mocking in tests

Replace any type with proper TypeScript types to satisfy linter while maintaining test functionality

Author: felixweinberger

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

@@ -8,16 +8,20 @@ import {
 import { DEFAULT_INSPECTOR_CONFIG } from "../../lib/constants";
 import { InspectorConfig } from "../../lib/configurationTypes";
 
+// Type for window with mutable location
+type WindowWithMutableLocation = Window & {
+  location: Location;
+};
+
 // Helper to mock window.location
 function mockWindowLocation(href: string) {
-  // Delete the property first if it exists and is configurable
-  Reflect.deleteProperty(window, "location");
-
-  Object.defineProperty(window, "location", {
-    value: { ...window.location, href },
-    writable: true,
-    configurable: true,
-  });
+  const windowWithLocation = window as unknown as WindowWithMutableLocation;
+  delete (windowWithLocation as WindowWithMutableLocation & { location?: Location }).location;
+  
+  windowWithLocation.location = {
+    ...window.location,
+    href,
+  } as Location;
 }
 
 describe("configUtils", () => {
@@ -99,7 +103,8 @@ describe("configUtils", () => {
     });
 
     afterEach(() => {
-      mockWindowLocation(originalLocation.href);
+      const windowWithLocation = window as unknown as WindowWithMutableLocation;
+      windowWithLocation.location = originalLocation;
     });
 
     test("returns transport type from URL query parameter", () => {
@@ -143,7 +148,8 @@ describe("configUtils", () => {
     });
 
     afterEach(() => {
-      mockWindowLocation(originalLocation.href);
+      const windowWithLocation = window as unknown as WindowWithMutableLocation;
+      windowWithLocation.location = originalLocation;
     });
 
     test("returns serverUrl from query parameter", () => {
@@ -186,7 +192,8 @@ describe("configUtils", () => {
     });
 
     afterEach(() => {
-      mockWindowLocation(originalLocation.href);
+      const windowWithLocation = window as unknown as WindowWithMutableLocation;
+      windowWithLocation.location = originalLocation;
     });
 
     test("returns serverCommand from query parameter", () => {
@@ -219,7 +226,8 @@ describe("configUtils", () => {
     });
 
     afterEach(() => {
-      mockWindowLocation(originalLocation.href);
+      const windowWithLocation = window as unknown as WindowWithMutableLocation;
+      windowWithLocation.location = originalLocation;
     });
 
     test("returns serverArgs from query parameter", () => {