From: Gert Doering Date: Fri, 26 Jun 2020 08:27:43 +0000 (+0200) Subject: t_client.sh: correctly report all failed instances in summary X-Git-Tag: v2.5_beta1~93 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=ec33bae311e7f3549b05de4a4c92fa7bc7144d29;p=thirdparty%2Fopenvpn.git 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 Acked-by: Antonio Quartulli 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 --- diff --git a/tests/t_client.sh.in b/tests/t_client.sh.in index 69866fb5b..294546be5 100755 --- 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