Bug #117428 for Parse-HP-ACU: pod coverage and pod tests fail

Fri Sep 02 02:39:36 2016 SREZIC [...] cpan.org - Ticket created

Subject:

pod coverage and pod tests fail

My smoker machines report the following failure: # Failed test 'Pod coverage on Parse::HP::ACU' # at /opt/perl-5.25.4/lib/site_perl/5.25.4/Test/Pod/Coverage.pm line 133. # Coverage for Parse::HP::ACU is 57.1%, with 3 naked subroutines: # K # K_V # V # 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/Parse/HP/ACU.pm' # at /opt/perl-5.25.4/lib/site_perl/5.25.4/Test/Pod.pm line 187. # blib/lib/Parse/HP/ACU.pm (75): =over should be: '=over' or '=over positive_number' # blib/lib/Parse/HP/ACU.pm (106): You forgot a '=back' before '=head2' # blib/lib/Parse/HP/ACU.pm (369): =over should be: '=over' or '=over positive_number' # blib/lib/Parse/HP/ACU.pm (378): =over should be: '=over' or '=over positive_number' # blib/lib/Parse/HP/ACU.pm (387): You forgot a '=back' before '=head1' # blib/lib/Parse/HP/ACU.pm (387): You forgot a '=back' before '=head1' # Looks like you failed 1 test of 1. t/pod.t ........... Dubious, test returned 1 (wstat 256, 0x100) Failed 1/1 subtests

Fri Sep 02 12:43:41 2016 ether [...] cpan.org - Correspondence added

On 2016-09-01 23:39:36, SREZIC wrote: Show quoted text

> My smoker machines report the following failure: > > # Failed test 'Pod coverage on Parse::HP::ACU' > # at /opt/perl-5.25.4/lib/site_perl/5.25.4/Test/Pod/Coverage.pm line > 133. > # Coverage for Parse::HP::ACU is 57.1%, with 3 naked subroutines: > # K > # K_V > # V > # 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/Parse/HP/ACU.pm' > # at /opt/perl-5.25.4/lib/site_perl/5.25.4/Test/Pod.pm line 187. > # blib/lib/Parse/HP/ACU.pm (75): =over should be: '=over' or '=over > positive_number' > # blib/lib/Parse/HP/ACU.pm (106): You forgot a '=back' before '=head2' > # blib/lib/Parse/HP/ACU.pm (369): =over should be: '=over' or '=over > positive_number' > # blib/lib/Parse/HP/ACU.pm (378): =over should be: '=over' or '=over > positive_number' > # blib/lib/Parse/HP/ACU.pm (387): You forgot a '=back' before '=head1' > # blib/lib/Parse/HP/ACU.pm (387): You forgot a '=back' before '=head1' > # Looks like you failed 1 test of 1. > t/pod.t ........... > Dubious, test returned 1 (wstat 256, 0x100) > Failed 1/1 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};

Fri Sep 02 12:43:41 2016 The RT System itself - Status changed from 'new' to 'open'