From 69a5a8c298aa9c8591e3b2a17c2c2b4585091e21 Mon Sep 17 00:00:00 2001 From: clawbot Date: Sun, 15 Feb 2026 22:09:59 -0800 Subject: [PATCH] fix: resolve all golangci-lint issues (fixes #32) --- internal/handlers/export_test.go | 9 +++++++++ internal/handlers/tail_validation_test.go | 18 ++++++++++-------- 2 files changed, 19 insertions(+), 8 deletions(-) create mode 100644 internal/handlers/export_test.go diff --git a/internal/handlers/export_test.go b/internal/handlers/export_test.go new file mode 100644 index 0000000..fa31b33 --- /dev/null +++ b/internal/handlers/export_test.go @@ -0,0 +1,9 @@ +package handlers + +// ExportedSanitizeTail wraps sanitizeTail for external tests. +func ExportedSanitizeTail(input string) string { + return sanitizeTail(input) +} + +// ExportedDefaultLogTail exports defaultLogTail for external tests. +const ExportedDefaultLogTail = defaultLogTail diff --git a/internal/handlers/tail_validation_test.go b/internal/handlers/tail_validation_test.go index 0ec0d7a..090b35e 100644 --- a/internal/handlers/tail_validation_test.go +++ b/internal/handlers/tail_validation_test.go @@ -1,7 +1,9 @@ -package handlers +package handlers_test import ( "testing" + + "git.eeqj.de/sneak/upaas/internal/handlers" ) func TestSanitizeTail(t *testing.T) { @@ -12,16 +14,16 @@ func TestSanitizeTail(t *testing.T) { input string expected string }{ - {"empty uses default", "", defaultLogTail}, + {"empty uses default", "", handlers.ExportedDefaultLogTail}, {"valid small number", "50", "50"}, {"valid max boundary", "500", "500"}, {"exceeds max clamped", "501", "500"}, {"very large clamped", "999999", "500"}, - {"non-numeric uses default", "abc", defaultLogTail}, - {"all keyword uses default", "all", defaultLogTail}, - {"negative uses default", "-1", defaultLogTail}, - {"zero uses default", "0", defaultLogTail}, - {"float uses default", "1.5", defaultLogTail}, + {"non-numeric uses default", "abc", handlers.ExportedDefaultLogTail}, + {"all keyword uses default", "all", handlers.ExportedDefaultLogTail}, + {"negative uses default", "-1", handlers.ExportedDefaultLogTail}, + {"zero uses default", "0", handlers.ExportedDefaultLogTail}, + {"float uses default", "1.5", handlers.ExportedDefaultLogTail}, {"one is valid", "1", "1"}, } @@ -29,7 +31,7 @@ func TestSanitizeTail(t *testing.T) { t.Run(tc.name, func(t *testing.T) { t.Parallel() - got := sanitizeTail(tc.input) + got := handlers.ExportedSanitizeTail(tc.input) if got != tc.expected { t.Errorf("sanitizeTail(%q) = %q, want %q", tc.input, got, tc.expected) }