Minor kasp test fixes

Two minor fixes in the kasp system test:

1. A wrong comment in ns3/setup.sh (we are subtracting 2 hours, not
   adding them).
2. 'get_keyids' used bad parameters "$1" "$2" when 'check_numkeys'
   failed. Also, 'check_numkeys' can use $DIR, $ZONE, and $NUMKEYS
   directly, no need to pass them.

diff --git a/bin/tests/system/kasp/ns3/setup.sh b/bin/tests/system/kasp/ns3/setup.sh
index 23ba82c8456a00f2bd00e18a1886db59bf4f01cd..2ffca051e938ffac6a2db4f6efc6d6551f55e69d 100644 (file)
--- a/bin/tests/system/kasp/ns3/setup.sh
+++ b/bin/tests/system/kasp/ns3/setup.sh
@@ -748,7 +748,7 @@ setup step5.ksk-doubleksk.autosign
 # Tsbm(N+1) = now - 50h - 2h = now - 52h
 # Tact(N+1) = Tret(N)
 # Tret(N+1) = now + 1390h - 2h = now + 1388h
-# Trem(N+1) = now + 60d + 2h = now + 1442h
+# Trem(N+1) = now + 60d - 2h = now + 1442h
 TactN="now-1492h"
 TretN="now-52h"
 TremN="now-2h"

diff --git a/bin/tests/system/kasp/tests.sh b/bin/tests/system/kasp/tests.sh
index ccb0c19032be0bc18c5f0ba608b849e23b8a9f76..2b423058e93b538a084416b9b5886e63ad40dc3c 100644 (file)
--- a/bin/tests/system/kasp/tests.sh
+++ b/bin/tests/system/kasp/tests.sh
@@ -855,7 +855,7 @@ status=$((status+ret))
 next_key_event_threshold=$((next_key_event_threshold+i))
 
 check_numkeys() {
-       _numkeys=$(get_keyids "$1" "$2" | wc -l)
+       _numkeys=$(get_keyids "$DIR" "$ZONE" | wc -l)
        test "$_numkeys" -eq "$NUM_KEYS" || return 1
        return 0
 }
@@ -872,9 +872,9 @@ check_keys() {
        ret=0
 
        echo_i "check number of keys for zone ${ZONE} in dir ${DIR} ($n)"
-       retry_quiet 10 check_numkeys "$DIR" "$ZONE" "$NUM_KEYS" || ret=1
+       retry_quiet 10 check_numkeys || ret=1
        if [ $ret -ne 0 ]; then
-               _numkeys=$(get_keyids "$1" "$2" | wc -l)
+               _numkeys=$(get_keyids "$DIR" "$ZONE" | wc -l)
                log_error "bad number of key files ($_numkeys) for zone $ZONE (expected $NUM_KEYS)"
                status=$((status+ret))
        fi