From 4167a9562ba09ee989fd2fbfa166f0fcb2ebe7d1 Mon Sep 17 00:00:00 2001 From: Ray Walker Date: Thu, 17 Sep 2026 08:39:24 +1000 Subject: [PATCH] test(integration): give afterAll teardown the 60s hookTimeout (LAB-3756) vitest.integration.config.ts set testTimeout: 60000 but left hookTimeout at vitest's 10s default. The Redis and Memcached Testcontainers suites pass 60000 to beforeAll explicitly, but their afterAll (client quit + container stop) takes no timeout argument, so teardown ran on the 10s default and could fail the lane after every test in it had passed. The report reads "Hook timed out in 10000ms" with a code frame pointing at the beforeAll closing line, which misdirects to setup. Setting hookTimeout at the config level gives teardown the same budget as setup and covers any future Testcontainers suite without per-hook arguments. --- packages/cachekit/vitest.integration.config.ts | 1 + 1 file changed, 1 insertion(+) diff --git a/packages/cachekit/vitest.integration.config.ts b/packages/cachekit/vitest.integration.config.ts index 0c45af3..292d109 100644 --- a/packages/cachekit/vitest.integration.config.ts +++ b/packages/cachekit/vitest.integration.config.ts @@ -6,5 +6,6 @@ export default defineConfig({ environment: 'node', include: ['test/integration/**/*.test.ts'], testTimeout: 60000, + hookTimeout: 60000, }, });