diff options
| author | Andy Polyakov <appro@openssl.org> | 2017-05-20 12:35:58 +0200 |
|---|---|---|
| committer | Andy Polyakov <appro@openssl.org> | 2017-05-21 11:34:01 +0200 |
| commit | c80bbcbf99fa3bf7954b2ff7aa2168c3f51f3ade (patch) | |
| tree | 8ebcb9a995377bf4835262bdd900b60d0b4435d4 /test/run_tests.pl | |
| parent | e2c1aa1ba3bc48eaaaed02c6a9a227d765e42fcb (diff) | |
| download | openssl-new-c80bbcbf99fa3bf7954b2ff7aa2168c3f51f3ade.tar.gz | |
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>
Diffstat (limited to 'test/run_tests.pl')
| -rw-r--r-- | test/run_tests.pl | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/test/run_tests.pl b/test/run_tests.pl index aa1dba0242..a04fd98a0a 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) = @_; |