From 269a32a6d2089bdddf7b87e185811a661a041ad9 Mon Sep 17 00:00:00 2001 From: Marcin Owsiany Date: Mon, 13 Oct 2025 10:31:27 +0200 Subject: [PATCH] test: protect unknown hook delete policies Signed-off-by: Marcin Owsiany --- pkg/action/hooks_test.go | 32 ++++++++++++++++++++++++++++++++ 1 file changed, 32 insertions(+) diff --git a/pkg/action/hooks_test.go b/pkg/action/hooks_test.go index 9502737d7..05b97db9c 100644 --- a/pkg/action/hooks_test.go +++ b/pkg/action/hooks_test.go @@ -408,3 +408,35 @@ data: }) } } + +func TestConfiguration_hookSetDeletePolicy(t *testing.T) { + tests := map[string]struct { + policies []release.HookDeletePolicy + expected []release.HookDeletePolicy + }{ + "no polices specified result in the default policy": { + policies: nil, + expected: []release.HookDeletePolicy{ + release.HookBeforeHookCreation, + }, + }, + "unknown policy is untouched": { + policies: []release.HookDeletePolicy{ + release.HookDeletePolicy("never"), + }, + expected: []release.HookDeletePolicy{ + release.HookDeletePolicy("never"), + }, + }, + } + for name, tt := range tests { + t.Run(name, func(t *testing.T) { + cfg := &Configuration{} + h := &release.Hook{ + DeletePolicies: tt.policies, + } + cfg.hookSetDeletePolicy(h) + assert.Equal(t, tt.expected, h.DeletePolicies) + }) + } +}