test(firewall): use require.NoError to halt on fatal errors

cmd/firewall/allow_test.go

@@ -94,7 +94,7 @@ func TestEmitAllowRevoke_PostsOneAllowToCoold(t *testing.T) {
 	rootCmdFor(inner)
 
 	err := emitAllowRevoke(context.Background(), inner, parent, local, fr, false)
-	assert.NoError(t, err)
+	require.NoError(t, err)
 
 	var posts []string
 	for _, c := range fr.calls {
@@ -185,7 +185,7 @@ func TestEmitAllowRevoke_RevokeIssuesDelete(t *testing.T) {
 	rootCmdFor(inner)
 
 	err := emitAllowRevoke(context.Background(), inner, parent, local, fr, true)
-	assert.NoError(t, err)
+	require.NoError(t, err)
 
 	var deletes []string
 	for _, c := range fr.calls {

cmd/firewall/containers_test.go

@@ -6,6 +6,7 @@ import (
 
 	"github.com/spf13/cobra"
 	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
 
 	"github.com/coollabsio/coolify-cli/cmd/common"
 )
@@ -67,7 +68,7 @@ func TestEmitContainers_AllNamespaces_FansOutAcrossNetworks(t *testing.T) {
 	rootCmdFor(inner)
 
 	err := emitContainers(context.Background(), inner, parent, fr)
-	assert.NoError(t, err)
+	require.NoError(t, err)
 
 	// Expect one `podman network ls` discovery call + one `podman ps` per
 	// discovered namespace (default + alpha = 2).