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 f1fcbfc05dd1a9478e6e018d61ff6f9d5fa9dfe6..148f2edc50384bf8a39abcf8ef422461efbdb755 100644 (file)
--- 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");
 }