Bug #114391 for App-VW: Pod test fails

Sun May 15 08:54:51 2016 SREZIC [...] cpan.org - Ticket created

Subject:

Pod test fails

See subject. Test output: # Failed test 'POD test for blib/lib/App/VW/Setup.pm' # at /opt/perl-5.20.3/lib/site_perl/5.20.3/Test/Pod.pm line 187. # blib/lib/App/VW/Setup.pm (94): =over without closing =back # Looks like you failed 1 test of 10. t/pod.t .......... Dubious, test returned 1 (wstat 256, 0x100) Failed 1/10 subtests

Sun May 15 17:12:10 2016 ether [...] cpan.org - Correspondence added

On 2016-05-15 05:54:51, SREZIC wrote: Show quoted text

> See subject. Test output: > > # Failed test 'POD test for blib/lib/App/VW/Setup.pm' > # at /opt/perl-5.20.3/lib/site_perl/5.20.3/Test/Pod.pm line 187. > # blib/lib/App/VW/Setup.pm (94): =over without closing =back > # Looks like you failed 1 test of 10. > t/pod.t .......... > Dubious, test returned 1 (wstat 256, 0x100) > Failed 1/10 subtests

Please don't allow pod, critic 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, both of which cannot be predicted in advance. Either move them to xt/, or guard them with: plan skip_all => "These tests are for authors only!" unless $ENV{AUTHOR_TESTING} or $ENV{RELEASE_TESTING};

Sun May 15 17:12:11 2016 The RT System itself - Status changed from 'new' to 'open'

Mon May 16 14:01:00 2016 beppu [...] cpan.org - Correspondence added 3 min

On Sun May 15 17:12:10 2016, ETHER wrote: Show quoted text

> On 2016-05-15 05:54:51, SREZIC wrote:

> > See subject. Test output: > > > > # Failed test 'POD test for blib/lib/App/VW/Setup.pm' > > # at /opt/perl-5.20.3/lib/site_perl/5.20.3/Test/Pod.pm line 187. > > # blib/lib/App/VW/Setup.pm (94): =over without closing =back > > # Looks like you failed 1 test of 10. > > t/pod.t .......... > > Dubious, test returned 1 (wstat 256, 0x100) > > Failed 1/10 subtests

> > Please don't allow pod, critic 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, both of which cannot be predicted in advance. > > Either move them to xt/, or guard them with: > > plan skip_all => "These tests are for authors only!" unless > $ENV{AUTHOR_TESTING} or $ENV{RELEASE_TESTING};

I've fixed the POD error and implemented the `plan skip_all` above. Thanks for the bug report and the suggestion. https://github.com/beppu/app-vw/commit/7b2e7018d7b5df49475381f7ef6c803dadddca7d

Mon May 16 14:01:01 2016 beppu [...] cpan.org - Status changed from 'open' to 'resolved'

Mon May 16 14:01:01 2016 beppu [...] cpan.org - Taken