From c80bbcbf99fa3bf7954b2ff7aa2168c3f51f3ade Mon Sep 17 00:00:00 2001 From: Andy Polyakov Date: Sat, 20 May 2017 12:35:58 +0200 Subject: [PATCH] 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 --- 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 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) = @_; -- 2.25.1