]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
add missing "failed" message in digdelv test
authorEvan Hunt <each@isc.org>
Mon, 3 Feb 2025 19:48:08 +0000 (11:48 -0800)
committerEvan Hunt <each@isc.org>
Tue, 18 Feb 2025 20:15:00 +0000 (20:15 +0000)
there was a test case that could fail (and did) without logging
the fact.

bin/tests/system/digdelv/tests.sh

index e5c28faf790a5ab468215e62a5b88c5daacd8595..66b20f6b0a23e9a200cff8a0dc96800af5b0fab8 100644 (file)
@@ -1483,6 +1483,7 @@ if [ -x "$DELV" ]; then
 
   n=$((n + 1))
   echo_i "check that delv handles REFUSED when chasing DS records ($n)"
+  ret=0
   delv_with_opts @10.53.0.2 +root xxx.example.tld A >delv.out.test$n 2>&1 || ret=1
   grep ";; resolution failed: broken trust chain" delv.out.test$n >/dev/null || ret=1
   if [ $ret -ne 0 ]; then echo_i "failed"; fi
@@ -1490,9 +1491,11 @@ if [ -x "$DELV" ]; then
 
   n=$((n + 1))
   echo_i "check NS output from delv +ns ($n)"
+  ret=0
   delv_with_opts -i +ns +nortrace +nostrace +nomtrace +novtrace +hint=../_common/root.hint ns example >delv.out.test$n || ret=1
   lines=$(awk '$1 == "example." && $4 == "NS" {print}' delv.out.test$n | wc -l)
   [ $lines -eq 2 ] || ret=1
+  if [ $ret -ne 0 ]; then echo_i "failed"; fi
   status=$((status + ret))
 
   n=$((n + 1))