Skip to content

Commit 0ac8487

Browse files
committed
Fix TestFlyDeployHAPlacement to assert success explicitly
The test was failing with 'Condition never satisfied' even when the deploy succeeded. The issue was that EventuallyWithT expects the function to call an assertion. When the deploy succeeded on the first try, we didn't call any assertions, so EventuallyWithT timed out waiting for a success signal. Fix: Add assert.True(c, true) when deploy succeeds so EventuallyWithT knows the condition was satisfied.
1 parent 9e42d1d commit 0ac8487

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
@@ -108,6 +108,9 @@ func TestFlyDeployHAPlacement(t *testing.T) {
108108
t.Logf("Deploy failed with unexpected error (will not retry): %s", stderr)
109109
assert.Fail(c, fmt.Sprintf("deploy failed with unexpected error: %s", stderr))
110110
}
111+
} else {
112+
// Explicitly assert success so EventuallyWithT knows we passed
113+
assert.True(c, true, "deploy succeeded")
111114
}
112115
}, 30*time.Second, 5*time.Second, "deploy should succeed after Corrosion replication, last error: %s", lastError)
113116

0 commit comments

Comments
 (0)