Skip to content

Commit 6ecce40

Browse files
committed
test(webapp): give onTestFinished cleanup the full timeout budget
engine.quit() drains workers and can take over 10s on CI. onTestFinished callbacks run under hookTimeout (default 10s), not testTimeout, so the cleanup added in the previous commit timed out ("Hook timed out in 10000ms") where the original in-body `await engine.quit()` had the 60s test budget. Set hookTimeout to 60s to match.
1 parent d055f7c commit 6ecce40

6 files changed

Lines changed: 6 additions & 6 deletions

apps/webapp/test/engine/triggerTask.debounce.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ import {
3636
MockTriggerTaskValidator,
3737
} from "./triggerTaskTestHelpers";
3838

39-
vi.setConfig({ testTimeout: 60_000 });
39+
vi.setConfig({ testTimeout: 60_000, hookTimeout: 60_000 });
4040

4141
describe("RunEngineTriggerTaskService", () => {
4242
containerTest(

apps/webapp/test/engine/triggerTask.idempotency.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ import {
3535
MockTriggerTaskValidator,
3636
} from "./triggerTaskTestHelpers";
3737

38-
vi.setConfig({ testTimeout: 60_000 });
38+
vi.setConfig({ testTimeout: 60_000, hookTimeout: 60_000 });
3939

4040
describe("RunEngineTriggerTaskService", () => {
4141
containerTest("should handle idempotency keys correctly", async ({ prisma, redisOptions }) => {

apps/webapp/test/engine/triggerTask.metadataCache.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ import {
3535
MockTriggerTaskValidator,
3636
} from "./triggerTaskTestHelpers";
3737

38-
vi.setConfig({ testTimeout: 60_000 });
38+
vi.setConfig({ testTimeout: 60_000, hookTimeout: 60_000 });
3939

4040
describe("DefaultQueueManager task metadata cache", () => {
4141
containerTest(

apps/webapp/test/engine/triggerTask.mollifier.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ import {
3535
MockTriggerTaskValidator,
3636
} from "./triggerTaskTestHelpers";
3737

38-
vi.setConfig({ testTimeout: 60_000 });
38+
vi.setConfig({ testTimeout: 60_000, hookTimeout: 60_000 });
3939

4040
describe("RunEngineTriggerTaskService", () => {
4141
// ─── Mollifier integration ──────────────────────────────────────────────────

apps/webapp/test/engine/triggerTask.residency.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ import {
3838
MockTriggerTaskValidator,
3939
} from "./triggerTaskTestHelpers";
4040

41-
vi.setConfig({ testTimeout: 60_000 });
41+
vi.setConfig({ testTimeout: 60_000, hookTimeout: 60_000 });
4242

4343
describe("RunEngineTriggerTaskService — child run residency inheritance", () => {
4444
// Helper: stand up an engine + service wired for a single (real) Postgres/Redis

apps/webapp/test/engine/triggerTask.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ import {
3333
MockTriggerTaskValidator,
3434
} from "./triggerTaskTestHelpers";
3535

36-
vi.setConfig({ testTimeout: 60_000 });
36+
vi.setConfig({ testTimeout: 60_000, hookTimeout: 60_000 });
3737

3838
describe("RunEngineTriggerTaskService", () => {
3939
containerTest("should trigger a task with minimal options", async ({ prisma, redisOptions }) => {

0 commit comments

Comments
 (0)