Add tests for clusterclassToCluster, godoc changes

Signed-off-by: Danil-Grigorev <[email protected]>

api/v1beta1/cluster_types.go

@@ -17,6 +17,7 @@ limitations under the License.
 package v1beta1
 
 import (
+	"cmp"
 	"fmt"
 	"net"
 	"strings"
@@ -511,6 +512,7 @@ type Topology struct {
 
 	// The namespace of the ClusterClass object to create the topology.
 	// Empty namespace assumes the namespace of the cluster object.
+	// Class namespace changes are not supported by the rebase procedure.
 	// +optional
 	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=253
@@ -1046,11 +1048,7 @@ func (c *Cluster) GetClassKey() types.NamespacedName {
 		return types.NamespacedName{}
 	}
 
-	namespace := c.Spec.Topology.ClassNamespace
-	if c.Spec.Topology == nil || c.Spec.Topology.ClassNamespace == "" {
-		namespace = c.Namespace
-	}
-
+	namespace := cmp.Or(c.Spec.Topology.ClassNamespace, c.Namespace)
 	return types.NamespacedName{Namespace: namespace, Name: c.Spec.Topology.Class}
 }
 

internal/controllers/topology/cluster/cluster_controller_test.go

@@ -1577,3 +1577,53 @@ func TestReconciler_ValidateCluster(t *testing.T) {
 		})
 	}
 }
+
+func TestClusterClassToCluster(t *testing.T) {
+	g := NewWithT(t)
+	classBuilder := builder.ClusterClass(metav1.NamespaceDefault, clusterClassName1)
+	topologyBase := builder.ClusterTopology().
+		WithClass(clusterClassName1).
+		WithVersion("1.22.2").
+		WithControlPlaneReplicas(3)
+	topologyBase2 := topologyBase.DeepCopy().WithClass(clusterClassName2)
+	class2Builder := builder.ClusterClass(metav1.NamespaceDefault, clusterClassName2)
+	clusterBuilder := builder.Cluster(metav1.NamespaceDefault, clusterName1).
+		WithTopology(topologyBase.DeepCopy().Build())
+	cluster2Builder := builder.Cluster(metav1.NamespaceDefault, clusterName2).
+		WithTopology(topologyBase2.DeepCopy().Build())
+	tests := []struct {
+		name         string
+		objects      []client.Object
+		clusterClass *clusterv1.ClusterClass
+		cluster      *clusterv1.Cluster
+		expected     []reconcile.Request
+	}{
+		{
+			name:         "ClusterClass change should request reconcile for the referenced class",
+			objects:      []client.Object{clusterBuilder.DeepCopy().Build(), cluster2Builder.DeepCopy().Build()},
+			clusterClass: classBuilder.DeepCopy().Build(),
+			cluster:      clusterBuilder.DeepCopy().Build(),
+			expected:     []reconcile.Request{{NamespacedName: client.ObjectKeyFromObject(clusterBuilder.DeepCopy().Build())}},
+		},
+		{
+			name:         "Different ClusterClass change should not request reconcile if no class references it",
+			objects:      []client.Object{clusterBuilder.DeepCopy().Build(), cluster2Builder.DeepCopy().Build()},
+			clusterClass: class2Builder.DeepCopy().Build(),
+			cluster:      clusterBuilder.DeepCopy().Build(),
+			expected:     []reconcile.Request{},
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(*testing.T) {
+			fakeClient := fake.NewClientBuilder().WithScheme(fakeScheme).WithObjects(tt.objects...).Build()
+			r := &Reconciler{
+				Client:    fakeClient,
+				APIReader: fakeClient,
+			}
+
+			requests := r.clusterClassToCluster(ctx, tt.clusterClass)
+			g.Expect(requests).To(Equal(tt.expected))
+		})
+	}
+}