Skip Menu |

This queue is for tickets about the File-Wildcard CPAN distribution.

Report information
The Basics
Id: 22923
Status: resolved
Priority: 0/
Queue: File-Wildcard

People
Owner: Nobody in particular
Requestors: mpapet [...] yahoo.com
Cc:
AdminCc:

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



Subject: Need Test::Pod::Coverage
This package uses Test::Pod::Coverage to run all test correctly. In strawberry-perl (http://win32.perl.org/wiki/index.php?title=Strawberry_Perl) there isn't a package dependency that downloads it too.
Show quoted text
> This package uses Test::Pod::Coverage to run all test correctly.
That's not correct. Test::Pod::Coverage installation is optional, and is of primary interest to me, the author, developer and maintainer, to guarantee that all methods are documented (i.e. no naked subs without _ prefix). You don't need Test::Pod::Coverage to run any of the tests that test the module code. The test t/pod_coverage.t will (correctly) be skipped if this module is not installed. Here's a paste from make test on my "virgin perl" environment: t/01_basic........ok t/02_derived......ok t/03_absolute.....ok t/04_append.......ok t/05_symlink......ok t/06_fwf..........ok t/pod.............skipped all skipped: Test::Pod 1.00 required for testing POD t/pod_coverage....skipped all skipped: Test::Pod::Coverage 1.00 required for testing POD coverage All tests successful, 2 tests skipped. Files=8, Tests=79, 1 wallclock secs ( 1.02 cusr + 0.13 csys = 1.15 CPU) /usr/bin/make test -- OK In terms of optional modules, besides Test::Pod and Test::Pod::Coverage, there's also Params::Validate. Not having this one will limit the amount of validating of input parameters the module performs - I recommend installing Params::Validate, more strongly than the other two, as this affects more than just tests. Early versions of File::Wildcard had a dependency on Params::Validate, but there were issues with this on some platforms, preventing people from using the module. That was until I wrote Module::Optional to get round the problem.
From: mpapet [...] yahoo.com
Okay, The error message doesn't make that clear. Test::Pod::Coverage works great in strawberry-perl and your module works too. Consider the issue closed. On Thu Nov 09 15:14:37 2006, IVORW wrote: Show quoted text
> > This package uses Test::Pod::Coverage to run all test correctly.
> > That's not correct. Test::Pod::Coverage installation is optional, and is > of primary interest to me, the author, developer and maintainer, to > guarantee that all methods are documented (i.e. no naked subs without _ > prefix). > > You don't need Test::Pod::Coverage to run any of the tests that test the > module code. > > The test t/pod_coverage.t will (correctly) be skipped if this module is > not installed. Here's a paste from make test on my "virgin perl" > environment: > > t/01_basic........ok > t/02_derived......ok > t/03_absolute.....ok > t/04_append.......ok > t/05_symlink......ok > t/06_fwf..........ok > t/pod.............skipped > all skipped: Test::Pod 1.00 required for testing POD > t/pod_coverage....skipped > all skipped: Test::Pod::Coverage 1.00 required for testing POD > coverage > All tests successful, 2 tests skipped. > Files=8, Tests=79, 1 wallclock secs ( 1.02 cusr + 0.13 csys = 1.15 CPU) > /usr/bin/make test -- OK > > In terms of optional modules, besides Test::Pod and Test::Pod::Coverage, > there's also Params::Validate. Not having this one will limit the amount > of validating of input parameters the module performs - I recommend > installing Params::Validate, more strongly than the other two, as this > affects more than just tests. > > Early versions of File::Wildcard had a dependency on Params::Validate, > but there were issues with this on some platforms, preventing people > from using the module. That was until I wrote Module::Optional to get > round the problem.