Skip Menu |

This queue is for tickets about the Test-Pod-Coverage CPAN distribution.

Report information
The Basics
Id: 23468
Status: new
Priority: 0/
Queue: Test-Pod-Coverage

People
Owner: Nobody in particular
Requestors: rjbs [...] cpan.org
Cc:
AdminCc:

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



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.