Skip to content

Commit 4f66a6b

Browse files
committed
test(proxy-init): enable strict mode in enforce-drop harness
Address CodeRabbit on #484: switch the test harness from set -u to set -euo pipefail (the repo's shell convention) so command/pipe failures in the setup phase surface instead of being masked. The run-all-assertions design is unaffected — asserts return 0 via if/else and expected-nonzero commands are guarded with || true; verified all 13 assertions still run and the harness exits 0. Assisted-By: Claude (Anthropic AI) <noreply@anthropic.com> Signed-off-by: Hai Huang <huang195@gmail.com>
1 parent 17c05ec commit 4f66a6b

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

authbridge/proxy-init/test-enforce-drop.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
# bash, the dummy kernel module. Runs on Linux / CI (e.g. ubuntu-latest); not
1717
# on macOS. Uses `unshare --net` (not named `ip netns`) so it also works inside
1818
# nested containers. Exit code 0 = all pass.
19-
set -u
19+
set -euo pipefail
2020

2121
SCRIPT_DIR=$(CDPATH= cd -- "$(dirname -- "$0")" && pwd)
2222
INIT="${INIT_SCRIPT:-${SCRIPT_DIR}/init-iptables.sh}"

0 commit comments

Comments
 (0)