[tap-l] SKIP_ALL tests should not get hidden

Andy Armstrong andy at hexten.net
Tue Oct 30 08:56:07 EST 2007


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






More information about the tap-l mailing list