From 7ea025c64d26494954ed4ade16b4d9ad1ed12dc7 Mon Sep 17 00:00:00 2001 From: Rob von Behren Date: Fri, 18 Sep 2026 21:24:59 +0000 Subject: [PATCH] fix(smoketests): give the snapshot devbox tests the shared provisioning timeout Both tests in snapshots.test.ts override jest's 5 minute smoketest default with a 30 second per-test timeout even though each one provisions a devbox. Devbox provisioning routinely takes 35 to 70 seconds right after a dev deploy while node agents restart, so the tests failed on latency that is within the platform's provisioning budget rather than on a real regression. Use the shared MEDIUM_TIMEOUT (5 minutes) like the other devbox-creating smoketests do. --- tests/smoketests/snapshots.test.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/smoketests/snapshots.test.ts b/tests/smoketests/snapshots.test.ts index 3faf58e22..c1abd32e9 100644 --- a/tests/smoketests/snapshots.test.ts +++ b/tests/smoketests/snapshots.test.ts @@ -1,5 +1,5 @@ import { DevboxView } from '@runloop/api-client/resources/devboxes'; -import { makeClient, uniqueName } from './utils'; +import { makeClient, MEDIUM_TIMEOUT, uniqueName } from './utils'; const client = makeClient(); @@ -29,7 +29,7 @@ describe('smoketest: devbox snapshots', () => { await client.devboxes.shutdown(devbox.id); } } - }, 30_000); + }, MEDIUM_TIMEOUT); test('launch devbox from snapshot (deprecated polling path)', async () => { const warnSpy = jest.spyOn(console, 'warn').mockImplementation((...args: unknown[]) => { @@ -54,5 +54,5 @@ describe('smoketest: devbox snapshots', () => { await client.devboxes.shutdown(devbox.id); } } - }, 30_000); + }, MEDIUM_TIMEOUT); });