Skip to content

Commit 451c3ea

Browse files
committed
Skip TestDeploy/Dockerfile in short mode to avoid timeout
The deploy test group takes close to 15 minutes total in CI. Adding this test (which includes a 10-second release command + health checks + BuildKit build) pushes the suite over the 15-minute timeout. Since we already have comprehensive BuildKit deploy coverage from: - TestDeployNodeApp/With_BuildKit - TestFlyDeploy_Dockerfile - TestFlyDeploy_DeployMachinesCheck - All other deploy tests with --buildkit flag ...this test is redundant in CI short mode and can run on master only.
1 parent 9f91bb2 commit 451c3ea

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

test/preflight/fly_deploy_test.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -456,6 +456,9 @@ func TestDeploy(t *testing.T) {
456456
testDeploy(t, filepath.Join(testlib.RepositoryRoot(), "test", "preflight", "fixtures", "example-buildpack"), "--depot")
457457
})
458458
t.Run("Dockerfile", func(t *testing.T) {
459+
if testing.Short() {
460+
t.Skip("Skipping in short mode: test suite approaches 15m timeout with this test included")
461+
}
459462
t.Parallel()
460463
// Dockerfiles explicitly use BuildKit with remote building
461464
testDeploy(t, filepath.Join(testlib.RepositoryRoot(), "test", "preflight", "fixtures", "example"), "--buildkit --remote-only")

0 commit comments

Comments
 (0)