Bug #107821 for Math-Random-MT-Perl: Kwalitee tests fail if build directory ends with "t"

Fri Oct 16 02:22:47 2015 SREZIC [...] cpan.org - Ticket created

Subject:

Kwalitee tests fail if build directory ends with "t"

CPAN.pm creates a random dist build directory like Math-Random-MT-Perl-1.13-iiFFih. If this random directory ends with "t", then this code line in t/kwalitee.t is true: if ( cwd() =~ m/t$/ ) { and subsequently a chdir to the global build directory is done, and all modules there are kwalitee-tested. This may explain some of the fail reports seen at http://matrix.cpantesters.org/?dist=Math-Random-MT-Perl%201.13

Fri Oct 16 12:27:28 2015 ether [...] cpan.org - Correspondence added

On 2015-10-15 23:22:47, SREZIC wrote: Show quoted text

> CPAN.pm creates a random dist build directory like Math-Random-MT- > Perl-1.13-iiFFih. If this random directory ends with "t", then this > code line in t/kwalitee.t is true: > > if ( cwd() =~ m/t$/ ) { > > and subsequently a chdir to the global build directory is done, and > all modules there are kwalitee-tested. > > This may explain some of the fail reports seen at > http://matrix.cpantesters.org/?dist=Math-Random-MT-Perl%201.13

Please don't allow pod or kwalitee tests to run for normal user installs, as the failure can be detected on the author side just as well, and this failure does not impair the normal functioning of the code. Either move the 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 Oct 16 12:27:29 2015 The RT System itself - Status changed from 'new' to 'open'

Mon Oct 19 15:21:54 2015 florent.angly [...] gmail.com - Correspondence added

Thanks for the report. Things should be as you both suggested in version 1.14, which I just uploaded. Cheers, Florent

Mon Oct 19 15:22:01 2015 florent.angly [...] gmail.com - Status changed from 'open' to 'resolved'