Skip to content

Commit bf6e3a3

Browse files
Merge pull request #196 from hdresearch/fix/integration-test-flake
fix: stop CI integration test flake by bumping APILong + test timeout
2 parents 78779c7 + 33882e2 commit bf6e3a3

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

internal/app/timeouts.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ func DefaultTimeouts() Timeouts {
1616
return Timeouts{
1717
APIShort: 10 * time.Second,
1818
APIMedium: 30 * time.Second,
19-
APILong: 60 * time.Second,
19+
APILong: 120 * time.Second,
2020
BuildUpload: 600 * time.Second,
2121
SSHConnect: 5 * time.Second,
2222
}

test/testutil/helpers.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,11 @@ import (
1414
)
1515

1616
const (
17-
DefaultTimeout = 60 * time.Second
17+
// DefaultTimeout caps each `vers` invocation in integration tests.
18+
// Must exceed the CLI's longest internal context budget (APILong = 120s)
19+
// so the CLI gets room to either succeed or return a clean
20+
// context-deadline error before the test wrapper kills the process.
21+
DefaultTimeout = 180 * time.Second
1822
)
1923

2024
var (

0 commit comments

Comments
 (0)