Skip to content

Commit bf2d41f

Browse files
committed
test: ownerReference on created static credential provider response Secret
1 parent 9a4fa80 commit bf2d41f

File tree

1 file changed

+18
-12
lines changed
  • pkg/handlers/generic/mutation/imageregistries/credentials

1 file changed

+18
-12
lines changed

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

Lines changed: 18 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@ var _ = Describe("Generate Image registry patches", func() {
144144

145145
testDefs := []struct {
146146
capitest.PatchTestDef
147-
expectOwnerReferenceOnSecret bool
147+
expectOwnerReferenceOnSecrets bool
148148
}{
149149
{
150150
PatchTestDef: capitest.PatchTestDef{
@@ -271,7 +271,7 @@ var _ = Describe("Generate Image registry patches", func() {
271271
},
272272
},
273273
},
274-
expectOwnerReferenceOnSecret: true,
274+
expectOwnerReferenceOnSecrets: true,
275275
},
276276
{
277277
PatchTestDef: capitest.PatchTestDef{
@@ -390,7 +390,7 @@ var _ = Describe("Generate Image registry patches", func() {
390390
},
391391
},
392392
},
393-
expectOwnerReferenceOnSecret: true,
393+
expectOwnerReferenceOnSecrets: true,
394394
},
395395
{
396396
PatchTestDef: capitest.PatchTestDef{
@@ -457,18 +457,24 @@ var _ = Describe("Generate Image registry patches", func() {
457457
It(tt.Name, func() {
458458
capitest.AssertGeneratePatches(GinkgoT(), patchGenerator, &tt.PatchTestDef)
459459

460-
// validate an OwnerReference was added to the Secret
461-
if tt.expectOwnerReferenceOnSecret {
460+
// validate an OwnerReference was added to the user provided and generated Secrets
461+
if tt.expectOwnerReferenceOnSecrets {
462462
client, err := helpers.TestEnv.GetK8sClientWithScheme(clientScheme)
463463
gomega.Expect(err).To(gomega.BeNil())
464464

465-
secret := newRegistryCredentialsSecret(validSecretName, request.Namespace)
466-
gomega.Expect(client.Get(
467-
context.Background(),
468-
ctrlclient.ObjectKeyFromObject(secret),
469-
secret,
470-
)).To(gomega.BeNil())
471-
gomega.Expect(secret.OwnerReferences).ToNot(gomega.BeEmpty())
465+
for _, name := range []string{validSecretName, credentialSecretName(request.ClusterName)} {
466+
key := ctrlclient.ObjectKey{
467+
Namespace: request.Namespace,
468+
Name: name,
469+
}
470+
secret := &corev1.Secret{}
471+
gomega.Expect(client.Get(
472+
context.Background(),
473+
key,
474+
secret,
475+
)).To(gomega.BeNil())
476+
gomega.Expect(secret.OwnerReferences).ToNot(gomega.BeEmpty())
477+
}
472478
}
473479
})
474480
}

0 commit comments

Comments
 (0)