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.
This commit is contained in:
+10
-10
@@ -10,38 +10,38 @@ const mockRunBackgroundUpdateCheck = mock(async () => {})
|
|||||||
const mockGetCachedVersion = mock(() => "3.6.0")
|
const mockGetCachedVersion = mock(() => "3.6.0")
|
||||||
const mockGetLocalDevVersion = mock<(directory: string) => string | null>(() => null)
|
const mockGetLocalDevVersion = mock<(directory: string) => string | null>(() => null)
|
||||||
|
|
||||||
mock.module("./hook/config-errors-toast", () => ({
|
mock.module("../hook/config-errors-toast", () => ({
|
||||||
showConfigErrorsIfAny: mockShowConfigErrorsIfAny,
|
showConfigErrorsIfAny: mockShowConfigErrorsIfAny,
|
||||||
}))
|
}))
|
||||||
|
|
||||||
mock.module("./hook/model-cache-warning", () => ({
|
mock.module("../hook/model-cache-warning", () => ({
|
||||||
showModelCacheWarningIfNeeded: mockShowModelCacheWarningIfNeeded,
|
showModelCacheWarningIfNeeded: mockShowModelCacheWarningIfNeeded,
|
||||||
}))
|
}))
|
||||||
|
|
||||||
mock.module("./hook/connected-providers-status", () => ({
|
mock.module("../hook/connected-providers-status", () => ({
|
||||||
updateAndShowConnectedProvidersCacheStatus:
|
updateAndShowConnectedProvidersCacheStatus:
|
||||||
mockUpdateAndShowConnectedProvidersCacheStatus,
|
mockUpdateAndShowConnectedProvidersCacheStatus,
|
||||||
}))
|
}))
|
||||||
|
|
||||||
mock.module("./hook/model-capabilities-status", () => ({
|
mock.module("../hook/model-capabilities-status", () => ({
|
||||||
refreshModelCapabilitiesOnStartup: mockRefreshModelCapabilitiesOnStartup,
|
refreshModelCapabilitiesOnStartup: mockRefreshModelCapabilitiesOnStartup,
|
||||||
}))
|
}))
|
||||||
|
|
||||||
mock.module("./hook/startup-toasts", () => ({
|
mock.module("../hook/startup-toasts", () => ({
|
||||||
showLocalDevToast: mockShowLocalDevToast,
|
showLocalDevToast: mockShowLocalDevToast,
|
||||||
showVersionToast: mockShowVersionToast,
|
showVersionToast: mockShowVersionToast,
|
||||||
}))
|
}))
|
||||||
|
|
||||||
mock.module("./hook/background-update-check", () => ({
|
mock.module("../hook/background-update-check", () => ({
|
||||||
runBackgroundUpdateCheck: mockRunBackgroundUpdateCheck,
|
runBackgroundUpdateCheck: mockRunBackgroundUpdateCheck,
|
||||||
}))
|
}))
|
||||||
|
|
||||||
mock.module("./checker", () => ({
|
mock.module("../checker", () => ({
|
||||||
getCachedVersion: mockGetCachedVersion,
|
getCachedVersion: mockGetCachedVersion,
|
||||||
getLocalDevVersion: mockGetLocalDevVersion,
|
getLocalDevVersion: mockGetLocalDevVersion,
|
||||||
}))
|
}))
|
||||||
|
|
||||||
mock.module("../../shared/logger", () => ({
|
mock.module("../../../shared/logger", () => ({
|
||||||
log: () => {},
|
log: () => {},
|
||||||
}))
|
}))
|
||||||
|
|
||||||
@@ -49,10 +49,10 @@ afterAll(() => {
|
|||||||
mock.restore()
|
mock.restore()
|
||||||
})
|
})
|
||||||
|
|
||||||
type HookFactory = typeof import("./hook").createAutoUpdateCheckerHook
|
type HookFactory = typeof import("../hook").createAutoUpdateCheckerHook
|
||||||
|
|
||||||
async function importFreshHookFactory(): Promise<HookFactory> {
|
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
|
return hookModule.createAutoUpdateCheckerHook
|
||||||
}
|
}
|
||||||
|
|
||||||
Reference in New Issue
Block a user