From efd5698597a11399c9f70f47cdfdbb43e421f773 Mon Sep 17 00:00:00 2001 From: AleksaMCode Date: Thu, 17 Sep 2026 22:03:11 +0200 Subject: [PATCH] lambda/event: fix TargetIDSet.IsEmpty and add regression test Signed-off-by: AleksaMCode --- internal/config/lambda/event/targetidset.go | 2 +- internal/config/lambda/event/targetidset_test.go | 16 ++++++++++++++++ 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/internal/config/lambda/event/targetidset.go b/internal/config/lambda/event/targetidset.go index eea481026..1779b6c8b 100644 --- a/internal/config/lambda/event/targetidset.go +++ b/internal/config/lambda/event/targetidset.go @@ -24,7 +24,7 @@ type TargetIDSet map[TargetID]struct{} // IsEmpty returns true if the set is empty. func (set TargetIDSet) IsEmpty() bool { - return len(set) != 0 + return len(set) == 0 } // Clone - returns copy of this set. diff --git a/internal/config/lambda/event/targetidset_test.go b/internal/config/lambda/event/targetidset_test.go index d798992ea..057b49fbd 100644 --- a/internal/config/lambda/event/targetidset_test.go +++ b/internal/config/lambda/event/targetidset_test.go @@ -22,6 +22,22 @@ import ( "testing" ) +func TestTargetIDSetIsEmpty(t *testing.T) { + testCases := []struct { + set TargetIDSet + expected bool + }{ + {NewTargetIDSet(), true}, + {NewTargetIDSet(TargetID{"1", "webhook"}), false}, + } + + for i, testCase := range testCases { + if got := testCase.set.IsEmpty(); got != testCase.expected { + t.Fatalf("test %v: expected: %v, got: %v", i+1, testCase.expected, got) + } + } +} + func TestTargetIDSetClone(t *testing.T) { testCases := []struct { set TargetIDSet