forked from mirror/coolify-cli
c9b6df3171
Rename `FirewallFlags` to `Flags` and `bindFirewallFlags` to `bindFlags` within the firewall package — the `Firewall` prefix is redundant inside the `firewall` package. Drop the unused error return from `discoverNamespacesOnHosts`; the function accumulates per-host errors into `ServerResult` slices and has no package-level error path, so the third return value was always nil. Also switches test assertions from `assert.Error/NoError` to `require.Error/NoError` where the test cannot continue meaningfully on failure, and adds broker service tests.