Add subtest todo information to the subtest_start event

[schwernbot]

From: @schwern Date: Friday Dec 02, 2011 at 19:29 GMT Orig: https://github.com/Test-More/test-more/issues/246

An entire subtest can be todo, that happen if a subtest is started in a todo context.

This is currently handled by making any failure output go to STDOUT, like a todo test, but results are not set todo. But the final subtest result is. I have issues with this, but I'm not going to worry about that now.

Test::Builder currently does this by setting $streamer->error_fh( $streamer->output_fh ). This doesn't always work, for example the debugging streamer (which Test::Builder::Tester uses) has no filehandles. This is a quirky TAP thing anyway, it's better done in the TAP formatter.

• Add "is_todo" and "reason" attributes to the subtest_start event
• Have Test::Builder fill them in instead of fooling with filehandles
• Change the TAP formatter to be aware that the subtest is todo and send output accordingly

[schwernbot]

From: @schwern Date: Tuesday Dec 06, 2011 at 06:16 GMT Orig: https://github.com/Test-More/test-more/issues/246#issuecomment-3028682

It turns out I was smart and put directives and a reason already into the subtest_start event and they're already filled in.