Test case without done_testing exits with 0.

[tokuhirom]

If Have_Plan is false, test script exit with status code 0. It's not expected behaviour i think.

[schwern]

Good catch, every failing test should be non-zero. I'll have a closer look at it later, but looks good at a glance.

One question, why'd you pick 254? The specific exit codes don't matter too much any more, just curious.

[tokuhirom]

I don't any reason to use 254. There is no code for specific exit code. And one of the reason is 254 is not 255 🐼

[karenetheridge]

This is awesome, BTW. And - bonus! - it doesn't break my tests that fork, although I expected it to -- after digging through the code I discovered that the 'if not have a plan, exit with error' logic doesn't apply if the exiting process doesn't match the one that Test::Builder was constructed under.

[schwern]

Great to hear! Not doing end-of-test actions in a child process has been in since 0.47, I'm glad it continues to just work.