From c7f6ec9e5afc575df260c21ba77cf3a45a8b9b39 Mon Sep 17 00:00:00 2001 From: Andy Polyakov Date: Sun, 21 May 2017 11:40:46 +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 Reviewed-by: Richard Levitte --- test/run_tests.pl | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) 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"); } -- 2.25.1