From edf0cae4f9b318224c081d6cc5352c9e5bfab6a4 Mon Sep 17 00:00:00 2001 From: rcourtman Date: Mon, 1 Dec 2025 16:08:21 +0000 Subject: [PATCH] test: Add edge case tests for matchesDockerIgnoredPrefix function MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - skips empty prefix in list - all empty prefixes returns false - empty name/id edge cases - matchesDockerIgnoredPrefix 92.3%→100% --- internal/alerts/alerts_test.go | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/internal/alerts/alerts_test.go b/internal/alerts/alerts_test.go index 9bafd94..202ce81 100644 --- a/internal/alerts/alerts_test.go +++ b/internal/alerts/alerts_test.go @@ -1454,6 +1454,11 @@ func TestMatchesDockerIgnoredPrefix(t *testing.T) { {name: "trimmed comparison", containerName: "runner-job", containerID: "abc", prefixes: []string{" runner- "}, want: true}, {name: "case insensitive", containerName: "Runner-Job", containerID: "abc", prefixes: []string{"runner-"}, want: true}, {name: "no match", containerName: "service", containerID: "xyz", prefixes: []string{"runner-"}, want: false}, + {name: "skips empty prefix in list", containerName: "runner-job", containerID: "abc", prefixes: []string{"", "runner-"}, want: true}, + {name: "all empty prefixes returns false", containerName: "runner-job", containerID: "abc", prefixes: []string{"", " ", ""}, want: false}, + {name: "empty name matches id", containerName: "", containerID: "runner-123", prefixes: []string{"runner-"}, want: true}, + {name: "empty id matches name", containerName: "runner-job", containerID: "", prefixes: []string{"runner-"}, want: true}, + {name: "both empty no match", containerName: "", containerID: "", prefixes: []string{"runner-"}, want: false}, } for _, tc := range tests {