Skip Menu |
 

This queue is for tickets about the Test-Perl-Critic-Progressive CPAN distribution.

Report information
The Basics
Id: 93744
Status: new
Priority: 0/
Queue: Test-Perl-Critic-Progressive

People
Owner: Nobody in particular
Requestors: ran.eilam [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: (no value)
Fixed in: (no value)



Subject: diag() and note() should be called after, not before $TEST->ok()
Download (untitled) / with headers
text/plain 415b
Test::More, and everybody else I could find do it this way. Except Test::Vars that is, but that is being fixed. Test::Perl::Critic does it the correct way, but Test::Perl::Critic::Progressive, for implementation reasons, first sends out diag() messages, then calls $TEST->ok(). I we don't fix this, there are several cases where it is impossible for TAP formatters to understand where the diag() messages belong.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.