Skip Menu |

This queue is for tickets about the Erlang-Parser CPAN distribution.

Report information
The Basics
Id: 115216
Status: resolved
Priority: 0/
Queue: Erlang-Parser

People
Owner: anneli [...] cpan.org
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.5
Fixed in: 0.6



Subject: Pod test fails
The currently indexd Erlang-Parser (version 0.5, which might not be the intended latest version; see the other ticket in RT) fails the pod test: ... # Failed test 'POD test for blib/lib/Erlang/Parser/Parser.pm' # at /usr/perl5.25.1p/lib/site_perl/5.25.1/Test/Pod.pm line 187. # blib/lib/Erlang/Parser/Parser.pm (373): =over without closing =back # Failed test 'POD test for blib/lib/Erlang/Parser/Lexer.pm' # at /usr/perl5.25.1p/lib/site_perl/5.25.1/Test/Pod.pm line 187. # blib/lib/Erlang/Parser/Lexer.pm (177): =over without closing =back # Looks like you failed 2 tests of 38. t/pod.t ........... Dubious, test returned 2 (wstat 512, 0x200) Failed 2/38 subtests ...
On 2016-06-09 11:06:58, SREZIC wrote: Show quoted text
> The currently indexd Erlang-Parser (version 0.5, which might not be > the intended latest version; see the other ticket in RT) fails the pod > test:
While this test failure is indeed an issue that should be corrected, 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};
On 2016-06-09 16:01:21, ETHER wrote: Show quoted text
> On 2016-06-09 11:06:58, SREZIC wrote:
> > The currently indexd Erlang-Parser (version 0.5, which might not be > > the intended latest version; see the other ticket in RT) fails the > > pod > > test:
> > While this test failure is indeed an issue that should be corrected, > 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};
Thanks for this. I currently don't have access to the source repository to make this change, but next chance I get I'll fix it up.
Resolved in 0.6.