failure message for no tests in a subtest is backwards

[karenetheridge]

When there are no tests in a subtest block:

#   Failed test 'No tests run for subtest "Test::Kwalitee import"'

This message is backwards - the test name should always assert the passing condition.

[karenetheridge]

http://grep.cpan.me/?q=No+tests+run+for+subtest+-file%3ATest%2FBuilder%5C.pm%24+-dist%3ATest-Simple+-dist%3Aperl

...shows that not too many things are referencing that exact string.

[exodist]

I think the main problem here is that diagnostics and assertions are being conflated. A subtest should fail when it has no assertions, but when it does the diagnostics should be more clear. There should be a line saying that the subtest failed, and another diagnostics line explaining why. The output should not imply that the test name involved having events within the subtest.

So yes, this is a bug, and it should be fixed, but I think simply dropping the 'No ' is a bad solution, and will still leave confusing diagnostics. I will ponder this and see if I can make the diagnostics clear and sane.