Bug #49646 for Getopt-Complete: t/pod.t is for authors

Sat Sep 12 09:56:02 2009 noreply - Ticket created

Subject:

t/pod.t is for authors

Pod tests are really only for authors(you), something you should run before releasing a module (make dist). Getopt-Complete-v0.10/t/pod.t should look like use strict; use warnings; use Test::More; if( $ENV{TEST_POD} or $ENV{TEST_ALL} or $ENV{AUTHOR} ){ eval "use Test::Pod 1.18"; plan skip_all => 'Test::Pod 1.18 required' if $@; all_pod_files_ok(); } else { plan skip_all => 'set TEST_POD or TEST_ALL to enable this test'; } Thank you

Thu Sep 17 14:23:19 2009 SAKOHT [...] cpan.org - Correspondence added

Thanks! Sorry about that. I'm putting out an update later today, and I'll add that in. On Sat Sep 12 09:56:02 2009, noreply wrote: Show quoted text

> Pod tests are really only for authors(you), something you should run > before releasing a module (make dist). > Getopt-Complete-v0.10/t/pod.t should look like > > use strict; > use warnings; > use Test::More; > > if( $ENV{TEST_POD} or $ENV{TEST_ALL} or $ENV{AUTHOR} ){ > eval "use Test::Pod 1.18"; > plan skip_all => 'Test::Pod 1.18 required' if $@; > all_pod_files_ok(); > } else { > plan skip_all => 'set TEST_POD or TEST_ALL to enable this test'; > } > > > > Thank you

Thu Sep 17 14:23:20 2009 The RT System itself - Status changed from 'new' to 'open'

Thu Sep 17 14:23:25 2009 SAKOHT [...] cpan.org - Taken

Fri Oct 23 04:42:21 2009 noreply - Correspondence added

I learned from http://search.cpan.org/dist/Test-XT/lib/Test/XT.pm http://use.perl.org/~Alias/journal/38822 that the reccomended variables are $ENV{AUTOMATED_TESTING} and $ENV{RELEASE_TESTING} and that you don't even have to write the tests yourself :) use Test::XT qw(WriteXT); # Write some specific tests: WriteXT( 'Test::Pod' => 't/pod.t', 'Test::CPAN::Meta' => 't/meta.t', 'Test::MinimumVersion' => 't/minimumversion.t', 'Test::Perl::Critic' => 't/critic.t', );

Sun Nov 01 13:26:49 2009 SAKOHT [...] cpan.org - Correspondence added

Sorry for the slow response. I've just uploaded v0.11 which eliminates the problem. Thanks much for the reminder this was still happening! Scott On Fri Oct 23 04:42:21 2009, noreply wrote: Show quoted text

> I learned from > http://search.cpan.org/dist/Test-XT/lib/Test/XT.pm > http://use.perl.org/~Alias/journal/38822 > > that the reccomended variables are > $ENV{AUTOMATED_TESTING} and $ENV{RELEASE_TESTING} > and that you don't even have to write the tests yourself :) > > use Test::XT qw(WriteXT); > > # Write some specific tests: > WriteXT( > 'Test::Pod' => 't/pod.t', > 'Test::CPAN::Meta' => 't/meta.t', > 'Test::MinimumVersion' => 't/minimumversion.t', > 'Test::Perl::Critic' => 't/critic.t', > ); > >

Sun Nov 01 13:27:10 2009 SAKOHT [...] cpan.org - Correspondence added

I removed the POD test from the distribution.

Sun Nov 01 13:27:11 2009 SAKOHT [...] cpan.org - Status changed from 'open' to 'resolved'