From 90f0768f5616f482d0f277c2ed7466d35226826d Mon Sep 17 00:00:00 2001 From: faizanahmad055 Date: Mon, 11 May 2026 22:36:46 +0200 Subject: [PATCH] Fix formatting Signed-off-by: faizanahmad055 --- internal/pkg/controller/controller.go | 4 ++- internal/pkg/controller/controller_test.go | 34 +++++++++++----------- 2 files changed, 20 insertions(+), 18 deletions(-) diff --git a/internal/pkg/controller/controller.go b/internal/pkg/controller/controller.go index 4c2ced8..287be4d 100644 --- a/internal/pkg/controller/controller.go +++ b/internal/pkg/controller/controller.go @@ -58,7 +58,9 @@ var selectedNamespacesCache atomic.Value // always stores map[string]struct{} // loadSelectedNamespaces returns the current namespace snapshot (never nil). func loadSelectedNamespaces() map[string]struct{} { if v := selectedNamespacesCache.Load(); v != nil { - return v.(map[string]struct{}) + if m, ok := v.(map[string]struct{}); ok { + return m + } } return map[string]struct{}{} } diff --git a/internal/pkg/controller/controller_test.go b/internal/pkg/controller/controller_test.go index c318c9f..b2ae32e 100644 --- a/internal/pkg/controller/controller_test.go +++ b/internal/pkg/controller/controller_test.go @@ -223,12 +223,12 @@ func TestResourceInSelectedNamespaces(t *testing.T) { for _, tt := range tests { t.Run( tt.name, func(t *testing.T) { - resetGlobalState() - storeSelectedNamespaces(tt.cachedNamespaces) + resetGlobalState() + storeSelectedNamespaces(tt.cachedNamespaces) - c := newTestController([]string{}, tt.namespaceSelector) - result := c.resourceInSelectedNamespaces(tt.resource) - assert.Equal(t, tt.expected, result) + c := newTestController([]string{}, tt.namespaceSelector) + result := c.resourceInSelectedNamespaces(tt.resource) + assert.Equal(t, tt.expected, result) }, ) } @@ -293,16 +293,16 @@ func TestRemoveSelectedNamespaceFromCache(t *testing.T) { for _, tt := range tests { t.Run( tt.name, func(t *testing.T) { - resetGlobalState() - storeSelectedNamespaces(tt.initialCache) + resetGlobalState() + storeSelectedNamespaces(tt.initialCache) - c := newTestController([]string{}, "env=prod") - ns := v1.Namespace{ - ObjectMeta: metav1.ObjectMeta{Name: tt.namespaceToRemove}, - } - c.removeSelectedNamespaceFromCache(ns) + c := newTestController([]string{}, "env=prod") + ns := v1.Namespace{ + ObjectMeta: metav1.ObjectMeta{Name: tt.namespaceToRemove}, + } + c.removeSelectedNamespaceFromCache(ns) - assert.ElementsMatch(t, tt.expectedCache, loadSelectedNamespacesList()) + assert.ElementsMatch(t, tt.expectedCache, loadSelectedNamespacesList()) }, ) } @@ -500,10 +500,10 @@ func TestUpdateHandler(t *testing.T) { for _, tt := range tests { t.Run( tt.name, func(t *testing.T) { - resetGlobalState() - if tt.cachedNamespaces != nil { - storeSelectedNamespaces(tt.cachedNamespaces) - } + resetGlobalState() + if tt.cachedNamespaces != nil { + storeSelectedNamespaces(tt.cachedNamespaces) + } c := newTestController(tt.ignoredNamespaces, tt.namespaceSelector) c.Update(tt.oldResource, tt.newResource)