From: Andy Polyakov <appro@openssl.org>
Date: Sun, 21 May 2017 09:40:46 +0000 (+0200)
Subject: test/run_tests.pl: don't mask test failures.
X-Git-Tag: OpenSSL_1_1_0f~8
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=c7f6ec9e5afc575df260c21ba77cf3a45a8b9b39;p=oweals%2Fopenssl.git

test/run_tests.pl: don't mask test failures.

Switch to TAP::Harness inadvertently masked test failures.
Test::Harness::runtests was terminating with non-zero exit code in case
of failure[s], while TAP::Harness apparently holds caller responsible
for doing so.

Reviewed-by: Rich Salz <rsalz@openssl.org>
Reviewed-by: Richard Levitte <levitte@openssl.org>
---

diff --git a/test/run_tests.pl b/test/run_tests.pl
index f1fcbfc05d..148f2edc50 100644
--- a/test/run_tests.pl
+++ b/test/run_tests.pl
@@ -62,7 +62,9 @@ if ($list_mode) {
     @tests = map { abs2rel($_, rel2abs(curdir())); } @tests;
 
     my $harness = $TAP_Harness->new(\%tapargs);
-    $harness->runtests(sort @tests);
+    my $ret = $harness->runtests(sort @tests);
+
+    exit $ret->has_errors if (ref($ret) eq "TAP::Parser::Aggregator");
 }