[PATCH 1/3] tests/run-all: Fix spurious report of exit status.

[G. Branden Robinson]

Reorder statements to we don't report the "NON-ZERO" exit status of 0 in
every case where this branch is taken (because that's the exit status of
the assignment on the previous line).
---
 tests/run-all | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/run-all b/tests/run-all
index 184c1a6a..b6267b83 100644
--- a/tests/run-all
+++ b/tests/run-all
@@ -90,8 +90,8 @@ do
                if sh $x; then
                        passed=$(( $passed + 1 ))
                else
-                       failed=$(( $failed + 1 ))
                        echo "${CSTART}${x}: NON-ZERO exit status $?${CEND}"
+                       failed=$(( $failed + 1 ))
                        if [ -n "$BASH_TSTOUT_SAVEDIR" ]; then
                                mv ${BASH_TSTOUT} 
${BASH_TSTOUT_SAVEDIR}/${x}-${SUFFIX}
                                echo "${CSTART}output saved to 
${BASH_TSTOUT_SAVEDIR}/${x}-${SUFFIX}${CEND}"
-- 
2.30.2

signature.asc
Description: PGP signature