|Subject:||'use_tap_harness' should die on test failure|
|Date:||Thu, 27 Aug 2009 15:44:14 -0700|
|To:||bug-module-build [...] rt.cpan.org|
|From:||"David E. Wheeler" <dwheeler [...] cpan.org>|
I'm using the "use_tap_harness" option in a project, and a colleague, who set up a pre-checkin hook to make sure that all tests pass, discovered that `./Build test` was exiting normally even when tests pass. Such is not the case when using Test::Harness, which `die`s on failure. TAP::Harness doesn't die, though; it's up to the caller to die. So the attached patch calls `exit 1` if the tests fail. I'd say that this is a bug, given that it means that the behavior of `./Build test` varies depending on whether or not one uses TAP::Harness. Best, David
Message body is not shown because sender requested not to inline it.