Merge pull request #841 from fluxcd/fix-test-flake

controller: address test flake
fluxcd · Dec 11, 2023 · 7d0aecf · 7d0aecf
2 parents f3e4789 + f64246b
commit 7d0aecf
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/internal/controller/helmrelease_controller_test.go b/internal/controller/helmrelease_controller_test.go
@@ -852,12 +852,14 @@ func TestHelmReleaseReconciler_reconcileDelete(t *testing.T) {
 		g.Expect(err).To(MatchError(helmdriver.ErrReleaseNotFound))
 
 		// Verify Helm chart has been removed.
-		err = testEnv.Get(context.TODO(), client.ObjectKey{
-			Namespace: hc.Namespace,
-			Name:      hc.Name,
-		}, &sourcev1b2.HelmChart{})
-		g.Expect(err).To(HaveOccurred())
-		g.Expect(apierrors.IsNotFound(err)).To(BeTrue())
+		g.Eventually(func(g Gomega) {
+			err = testEnv.Get(context.TODO(), client.ObjectKey{
+				Namespace: hc.Namespace,
+				Name:      hc.Name,
+			}, &sourcev1b2.HelmChart{})
+			g.Expect(err).To(HaveOccurred())
+			g.Expect(apierrors.IsNotFound(err)).To(BeTrue())
+		}).Should(Succeed())
 	})
 
 	t.Run("removes finalizer for suspended resource with DeletionTimestamp", func(t *testing.T) {