Skip to content

Commit 24e7add

Browse files
committed
Fix improper Eventually usage in servicebrokerstub
The ensureAppIsDeployed helper was calling appResponds() immediately and passing the result (a boolean) to Eventually. This prevented Gomega from polling the function, causing failures if the app wasn't ready instantly. This change passes the function reference appResponds to Eventually so it can be polled correctly, and adds a 1-minute timeout to allow for slower environments.
1 parent cec2d2d commit 24e7add

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

integration/helpers/servicebrokerstub/app_deploy.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ func initialize() {
3737
func ensureAppIsDeployed() {
3838
if !appResponds() {
3939
ensureAppIsPushed()
40-
Eventually(appResponds()).Should(BeTrue())
40+
Eventually(appResponds, time.Minute).Should(BeTrue())
4141
}
4242
}
4343

0 commit comments

Comments
 (0)