From fd5c8f66384a12df6367435111e4c3701986d5e0 Mon Sep 17 00:00:00 2001 From: Nader Ziada Date: Wed, 16 Oct 2024 16:22:10 -0400 Subject: [PATCH] bump wait time in integration tests change the Eventually wait to 20 seconds to make the tests pass Co-authored-by: Chris Selzo --- .gitignore | 1 + integration/deployment/backup_test.go | 2 +- integration/director/backup_test.go | 6 +++--- integration/director/restore_test.go | 2 +- 4 files changed, 6 insertions(+), 5 deletions(-) diff --git a/.gitignore b/.gitignore index 1a38030ca..af1ab59d9 100644 --- a/.gitignore +++ b/.gitignore @@ -5,3 +5,4 @@ vendor/* releases/* .idea bbr-*.err.log +integration/deployment/backup-workspace-* \ No newline at end of file diff --git a/integration/deployment/backup_test.go b/integration/deployment/backup_test.go index 43a16d114..e28a2f186 100644 --- a/integration/deployment/backup_test.go +++ b/integration/deployment/backup_test.go @@ -220,7 +220,7 @@ printf "backupcontent2" > $BBR_ARTIFACT_DIRECTORY/backupdump2 fmt.Fprintln(stdin, "no") By("waiting for the backup to finish successfully", func() { - Eventually(session, 10).Should(gexec.Exit(0)) + Eventually(session, 20).Should(gexec.Exit(0)) }) By("still completing the backup", func() { diff --git a/integration/director/backup_test.go b/integration/director/backup_test.go index 309c2000f..00d6ab993 100644 --- a/integration/director/backup_test.go +++ b/integration/director/backup_test.go @@ -351,7 +351,7 @@ backup_should_be_locked_before: Context("and the user decides to cancel the backup", func() { It("terminates", func() { - Eventually(session).Should(gbytes.Say("Backing up")) + Eventually(session, 10*time.Second).Should(gbytes.Say("Backing up")) session.Interrupt() By("printing a helpful message and waiting for user input", func() { @@ -362,7 +362,7 @@ backup_should_be_locked_before: stdin.Write([]byte("yes\n")) By("then exiting with a failure", func() { - Eventually(session, 10*time.Second).Should(gexec.Exit(1)) + Eventually(session, 20*time.Second).Should(gexec.Exit(1)) }) By("outputting a warning about cleanup", func() { @@ -389,7 +389,7 @@ backup_should_be_locked_before: stdin.Write([]byte("no\n")) By("waiting for the backup to finish successfully", func() { - Eventually(session, 10).Should(gexec.Exit(0)) + Eventually(session, 20).Should(gexec.Exit(0)) }) By("still completing the backup", func() { diff --git a/integration/director/restore_test.go b/integration/director/restore_test.go index cca9fa245..f51ec3ade 100644 --- a/integration/director/restore_test.go +++ b/integration/director/restore_test.go @@ -194,7 +194,7 @@ cat $BBR_ARTIFACT_DIRECTORY/backup > /var/vcap/store/bosh/restored_file stdin.Write([]byte("no\n")) By("waiting for the restore to finish successfully", func() { - Eventually(session, 10).Should(gexec.Exit(0)) + Eventually(session, 20).Should(gexec.Exit(0)) }) By("still completing the restore", func() {