Bug #120479 for Device-Delcom-VSI: pod coverage and pod tests fail

Thu Mar 02 15:14:44 2017 SREZIC [...] cpan.org - Ticket created

Subject:

pod coverage and pod tests fail

See subject. On my smoker systems I see the following: # Failed test 'Pod coverage on Device::Delcom::VSI' # at /opt/perl-5.24.0/lib/site_perl/5.24.0/Test/Pod/Coverage.pm line 133. # Coverage for Device::Delcom::VSI is 81.8%, with 4 naked subroutines: # dprint # dprintf # read_firmware # set_event_counter # Looks like you failed 1 test of 1. t/pod-coverage.t .. Dubious, test returned 1 (wstat 256, 0x100) Failed 1/1 subtests # Failed test 'POD test for blib/lib/Device/Delcom/VSI.pm' # at /opt/perl-5.24.0/lib/site_perl/5.24.0/Test/Pod.pm line 187. # blib/lib/Device/Delcom/VSI.pm (72): =over without closing =back # blib/lib/Device/Delcom/VSI.pm (179): Expected text after =item, not a number # blib/lib/Device/Delcom/VSI.pm (187): Expected text after =item, not a number # Looks like you failed 1 test of 1. t/pod.t ........... Dubious, test returned 1 (wstat 256, 0x100) Failed 1/1 subtests

Fri Mar 03 15:14:44 2017 ether [...] cpan.org - Correspondence added

On 2017-03-02 12:14:44, SREZIC wrote: Show quoted text

> See subject. On my smoker systems I see the following: > > # Failed test 'Pod coverage on Device::Delcom::VSI' > # at /opt/perl-5.24.0/lib/site_perl/5.24.0/Test/Pod/Coverage.pm line > 133.

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};

Fri Mar 03 15:14:44 2017 The RT System itself - Status changed from 'new' to 'open'