From c7ae975ecf70fb6fc7c836f39b7f1ab4f2fe0129 Mon Sep 17 00:00:00 2001 From: Mark Andrews Date: Tue, 28 Feb 2023 14:24:20 +1100 Subject: [PATCH] Fix 'checking drop statistics' test Wait for the desired log message to appear in ns3/named.stats rather than the creation of the file. --- bin/tests/system/fetchlimit/clean.sh | 4 ++-- bin/tests/system/fetchlimit/tests.sh | 6 ++---- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/bin/tests/system/fetchlimit/clean.sh b/bin/tests/system/fetchlimit/clean.sh index f0158b1710..935d91bba8 100644 --- a/bin/tests/system/fetchlimit/clean.sh +++ b/bin/tests/system/fetchlimit/clean.sh @@ -12,8 +12,8 @@ # information regarding copyright ownership. rm -f */named.conf */named.memstats */ans.run */named.recursing */named.run -rm -f dig.out* rm -f ans4/norespond -rm -f ns3/named.stats ns3/named_dump.db rm -f burst.input.* +rm -f dig.out* rm -f ns*/managed-keys.bind* +rm -f ns3/named.stats ns3/named.stats.prev ns3/named_dump.db diff --git a/bin/tests/system/fetchlimit/tests.sh b/bin/tests/system/fetchlimit/tests.sh index 9ca4ff7a7e..443556a71f 100644 --- a/bin/tests/system/fetchlimit/tests.sh +++ b/bin/tests/system/fetchlimit/tests.sh @@ -183,11 +183,9 @@ status=$((status+ret)) echo_i "checking drop statistics" rm -f ns3/named.stats +touch ns3/named.stats $RNDCCMD stats -for try in 1 2 3 4 5; do - [ -f ns3/named.stats ] && break - sleep 1 -done +wait_for_log 5 "queries dropped due to recursive client limit" ns3/named.stats || ret=1 drops=`grep 'queries dropped due to recursive client limit' ns3/named.stats | sed 's/\([0-9][0-9]*\) queries.*/\1/'` [ "${drops:-0}" -ne 0 ] || ret=1 if [ $ret != 0 ]; then echo_i "failed"; fi