From 664a99cd005bc5c9d28277100b857b67d0f9a216 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Manuel=20R=C3=BCger?= Date: Thu, 12 Mar 2026 23:18:49 +0100 Subject: [PATCH] test: restore global log level after each SetLogLevel subtest SetLogLevel mutates a process-global logger, leaking state into subsequent tests and causing order-dependent failures. Save the current level before each subtest and restore it via t.Cleanup. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- util/cli_test.go | 2 ++ 1 file changed, 2 insertions(+) diff --git a/util/cli_test.go b/util/cli_test.go index 10b4809..1ebb082 100644 --- a/util/cli_test.go +++ b/util/cli_test.go @@ -97,6 +97,8 @@ func Test_setLogLevel(t *testing.T) { } for name, tt := range tests { t.Run(name, func(t *testing.T) { + prev := log.GetLevel() + t.Cleanup(func() { log.SetLevel(prev) }) cmd := &cli.Command{ Name: "test", Flags: []cli.Flag{