From 9ebb9645a4b6debd188e42a8650c9c124fbeb65f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jo=C3=A3o=20Pereira?= Date: Thu, 15 Aug 2024 10:58:04 -0500 Subject: [PATCH] Change the test to polling instead of just a single sleep MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Avoid flakyness due to time it takes to bind a service Signed-off-by: João Pereira --- integration/v7/isolated/service_command_test.go | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/integration/v7/isolated/service_command_test.go b/integration/v7/isolated/service_command_test.go index c2f1c340c8a..5126bf5dea4 100644 --- a/integration/v7/isolated/service_command_test.go +++ b/integration/v7/isolated/service_command_test.go @@ -171,12 +171,11 @@ var _ = Describe("service command", func() { }) It("displays the bound apps", func() { - //Delay to reduce flakiness - time.Sleep(10 * time.Second) - session := helpers.CF(serviceCommand, serviceInstanceName, "-v") - Eventually(session).Should(Exit(0)) - - Expect(session).To(SatisfyAll( + Eventually(func() *Session { + session := helpers.CF(serviceCommand, serviceInstanceName, "-v") + Eventually(session).Should(Exit(0)) + return session + }).WithPolling(5 * time.Second).WithTimeout(3 * time.Minute).Should(SatisfyAll( Say(`Showing bound apps:\n`), Say(`name\s+binding name\s+status\s+message\n`), Say(`%s\s+%s\s+create succeeded\s*\n`, appName1, bindingName1),