Skip Menu |

This queue is for tickets about the Net-CalDAVTalk CPAN distribution.

Report information
The Basics
Id: 123617
Status: open
Priority: 0/
Queue: Net-CalDAVTalk

People
Owner: Nobody in particular
Requestors: ppisar [...] redhat.com
Cc:
AdminCc:

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



Subject: t/pod-coverage.t fails: 1 naked subroutine: SyncEventLinks
Since 0.11, t/pod-coverage.t test fails: # Failed test 'Pod coverage on Net::CalDAVTalk' # at /usr/share/perl5/vendor_perl/Test/Pod/Coverage.pm line 133. # Coverage for Net::CalDAVTalk is 96.0%, with 1 naked subroutine: # SyncEventLinks # Looks like you failed 1 test of 2. t/pod-coverage.t .. Dubious, test returned 1 (wstat 256, 0x100) Failed 1/2 subtests
On 2017-11-13 07:50:50, ppisar wrote: Show quoted text
> Since 0.11, t/pod-coverage.t test fails: > > # Failed test 'Pod coverage on Net::CalDAVTalk' > # at /usr/share/perl5/vendor_perl/Test/Pod/Coverage.pm line 133. > # Coverage for Net::CalDAVTalk is 96.0%, with 1 naked subroutine: > # SyncEventLinks > # Looks like you failed 1 test of 2. > t/pod-coverage.t .. > Dubious, test returned 1 (wstat 256, 0x100) > Failed 1/2 subtests
While this test failure is indeed an issue that should be corrected, please don't allow pod, critic, coverage or kwalitee tests to run for normal user installs, as they make the installation fail unnecessarily when newer versions of the tester module introduces errors that you cannot anticipate. Also spelling tests are dependent on what dictionaries are installed locally, and critic tests use all plugins that are installed, neither of which can be predicted in advance. Either move these tests to xt/, or guard them with: plan skip_all => "These tests are for authors only!" unless $ENV{AUTHOR_TESTING} or $ENV{RELEASE_TESTING};
This is fixed in 0.12, though only the naked subroutine bit. The "don't put coverage tests in by default" should probably be a bug report against whatever I used to bootstrap this project, which I don't even remember what I used - but I'll fix it in the next release.