coverage

codeready-toolchain · May 1, 2024 · cde7536 · cde7536
1 parent cdd1bcd
commit cde7536
Showing 1 changed file with 20 additions and 0 deletions.
diff --git a/controllers/deactivation/deactivation_controller_test.go b/controllers/deactivation/deactivation_controller_test.go
@@ -200,8 +200,28 @@ func TestReconcile(t *testing.T) {
 			userSignupRedhat.Status.ScheduledDeactivationTimestamp = &now
 
 			r, req, cl := prepareReconcile(t, mur.Name, userTier30, mur, userSignupRedhat, config)
+
+			// First cause the status update to fail
+			cl.MockStatusUpdate = func(ctx context.Context, obj runtimeclient.Object, opts ...runtimeclient.UpdateOption) error {
+				switch obj.(type) {
+				case *toolchainv1alpha1.UserSignup:
+					return errors.New("mock error")
+				default:
+					return cl.Client.Status().Update(ctx, obj)
+				}
+			}
+
 			// when
 			res, err := r.Reconcile(context.TODO(), req)
+			require.Error(t, err)
+			require.Equal(t, "mock error", err.Error())
+
+			// Remove the mock update
+			cl.MockStatusUpdate = nil
+
+			// Attempt the reconcile again
+			res, err = r.Reconcile(context.TODO(), req)
+
 			// then
 			require.NoError(t, err)
 			require.False(t, res.Requeue, "requeue should not be set")