On 30 Oct 2007, at 09:50, Ovid wrote:
This sounds reasonable. It seems to me that TAP::Formatter::Console::Session somewhere should have a line like:

 if ( $parser->skip_all ) {
     $formatter->_output( '# SKIP:  ' . $parser->skip_all );
 }

Or something like that.

I've added logic that produces output like this:

13:54] andy $ prove t/sample-tests/skipall t/sample-tests/ skipall_nomsg t/sample-tests/simple
t/sample-tests/skipall..........skipped: rope
t/sample-tests/skipall_nomsg....skipped: (no reason given)
t/sample-tests/simple...........ok
All tests successful.
Files=3, Tests=5, 0 wallclock secs ( 0.01 usr 0.01 sys + 0.01 cusr 0.00 csys = 0.03 CPU)
Result: PASS

Do we like that?

However, TAP::Parser doesn't seem to have a &skip_all method (returning boolean or explanation? What if no skip reason given?).


TAP::Parser->skip_all returns a true value if all tests are skipped. It either returns the skip reason or '(no reason given)' as above.

--
Andy Armstrong, Hexten




Reply via email to