From ec33bae311e7f3549b05de4a4c92fa7bc7144d29 Mon Sep 17 00:00:00 2001 From: Gert Doering Date: Fri, 26 Jun 2020 10:27:43 +0200 Subject: [PATCH] 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 --- tests/t_client.sh.in | 1 + 1 file changed, 1 insertion(+) 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 -- 2.47.3