From 34fcb2591f94d8d94f25e143e0e18559d9a9df02 Mon Sep 17 00:00:00 2001 From: Anso Date: Sun, 3 May 2026 21:40:20 -0400 Subject: [PATCH] fix(frontend): remove unused MockWS constructor param (#904) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The `_url` parameter on the test-only MockWS class was unused, and ESLint's no-unused-vars rule does not honor the underscore-prefix convention in this project's config, so the lint job failed. The mock is installed via vi.stubGlobal at runtime, so dropping the parameter is safe — JS still permits callers to pass a url argument. --- .../src/components/EditorLayout/hooks/useContainerStats.test.ts | 2 +- .../src/components/EditorLayout/hooks/useNotifications.test.ts | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/frontend/src/components/EditorLayout/hooks/useContainerStats.test.ts b/frontend/src/components/EditorLayout/hooks/useContainerStats.test.ts index ff5762e5..ac195086 100644 --- a/frontend/src/components/EditorLayout/hooks/useContainerStats.test.ts +++ b/frontend/src/components/EditorLayout/hooks/useContainerStats.test.ts @@ -12,7 +12,7 @@ class MockWS { readyState = 1; send = vi.fn(); close = vi.fn(); - constructor(_url: string) { MockWS.instances.push(this); } + constructor() { MockWS.instances.push(this); } static reset() { MockWS.instances = []; } } diff --git a/frontend/src/components/EditorLayout/hooks/useNotifications.test.ts b/frontend/src/components/EditorLayout/hooks/useNotifications.test.ts index b1b8bf5b..786d783a 100644 --- a/frontend/src/components/EditorLayout/hooks/useNotifications.test.ts +++ b/frontend/src/components/EditorLayout/hooks/useNotifications.test.ts @@ -27,7 +27,7 @@ class MockWS { readyState = 1; // OPEN send = vi.fn(); close = vi.fn(); - constructor(_url: string) { MockWS.instances.push(this); } + constructor() { MockWS.instances.push(this); } static reset() { MockWS.instances = []; } }