Fix a flaky reconcile test when delete

kubernetes-sigs · Jan 8, 2019 · 985a292 · 985a292
1 parent 6dcb234
commit 985a292
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/pkg/scaffold/controller/controllertest.go b/pkg/scaffold/controller/controllertest.go
@@ -138,7 +138,8 @@ func TestReconcile(t *testing.T) {
 		Should(gomega.Succeed())
 
 	// Manually delete Deployment since GC isn't enabled in the test control plane
-	g.Expect(c.Delete(context.TODO(), deploy)).To(gomega.Succeed())
+	g.Eventually(func() error { return c.Delete(context.TODO(), deploy) }, timeout).
+	    Should(gomega.MatchError("deployments.apps \"foo-deployment\" not found"))
 {{ end }}
 }
 `
diff --git a/test/project/pkg/controller/firstmate/firstmate_controller_test.go b/test/project/pkg/controller/firstmate/firstmate_controller_test.go
@@ -82,6 +82,7 @@ func TestReconcile(t *testing.T) {
 		Should(gomega.Succeed())
 
 	// Manually delete Deployment since GC isn't enabled in the test control plane
-	g.Expect(c.Delete(context.TODO(), deploy)).To(gomega.Succeed())
+	g.Eventually(func() error { return c.Delete(context.TODO(), deploy) }, timeout).
+		Should(gomega.MatchError("deployments.apps \"foo-deployment\" not found"))
 
 }