From ac11efc5c7a973ca1e97b26784c57b40cdb0cfa1 Mon Sep 17 00:00:00 2001
From: shamoon <4887959+shamoon@users.noreply.github.com>
Date: Wed, 4 Feb 2026 22:06:33 -0800
Subject: [PATCH] Fix eslint warnings in test

---
 src/__tests__/pages/index.test.jsx            |  2 +-
 src/components/quicklaunch.test.jsx           |  3 +--
 src/components/widgets/search/search.test.jsx | 12 +++++++++---
 3 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/src/__tests__/pages/index.test.jsx b/src/__tests__/pages/index.test.jsx
index 4355e9387..2a53425fc 100644
--- a/src/__tests__/pages/index.test.jsx
+++ b/src/__tests__/pages/index.test.jsx
@@ -339,7 +339,7 @@ describe("pages/index Index routing + SWR branches", () => {
     localStorage.setItem("hash", "old-hash");
 
     const fetchSpy = vi.fn(async () => ({ ok: true }));
-    // eslint-disable-next-line no-global-assign
+
     fetch = fetchSpy;
 
     let reloadSpy;
diff --git a/src/components/quicklaunch.test.jsx b/src/components/quicklaunch.test.jsx
index 77262db1e..65cf7a508 100644
--- a/src/components/quicklaunch.test.jsx
+++ b/src/components/quicklaunch.test.jsx
@@ -277,7 +277,7 @@ describe("components/quicklaunch", () => {
     const fetchSpy = vi.fn(async () => ({
       json: async () => ["test", ["test 1", "test 2", "test 3", "test 4", "test 5"]],
     }));
-    // eslint-disable-next-line no-global-assign
+
     fetch = fetchSpy;
 
     renderWithProviders(<Wrapper />, {
@@ -315,7 +315,6 @@ describe("components/quicklaunch", () => {
 
     expect(input).toHaveValue("test 1");
 
-    // eslint-disable-next-line no-global-assign
     fetch = originalFetch;
   });
 
diff --git a/src/components/widgets/search/search.test.jsx b/src/components/widgets/search/search.test.jsx
index 80f8ec421..cbb603884 100644
--- a/src/components/widgets/search/search.test.jsx
+++ b/src/components/widgets/search/search.test.jsx
@@ -32,7 +32,13 @@ vi.mock("@headlessui/react", async () => {
       const ctx = useContext(ListboxContext);
       const content = typeof children === "function" ? children({ active: false }) : children;
       return (
-        <div role="option" data-provider={value?.name} onClick={() => ctx?.onChange?.(value)} {...props}>
+        <div
+          role="option"
+          data-provider={value?.name}
+          aria-selected={ctx?.value === value}
+          onClick={() => ctx?.onChange?.(value)}
+          {...props}
+        >
           {content}
         </div>
       );
@@ -166,7 +172,7 @@ describe("components/widgets/search", () => {
     const fetchSpy = vi.fn(async () => ({
       json: async () => ["hel", ["hello", "help", "helm", "helium", "held"]],
     }));
-    // eslint-disable-next-line no-global-assign
+
     fetch = fetchSpy;
 
     renderWithProviders(<Search options={{ provider: ["google"], showSearchSuggestions: true, target: "_self" }} />, {
@@ -192,7 +198,7 @@ describe("components/widgets/search", () => {
     expect(openSpy).toHaveBeenCalledWith("https://www.google.com/search?q=hello", "_self");
 
     openSpy.mockRestore();
-    // eslint-disable-next-line no-global-assign
+
     fetch = originalFetch;
   });
 });