Eric Wilhelm wrote:
# from David Cantrell
No. If AUTHOR_TESTING, fail miserably unless the pod and coverage
both 1. gets tested and 2. passes. That means the Test::Pod::*
module in question must load.
Wrong. If AUTHOR_TESTING then surely all tests *must* pass both with
and *without* the optional module!
What good does it do to skip the test if AUTHOR_TESTING? That just gets
us back into the same situation as the current one where the author
didn't have Test::Pod and pod which fails the tests got shipped.
Let us assume that I write a module that, if you have a particular
module installed will do some extra stuff. This is very common.
Before uploading my module to the CPAN I need to make sure that it
passes its tests both with and without the optional module. In fact,
ideally I need to test with all possible combinations of optional
modules including none of them. It makes no difference whether the
optional module is Test::Pod or DateTime.
Skipping tests because you correctly identify that the optional module
isn't available is, of course, counted as passing.
--
David Cantrell
