Subject: | patch: per-module overrides to all_pod_coverage_ok |
Date: | Fri, 17 Nov 2006 17:41:05 -0500 |
To: | bug-test-pod-coverage [...] rt.cpan.org |
From: | Ricardo SIGNES <rjbs [...] cpan.org> |
This patch adds a new parm to all_pod_coverage_ok, C<for>.
This parameter lets you say, "Test everything, and use the parameters I gave
you... but this one module, it's special. Let me override some of its
parameters." This is great if you have a dist with a whole lot of modules, and
only one needs special settings. You don't need to use pod_coverage_ok for
every single module and keep adding more. You can just use all_pod_coverage_ok
as usual, but add the exceptions as they arise.
There are a few things to note:
(a) I found a bug in all_modules, for which I added a TODO test, in this
patch. It briefly stymied my adding tests.
(b) I refactored all_pod_coverage_ok to make it simpler to test.
(c) The tests which I added do not, sadly, pass. I don't understand the test
tester sufficiently, but as they tests do show the correct responses, I
figure it's a SMOP to fix the tests. If this is not true, I will spend more
time learning how to fix them myself.
Patch attached.
--
rjbs
Message body is not shown because sender requested not to inline it.