tmp

exp/controllers/suite_test.go

@@ -52,12 +52,12 @@ var _ = BeforeSuite(func(done Done) {
 	Expect((&MachinePoolReconciler{
 		Client:   testEnv,
 		recorder: testEnv.GetEventRecorderFor("machinepool-controller"),
-	}).SetupWithManager(ctx, testEnv.Manager, controller.Options{MaxConcurrentReconciles: 1})).To(Succeed())
+	}).SetupWithManager(ctx, testEnv.Manager, controller.Options{MaxConcurrentReconciles: 1})).To(Succeed(), "Failed to setup the MachinePoolReconciler")
 
 	By("starting the manager")
 	go func() {
 		defer GinkgoRecover()
-		Expect(testEnv.StartManager(ctx)).To(Succeed())
+		Expect(testEnv.StartManager(ctx)).To(Succeed(), "Failed to start test env")
 	}()
 
 	close(done)
@@ -66,6 +66,6 @@ var _ = BeforeSuite(func(done Done) {
 var _ = AfterSuite(func() {
 	if testEnv != nil {
 		By("tearing down the test environment")
-		Expect(testEnv.Stop()).To(Succeed())
+		Expect(testEnv.Stop()).To(Succeed(), "Failed to stop test env")
 	}
 })

util/patch/patch_test.go

@@ -479,8 +479,12 @@ var _ = Describe("Patch Helper", func() {
 					readyBefore := conditions.Get(obj, clusterv1.ReadyCondition)
 					readyAfter := conditions.Get(objAfter, clusterv1.ReadyCondition)
 
+					if !cmp.Equal(readyBefore, readyAfter) {
+						By(cmp.Diff(readyBefore, readyAfter))
+					}
+
 					return cmp.Equal(readyBefore, readyAfter)
-				}, timeout).Should(BeTrue())
+				}, timeout).Should(BeTrue(), "Failed to validating the object has been updated")
 			})
 
 		})