[openssl-commits] [openssl] master update

[Andy Polyakov]

The branch master has been updated
       via  c80bbcbf99fa3bf7954b2ff7aa2168c3f51f3ade (commit)
      from  e2c1aa1ba3bc48eaaaed02c6a9a227d765e42fcb (commit)


- Log -----------------------------------------------------------------
commit c80bbcbf99fa3bf7954b2ff7aa2168c3f51f3ade
Author: Andy Polyakov <appro at openssl.org>
Date:   Sat May 20 12:35:58 2017 +0200

    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 at openssl.org>

-----------------------------------------------------------------------

Summary of changes:
 test/run_tests.pl | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/test/run_tests.pl b/test/run_tests.pl
index aa1dba0..a04fd98 100644
--- a/test/run_tests.pl
+++ b/test/run_tests.pl
@@ -82,7 +82,10 @@ foreach my $arg (@ARGV ? @ARGV : ('alltests')) {
 }
 
 my $harness = $TAP_Harness->new(\%tapargs);
-$harness->runtests(map { abs2rel($_, rel2abs(curdir())); } sort keys %tests);
+my $ret = $harness->runtests(map { abs2rel($_, rel2abs(curdir())); }
+                                 sort keys %tests);
+
+exit $ret->has_errors if (ref($ret) eq "TAP::Parser::Aggregator");
 
 sub find_matching_tests {
     my ($glob) = @_;