From c273b8eead91e4781df298d418e2463cbe1a4d5c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Mat=C3=ADas=20Insaurralde?= Date: Thu, 8 Oct 2026 17:13:43 -0300 Subject: [PATCH] fix(controlplane): use require.Fail instead of forbidden t.Fatal in panicguard test MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The forbidigo linter forbids t.Fatal in favor of testify, which failed the controlplane and main-module lint jobs. Replace it with require.Fail, matching the existing testify idiom in the codebase. Chainloop-Trace-Sessions: caeadc08-eff1-48f5-b81f-40cdb9c97088 Signed-off-by: Matías Insaurralde --- app/controlplane/internal/panicguard/panicguard_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/controlplane/internal/panicguard/panicguard_test.go b/app/controlplane/internal/panicguard/panicguard_test.go index b6a8e7b4b..81ad6f140 100644 --- a/app/controlplane/internal/panicguard/panicguard_test.go +++ b/app/controlplane/internal/panicguard/panicguard_test.go @@ -99,6 +99,6 @@ func TestGo_NilLoggerDoesNotPanic(t *testing.T) { select { case <-done: case <-time.After(time.Second): - t.Fatal("goroutine did not run") + require.Fail(t, "goroutine did not run") } }