diff options
| author | Craig Jennings <c@cjennings.net> | 2026-01-20 06:54:47 -0600 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-01-20 06:54:47 -0600 |
| commit | 077910c9030c3e39ff6595e4954d9d88f8d2954e (patch) | |
| tree | 663f9a0c85d2a1836ca71e8042dc6ebb3673de89 /scripts | |
| parent | 5f690e158438bdb99607d4cc10e9b983251b0fe9 (diff) | |
| download | archangel-077910c9030c3e39ff6595e4954d9d88f8d2954e.tar.gz archangel-077910c9030c3e39ff6595e4954d9d88f8d2954e.zip | |
Fix full-test.sh exiting after first passed test
Bug: ((TESTS_PASSED++)) returns exit code 1 when TESTS_PASSED is 0,
because post-increment evaluates the old value (0) which is falsy.
With set -e, this caused the script to exit after the first test passed.
Fix: Use pre-increment ((++TESTS_PASSED)) which returns the new value.
Diffstat (limited to 'scripts')
| -rwxr-xr-x | scripts/full-test.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/full-test.sh b/scripts/full-test.sh index 085e248..811dfa6 100755 --- a/scripts/full-test.sh +++ b/scripts/full-test.sh @@ -71,7 +71,7 @@ FAILED_TESTS=() info() { echo -e "${GREEN}[INFO]${NC} $1"; } warn() { echo -e "${YELLOW}[WARN]${NC} $1"; } error() { echo -e "${RED}[ERROR]${NC} $1"; } -pass() { echo -e "${GREEN}[PASS]${NC} $1"; ((TESTS_PASSED++)); } +pass() { echo -e "${GREEN}[PASS]${NC} $1"; ((++TESTS_PASSED)); } fail() { echo -e "${RED}[FAIL]${NC} $1"; ((TESTS_FAILED++)); FAILED_TESTS+=("$1"); } banner() { |