From c5f7be9adbb77c75eae7f652a9f3cdbb9354f13c Mon Sep 17 00:00:00 2001 From: Kynan Ware <47394200+BagToad@users.noreply.github.com> Date: Tue, 2 Sep 2025 13:14:37 -0600 Subject: [PATCH] Replace panic with require.FailNow in test Updated the test to use require.FailNow instead of panic when CapiClient is called with --web, improving test failure reporting and consistency. --- pkg/cmd/agent-task/list/list_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/cmd/agent-task/list/list_test.go b/pkg/cmd/agent-task/list/list_test.go index a1c34ac32..ef3cf8a7b 100644 --- a/pkg/cmd/agent-task/list/list_test.go +++ b/pkg/cmd/agent-task/list/list_test.go @@ -231,7 +231,7 @@ func Test_listRun(t *testing.T) { Browser: br, CapiClient: func() (*capi.CAPIClient, error) { if tt.web { - panic("CapiClient should not be invoked when --web is set") + require.FailNow(t, "CapiClient was called with --web") } return capiClient, nil },