t_client.sh: correctly report all failed instances in summary

t_client.sh reports a summary at the end:

  Test sets succeeded: none.
  Test sets failed: 1 2 3 4 5.

for tests that are skipped due to the pre-test ping check ("vpn target
IP must not ping before VPN ist started") the script forgot to add
the instance number to the summary line.  Fixed.

Signed-off-by: Gert Doering <gert@greenie.muc.de>
Acked-by: Antonio Quartulli <antonio@openvpn.net>
Message-Id: <20200626082743.15397-1-gert@greenie.muc.de>
URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg20130.html
Signed-off-by: Gert Doering <gert@greenie.muc.de>

diff --git a/tests/t_client.sh.in b/tests/t_client.sh.in
index 69866fb5bde613206cbdcd90919adeb446a207ca..294546be5807fecefe1b5e47f190a7e0afcbdfb4 100755 (executable)
--- a/tests/t_client.sh.in
+++ b/tests/t_client.sh.in
@@ -298,6 +298,7 @@ do
         echo -e "OK.\n"
     else
        echo -e "FAIL: make sure that ping hosts are ONLY reachable via VPN, SKIP test $SUF".
+       SUMMARY_FAIL="$SUMMARY_FAIL $SUF"
        exit_code=31
        continue
     fi