diff --git a/internal/controllers/fake_configclient_test.go b/internal/controllers/fake_configclient_test.go index a0cb3e0..bcb128d 100644 --- a/internal/controllers/fake_configclient_test.go +++ b/internal/controllers/fake_configclient_test.go @@ -44,14 +44,14 @@ func NewFakeConfigClient(fakeControllerRuntimeClient client.Client) util.ConfigC } } -func (f FakeConfigClient) SetContext(ctx context.Context) { +func (f *FakeConfigClient) SetContext(ctx context.Context) { f.ctx = ctx } -func (f FakeConfigClient) GetConfigMap(name types.NamespacedName, out *corev1.ConfigMap) error { +func (f *FakeConfigClient) GetConfigMap(name types.NamespacedName, out *corev1.ConfigMap) error { return f.client.Get(f.ctx, name, out) } -func (f FakeConfigClient) GetSecret(name types.NamespacedName, out *corev1.Secret) error { +func (f *FakeConfigClient) GetSecret(name types.NamespacedName, out *corev1.Secret) error { return f.client.Get(f.ctx, name, out) } diff --git a/internal/issuer/util/configclient.go b/internal/issuer/util/configclient.go index f8f9944..990e9a1 100644 --- a/internal/issuer/util/configclient.go +++ b/internal/issuer/util/configclient.go @@ -107,7 +107,7 @@ func (c *configClient) GetConfigMap(name types.NamespacedName, out *corev1.Confi } // Check if the client has access to the configmap resource - if ok, _ := c.accessCache[name.String()]; !ok { + if _, ok := c.accessCache[name.String()]; !ok { err := c.verifyAccessFunc("configmaps", name) if err != nil { return err @@ -132,7 +132,7 @@ func (c *configClient) GetSecret(name types.NamespacedName, out *corev1.Secret) } // Check if the client has access to the secret resource - if ok, _ := c.accessCache[name.String()]; !ok { + if _, ok := c.accessCache[name.String()]; !ok { err := c.verifyAccessFunc("secrets", name) if err != nil { return err