From 51ba689a9934a9058a06420e0c2c31181afa667f Mon Sep 17 00:00:00 2001 From: Andrew Nester Date: Mon, 9 Dec 2024 15:18:05 +0100 Subject: [PATCH] rename --- bundle/permissions/mutator.go | 2 +- bundle/permissions/mutator_test.go | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/bundle/permissions/mutator.go b/bundle/permissions/mutator.go index bb6a657bf..c4b970857 100644 --- a/bundle/permissions/mutator.go +++ b/bundle/permissions/mutator.go @@ -17,7 +17,7 @@ const CAN_MANAGE = "CAN_MANAGE" const CAN_VIEW = "CAN_VIEW" const CAN_RUN = "CAN_RUN" -var notSupportedResources = []string{"clusters", "volumes", "schemas", "quality_monitors", "registered_models"} +var unsupportedResources = []string{"clusters", "volumes", "schemas", "quality_monitors", "registered_models"} var allowedLevels = []string{CAN_MANAGE, CAN_VIEW, CAN_RUN} var levelsMap = map[string](map[string]string){ diff --git a/bundle/permissions/mutator_test.go b/bundle/permissions/mutator_test.go index 0c3f87e05..78703e90f 100644 --- a/bundle/permissions/mutator_test.go +++ b/bundle/permissions/mutator_test.go @@ -162,15 +162,15 @@ func TestWarningOnOverlapPermission(t *testing.T) { func TestAllResourcesExplicitlyDefinedForPermissionsSupport(t *testing.T) { r := config.Resources{} - for _, resource := range notSupportedResources { + for _, resource := range unsupportedResources { _, ok := levelsMap[resource] - assert.False(t, ok, fmt.Sprintf("Resource %s is defined in both levelsMap and notSupportedResources", resource)) + assert.False(t, ok, fmt.Sprintf("Resource %s is defined in both levelsMap and unsupportedResources", resource)) } for _, resource := range r.AllResources() { _, ok := levelsMap[resource.Description.PluralName] - if !slices.Contains(notSupportedResources, resource.Description.PluralName) && !ok { - assert.Fail(t, fmt.Sprintf("Resource %s is not explicitly defined in levelsMap or notSupportedResources", resource.Description.PluralName)) + if !slices.Contains(unsupportedResources, resource.Description.PluralName) && !ok { + assert.Fail(t, fmt.Sprintf("Resource %s is not explicitly defined in levelsMap or unsupportedResources", resource.Description.PluralName)) } } }