diff options
author | Andy Polyakov <appro@openssl.org> | 2017-05-21 11:40:46 +0200 |
---|---|---|
committer | Andy Polyakov <appro@openssl.org> | 2017-05-22 10:18:15 +0200 |
commit | c7f6ec9e5afc575df260c21ba77cf3a45a8b9b39 (patch) | |
tree | c4f42d3206690e5dc35dba6bd3ca3bda1e5dabc2 /test | |
parent | 8f59b3267ddbffc6dd0da780fe5a1f3dcc4a2259 (diff) |
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>
Diffstat (limited to 'test')
-rw-r--r-- | test/run_tests.pl | 4 |
1 files changed, 3 insertions, 1 deletions
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"); } |