From: Mark Andrews <marka@isc.org>
Date: Tue, 5 Jul 2022 03:01:11 +0000 (+1000)
Subject: Only report not matching stderr content when we look for it
X-Git-Tag: v9.19.3~9^2~1
X-Git-Url: http://git.ipfire.org/gitweb/index.cgi?a=commitdiff_plain;h=304d33fb32da4199e832daef027bd9228e4470a3;p=thirdparty%2Fbind9.git

Only report not matching stderr content when we look for it

The previous test code could emit "D:cds:stderr did not match ''" rather
that just showing the contents of stderr.  Moved the debug line inside
the if/else block.

Replaced backquotes with $() and $(()) as approriate.
---

diff --git a/bin/tests/system/cds/tests.sh b/bin/tests/system/cds/tests.sh
index 84fb8a5fda6..e70ec64eeac 100644
--- a/bin/tests/system/cds/tests.sh
+++ b/bin/tests/system/cds/tests.sh
@@ -17,7 +17,7 @@ status=0
 n=0
 fail() {
 	echo_i "failed"
-	status=`expr $status + 1`
+	status=$((status + 1))
 }
 
 runcmd() {
@@ -26,11 +26,11 @@ runcmd() {
 }
 
 testcase() {
-	n=`expr $n + 1`
+	n=$((n + 1))
 	echo_i "$name ($n)"
 	expect=$1
 	shift
-	result=`runcmd "$@"`
+	result=$(runcmd "$@")
 	check_stdout
 	check_stderr
 	if [ "$expect" -ne "$result" ]; then
@@ -43,10 +43,10 @@ testcase() {
 check_stderr() {
 	if [ -n "${err:=}" ]; then
 		egrep "$err" err.$n >/dev/null && return 0
+		echo_d "stderr did not match '$err'"
 	else
 		[ -s err.$n ] || return 0
 	fi
-	echo_d "stderr did not match '$err'"
 	cat err.$n | cat_d
 	fail
 }