Fixing multipleNamespaceTest bug - Missing expect statement in test (#…

…3983) * Fixing multipleNamespaceTest bug - No expect statement in test * adding changelog Signed-off-by: Jai Subash Devmane <[email protected]>
vmware-tanzu · Jul 27, 2021 · 36b1aaa · 36b1aaa
1 parent 79b7ed1
commit 36b1aaa
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/changelogs/unreleased/3983-jaidevmane b/changelogs/unreleased/3983-jaidevmane
@@ -0,0 +1 @@
+Fixing multipleNamespaceTest bug - Missing expect statement in test
diff --git a/test/e2e/multiple_namespaces_test.go b/test/e2e/multiple_namespaces_test.go
@@ -47,8 +47,8 @@ var _ = Describe("[Basic] Backup/restore of 2 namespaces", func() {
 			backupName := "backup-" + uuidgen.String()
 			restoreName := "restore-" + uuidgen.String()
 			fiveMinTimeout, _ := context.WithTimeout(context.Background(), 5*time.Minute)
-			RunMultipleNamespaceTest(fiveMinTimeout, client, "nstest-"+uuidgen.String(), 2,
-				backupName, restoreName)
+			Expect(RunMultipleNamespaceTest(fiveMinTimeout, client, "nstest-"+uuidgen.String(), 2,
+				backupName, restoreName)).To(Succeed(), "Failed to successfully backup and restore multiple namespaces")
 		})
 	})
 })
@@ -84,8 +84,8 @@ var _ = Describe("[Scale] Backup/restore of 2500 namespaces", func() {
 			backupName := "backup-" + uuidgen.String()
 			restoreName := "restore-" + uuidgen.String()
 			oneHourTimeout, _ := context.WithTimeout(context.Background(), 1*time.Hour)
-			RunMultipleNamespaceTest(oneHourTimeout, client, "nstest-"+uuidgen.String(), 2500,
-				backupName, restoreName)
+			Expect(RunMultipleNamespaceTest(oneHourTimeout, client, "nstest-"+uuidgen.String(), 2500,
+				backupName, restoreName)).To(Succeed(), "Failed to successfully backup and restore multiple namespaces")
 		})
 	})
 })
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Fixing multipleNamespaceTest bug - Missing expect statement in test