| Subject: | diag() and note() should be called after, not before $TEST->ok() |
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.