--- Geoffrey Young <[EMAIL PROTECTED]> wrote:

> we've discussed this a few times on this list, and I even submitted a
> 
> patch to make it so:
> 
>    http://www.mail-archive.com/perl-qa@perl.org/msg08973.html
> 
> your interface is very nice, but I think I'd also like to preserve
> the 
> ability to set it at will, outside of import.
> 
> hopefully none of us will need to run patched versions for this
> feature 
> at some point in the (near) future :)

Well, your feature is slightly different in that it calls BAIL_OUT on
failure and halts the entire test suite.  It should be simple to add if
you wanted it, though, and a my custom Test::More uses documented
behavior rather than patching.

Your desire to add set the 'die on failure' behavior at will is great.
You can wrap the 'die' behavior like this:

  sub die_on_failure {
      wrap 'Test::Builder::ok', post => sub {
          if (![ $_[0]->summary ]->[-1]) {
              die "Test failed.  Halting";
          }
      };
  }

Then just add 'die_on_failure' to @EXPORT and modify import like this:

  for my $i (0 .. $#_) {
      if ('fail' eq $_[$i]) {
          splice @_, $i, 1;
          halt_on_failure();
          last;
      }
  }

Cheers,
Ovid

--
Buy the book  - http://www.oreilly.com/catalog/perlhks/
Perl and CGI  - http://users.easystreet.com/ovid/cgi_course/
Personal blog - http://publius-ovidius.livejournal.com/
Tech blog     - http://use.perl.org/~Ovid/journal/

Reply via email to