From 0cc263bcd144f57fc18e708bd27b8584a452a1b0 Mon Sep 17 00:00:00 2001
From: YeonGyu-Kim <code.yeon.gyu@gmail.com>
Date: Sat, 4 Apr 2026 17:24:15 +0900
Subject: [PATCH] fix(test): isolate auto-update-checker hook.test.ts to
 prevent checker mock leak

hook.test.ts mocks ./checker at module load time, which contaminates
checker.test.ts and invalidate-package.test.ts running in parallel in
the same CI batch. Move to hook-integration/ subdirectory for CI batch
isolation.
---
 .../{ => hook-integration}/hook.test.ts       | 20 +++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)
 rename src/hooks/auto-update-checker/{ => hook-integration}/hook.test.ts (93%)

diff --git a/src/hooks/auto-update-checker/hook.test.ts b/src/hooks/auto-update-checker/hook-integration/hook.test.ts
similarity index 93%
rename from src/hooks/auto-update-checker/hook.test.ts
rename to src/hooks/auto-update-checker/hook-integration/hook.test.ts
index 5f5e54218..6c98f773f 100644
--- a/src/hooks/auto-update-checker/hook.test.ts
+++ b/src/hooks/auto-update-checker/hook-integration/hook.test.ts
@@ -10,38 +10,38 @@ const mockRunBackgroundUpdateCheck = mock(async () => {})
 const mockGetCachedVersion = mock(() => "3.6.0")
 const mockGetLocalDevVersion = mock<(directory: string) => string | null>(() => null)
 
-mock.module("./hook/config-errors-toast", () => ({
+mock.module("../hook/config-errors-toast", () => ({
   showConfigErrorsIfAny: mockShowConfigErrorsIfAny,
 }))
 
-mock.module("./hook/model-cache-warning", () => ({
+mock.module("../hook/model-cache-warning", () => ({
   showModelCacheWarningIfNeeded: mockShowModelCacheWarningIfNeeded,
 }))
 
-mock.module("./hook/connected-providers-status", () => ({
+mock.module("../hook/connected-providers-status", () => ({
   updateAndShowConnectedProvidersCacheStatus:
     mockUpdateAndShowConnectedProvidersCacheStatus,
 }))
 
-mock.module("./hook/model-capabilities-status", () => ({
+mock.module("../hook/model-capabilities-status", () => ({
   refreshModelCapabilitiesOnStartup: mockRefreshModelCapabilitiesOnStartup,
 }))
 
-mock.module("./hook/startup-toasts", () => ({
+mock.module("../hook/startup-toasts", () => ({
   showLocalDevToast: mockShowLocalDevToast,
   showVersionToast: mockShowVersionToast,
 }))
 
-mock.module("./hook/background-update-check", () => ({
+mock.module("../hook/background-update-check", () => ({
   runBackgroundUpdateCheck: mockRunBackgroundUpdateCheck,
 }))
 
-mock.module("./checker", () => ({
+mock.module("../checker", () => ({
   getCachedVersion: mockGetCachedVersion,
   getLocalDevVersion: mockGetLocalDevVersion,
 }))
 
-mock.module("../../shared/logger", () => ({
+mock.module("../../../shared/logger", () => ({
   log: () => {},
 }))
 
@@ -49,10 +49,10 @@ afterAll(() => {
   mock.restore()
 })
 
-type HookFactory = typeof import("./hook").createAutoUpdateCheckerHook
+type HookFactory = typeof import("../hook").createAutoUpdateCheckerHook
 
 async function importFreshHookFactory(): Promise<HookFactory> {
-  const hookModule = await import(`./hook?test-${Date.now()}-${Math.random()}`)
+  const hookModule = await import(`../hook?test-${Date.now()}-${Math.random()}`)
   return hookModule.createAutoUpdateCheckerHook
 }