tests: delete namespace until not found

Kuadrant · Apr 24, 2024 · 7cfb10c · 7cfb10c
1 parent bed695f
commit 7cfb10c
Showing 1 changed file with 10 additions and 16 deletions.
diff --git a/controllers/helper_test.go b/controllers/helper_test.go
@@ -120,29 +120,23 @@ func CreateNamespace(namespace *string) {
 }
 
 func DeleteNamespaceCallbackWithContext(ctx context.Context, namespace *string) {
-	desiredTestNamespace := &v1.Namespace{ObjectMeta: metav1.ObjectMeta{Name: *namespace}}
-	Eventually(func(g Gomega) {
-		err := k8sClient.Delete(ctx, desiredTestNamespace, client.PropagationPolicy(metav1.DeletePropagationForeground))
-		g.Expect(err).ToNot(BeNil())
-		g.Expect(apierrors.IsNotFound(err)).To(BeTrue())
+	obj := &v1.Namespace{ObjectMeta: metav1.ObjectMeta{Name: *namespace}}
+	Eventually(func() bool {
+		testClient().Delete(ctx, obj, client.PropagationPolicy(metav1.DeletePropagationForeground))
+		existingNamespace := &v1.Namespace{}
+		err := testClient().Get(ctx, types.NamespacedName{Name: *namespace}, existingNamespace)
+		return err != nil && apierrors.IsNotFound(err)
 	}).WithContext(ctx).Should(Succeed())
-
 }
 
 func DeleteNamespaceCallback(namespace *string) func() {
+	obj := &v1.Namespace{ObjectMeta: metav1.ObjectMeta{Name: *namespace}}
 	return func() {
-		desiredTestNamespace := &v1.Namespace{ObjectMeta: metav1.ObjectMeta{Name: *namespace}}
-		err := testClient().Delete(context.Background(), desiredTestNamespace, client.PropagationPolicy(metav1.DeletePropagationForeground))
-
-		Expect(err).ToNot(HaveOccurred())
-
-		existingNamespace := &v1.Namespace{}
 		Eventually(func() bool {
+			testClient().Delete(context.Background(), obj, client.PropagationPolicy(metav1.DeletePropagationForeground))
+			existingNamespace := &v1.Namespace{}
 			err := testClient().Get(context.Background(), types.NamespacedName{Name: *namespace}, existingNamespace)
-			if err != nil && apierrors.IsNotFound(err) {
-				return true
-			}
-			return false
+			return err != nil && apierrors.IsNotFound(err)
 		}, 3*time.Minute, 2*time.Second).Should(BeTrue())
 	}
 }