From 2a8183daf2c826c66e86f3d45aaca1322a8fa60b Mon Sep 17 00:00:00 2001
From: Shaun Arman <Shaun.Arman@motorolasolutions.com>
Date: Tue, 9 Jun 2026 13:36:36 -0500
Subject: [PATCH] fix(lint): remove unused variables in test files

Remove unused import and variable in criticalUIFixes test
Update PodList test mocks to use new Interactive* modal components

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
---
 tests/unit/PodList.test.tsx         | 8 ++++----
 tests/unit/criticalUIFixes.test.tsx | 2 --
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/tests/unit/PodList.test.tsx b/tests/unit/PodList.test.tsx
index 8088b4a8..14d4fec5 100644
--- a/tests/unit/PodList.test.tsx
+++ b/tests/unit/PodList.test.tsx
@@ -13,13 +13,13 @@ vi.mock("@/components/Kubernetes/LogStreamPanel", () => ({
     <div data-testid="logs-modal" data-namespace={namespace} />
   ),
 }));
-vi.mock("@/components/Kubernetes/ShellExecModal", () => ({
-  ShellExecModal: ({ namespace }: { namespace: string }) => (
+vi.mock("@/components/Kubernetes/InteractiveShellModal", () => ({
+  InteractiveShellModal: ({ namespace }: { namespace: string }) => (
     <div data-testid="shell-modal" data-namespace={namespace} />
   ),
 }));
-vi.mock("@/components/Kubernetes/AttachModal", () => ({
-  AttachModal: ({ namespace }: { namespace: string }) => (
+vi.mock("@/components/Kubernetes/InteractiveAttachModal", () => ({
+  InteractiveAttachModal: ({ namespace }: { namespace: string }) => (
     <div data-testid="attach-modal" data-namespace={namespace} />
   ),
 }));
diff --git a/tests/unit/criticalUIFixes.test.tsx b/tests/unit/criticalUIFixes.test.tsx
index 7249dbad..55fe5851 100644
--- a/tests/unit/criticalUIFixes.test.tsx
+++ b/tests/unit/criticalUIFixes.test.tsx
@@ -6,7 +6,6 @@
  * 4. YAML editor loading race condition
  */
 
-import React from "react";
 import { describe, it, expect, vi, beforeEach } from "vitest";
 import { render, screen, fireEvent, waitFor } from "@testing-library/react";
 import { invoke } from "@tauri-apps/api/core";
@@ -219,7 +218,6 @@ describe("Dark mode – text visibility", () => {
     const root = document.documentElement;
     root.classList.add("dark");
 
-    const computedStyle = window.getComputedStyle(root);
     expect(root.classList.contains("dark")).toBe(true);
 
     root.classList.remove("dark");