Skip to content

Commit d7920c9

Browse files
committed
test: assert ownerReference is the Cluster object
1 parent edaf1fe commit d7920c9

File tree

1 file changed

+21
-0
lines changed
  • pkg/handlers/generic/mutation/imageregistries/credentials

1 file changed

+21
-0
lines changed

pkg/handlers/generic/mutation/imageregistries/credentials/inject_test.go

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ import (
1919
clientgoscheme "k8s.io/client-go/kubernetes/scheme"
2020
clusterv1 "sigs.k8s.io/cluster-api/api/v1beta1"
2121
runtimehooksv1 "sigs.k8s.io/cluster-api/exp/runtime/hooks/api/v1alpha1"
22+
"sigs.k8s.io/cluster-api/util"
2223
ctrlclient "sigs.k8s.io/controller-runtime/pkg/client"
2324

2425
"github.com/nutanix-cloud-native/cluster-api-runtime-extensions-nutanix/api/v1alpha1"
@@ -462,6 +463,17 @@ var _ = Describe("Generate Image registry patches", func() {
462463
client, err := helpers.TestEnv.GetK8sClientWithScheme(clientScheme)
463464
gomega.Expect(err).To(gomega.BeNil())
464465

466+
// get the Cluster to use for the owner reference assertion
467+
clusterKey := ctrlclient.ObjectKey{
468+
Namespace: request.Namespace,
469+
Name: request.ClusterName,
470+
}
471+
cluster := &clusterv1.Cluster{}
472+
gomega.Expect(client.Get(
473+
context.Background(),
474+
clusterKey,
475+
cluster,
476+
)).To(gomega.BeNil())
465477
for _, name := range []string{validSecretName, credentialSecretName(request.ClusterName)} {
466478
key := ctrlclient.ObjectKey{
467479
Namespace: request.Namespace,
@@ -473,7 +485,16 @@ var _ = Describe("Generate Image registry patches", func() {
473485
key,
474486
secret,
475487
)).To(gomega.BeNil())
488+
489+
// assert the owner reference with the Cluster was added to the Secret
476490
gomega.Expect(secret.OwnerReferences).ToNot(gomega.BeEmpty())
491+
ownerRef := metav1.OwnerReference{
492+
APIVersion: clusterv1.GroupVersion.String(),
493+
Kind: cluster.Kind,
494+
UID: cluster.UID,
495+
Name: cluster.Name,
496+
}
497+
util.HasOwnerRef(secret.OwnerReferences, ownerRef)
477498
}
478499
}
479500
})

0 commit comments

Comments
 (0)