Skip to content

Commit 47a5f0e

Browse files
committed
Merge branch 'fix/cli-test-timeouts' into 'main'
fix(cli): increase integration test cleanup timeouts to 30s See merge request postgres-ai/postgres_ai!135
2 parents bc4d77b + 8e659a7 commit 47a5f0e

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

cli/test/checkup.integration.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,11 +112,12 @@ async function createTempPostgres(): Promise<TempPostgres> {
112112
const cleanup = async () => {
113113
postgresProc.kill("SIGTERM");
114114
try {
115+
// 30s timeout to handle slower CI environments gracefully
115116
await waitFor(
116117
async () => {
117118
if (postgresProc.exitCode === null) throw new Error("still running");
118119
},
119-
{ timeoutMs: 5000, intervalMs: 100 }
120+
{ timeoutMs: 30000, intervalMs: 100 }
120121
);
121122
} catch {
122123
postgresProc.kill("SIGKILL");

cli/test/init.integration.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,11 +118,12 @@ async function createTempPostgres(): Promise<TempPostgres> {
118118
const cleanup = async () => {
119119
postgresProc.kill("SIGTERM");
120120
try {
121+
// 30s timeout to handle slower CI environments gracefully
121122
await waitFor(
122123
async () => {
123124
if (postgresProc.exitCode === null) throw new Error("still running");
124125
},
125-
{ timeoutMs: 5000, intervalMs: 100 }
126+
{ timeoutMs: 30000, intervalMs: 100 }
126127
);
127128
} catch {
128129
postgresProc.kill("SIGKILL");

0 commit comments

Comments
 (0)