Skip to content

Commit 8e659a7

Browse files
Dementii PriadkoNikolayS
authored andcommitted
fix(cli): increase integration test cleanup timeouts to 30s
1 parent 3e7e1c4 commit 8e659a7

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

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)