🌱 Return Unknown condition for NewAggregateCondition when no…

util/conditions/v1beta2/aggregate.go

@@ -59,7 +59,11 @@ func (o *AggregateOptions) ApplyOptions(opts []AggregateOption) *AggregateOption
 // Additionally, it is possible to inject custom merge strategies using the CustomMergeStrategy option.
 func NewAggregateCondition[T Getter](sourceObjs []T, sourceConditionType string, opts ...AggregateOption) (*metav1.Condition, error) {
 	if len(sourceObjs) == 0 {
-		return nil, errors.New("sourceObjs can't be empty")
+		return &metav1.Condition{
+			Type:   sourceConditionType,
+			Status: metav1.ConditionUnknown,
+			Reason: NotYetReportedReason,
+		}, nil
 	}
 
 	aggregateOpt := &AggregateOptions{

util/conditions/v1beta2/aggregate_test.go

@@ -69,8 +69,10 @@
 			wantErr: false,
 		},
 		{
-			name:          "Error if negative polarity conditions are misconfigured",
-			conditions:    [][]metav1.Condition{},
+			name: "Error if negative polarity conditions are misconfigured",
+			conditions: [][]metav1.Condition{
+				{{Type: clusterv1.ScalingUpV1Beta2Condition, Status: metav1.ConditionTrue, Reason: "Reason-1", Message: "Message-1"}},
+			},
 			conditionType: clusterv1.ScalingUpV1Beta2Condition,
 			options:       []AggregateOption{NegativePolarityConditionTypes{"foo"}}, // NegativePolarityConditionTypes if set must equal source condition
 			want:          nil,
@@ -420,6 +422,16 @@
 			},
 			wantErr: false,
 		},
+		{
+			name:          "Empty source objects return an unknown condition with status not yet reported",
+			conditionType: clusterv1.AvailableV1Beta2Condition,
+			want: &metav1.Condition{
+				Type:   clusterv1.AvailableV1Beta2Condition,
+				Status: metav1.ConditionUnknown,
+				Reason: NotYetReportedReason,
+			},
+			wantErr: false,
+		},
 	}
 
 	for _, tt := range tests {
@@ -446,10 +458,4 @@
 		})
 	}
 
-	t.Run("Fails if source objects are empty", func(t *testing.T) {
-		var objs []*builder.Phase3Obj
-		g := NewWithT(t)
-		_, err := NewAggregateCondition(objs, clusterv1.AvailableV1Beta2Condition)
-		g.Expect(err).To(HaveOccurred())
-	})
 }