Bug #49112 for Test-Apocalypse: Suggestion: Trick quality for Test::Pod etc dependencies

Fri Aug 28 23:48:39 2009 frequency [...] cpan.org - Ticket created

Subject:

Suggestion: Trick quality for Test::Pod etc dependencies

Hi Apocalypse: I noticed in your test suite for Test::Apocalypse, in t/a_apocalypse.t, you've got: require Test::NoWarnings; require Test::Pod; require Test::Pod::Coverage; # lousy hack for kwalitee I came across this issue myself. You can hack around the Kwalitee detection by using: 0 and require Test::NoWarnings; This way you don't actually require it (it can be used optionally via eval as per the recommended usage), though I'm not sure if you'd rather those modules always be available whenever a module tests against Test::Apocalypse. Also, you may want to differentiate between AUTOMATED_TESTING and RELEASE_TESTING (rather than simply the TEST_AUTHOR env var from the old days). The reasons why are discussed by Alias in his blog, here: http://use.perl.org/~Alias/journal/38822 Cheers, Jonathan

Fri May 14 14:31:50 2010 APOCAL [...] cpan.org - Correspondence added

Hello, Sorry it took me a while to get back to you about this. In the process of converting my dists to Dist::Zilla I decided to attack this problem. Your suggestion worked in tricking the kwalitee but I ended up with a different problem! Dist::Zilla automatically picks up your prereqs by scanning the perl code. Unfortunately, the kwalitee tests are now part of my prereqs! I found another solution to this that worked with dzil and the kwalitee tests :) require 'Test/NoWarnings.pm'; require 'Test/Pod.pm'; require 'Test/Pod/Coverage.pm'; This zany format worked, and I'll be suggesting it in an updated Test::Apocalypse that should be out *soon* heh... Thanks again for helping me with this! -- ~Apocalypse

Fri May 14 14:31:51 2010 The RT System itself - Status changed from 'new' to 'open'

Fri Jul 05 15:29:55 2013 ether [...] cpan.org - Correspondence added

On 2010-05-14 11:31:50, APOCAL wrote: Show quoted text

> Hello, > > Sorry it took me a while to get back to you about this. In the process > of converting my dists to Dist::Zilla I decided to attack this problem. > Your suggestion worked in tricking the kwalitee but I ended up with a > different problem! Dist::Zilla automatically picks up your prereqs by > scanning the perl code. > > Unfortunately, the kwalitee tests are now part of my prereqs! I found > another solution to this that worked with dzil and the kwalitee tests :) > > require 'Test/NoWarnings.pm'; require 'Test/Pod.pm'; require > 'Test/Pod/Coverage.pm'; > > This zany format worked, and I'll be suggesting it in an updated > Test::Apocalypse that should be out *soon* heh... Thanks again for > helping me with this!

You can exclude specific matches from [AutoPrereqs]. Also, if you remove [ExtraTests], your test won't be moved from xt/ to t/, so AutoPrereqs won't even see it. Plus, the pod checks are being removed from kwalitee soon anyway. :)

Fri Oct 24 18:58:26 2014 APOCAL [...] cpan.org - Correspondence added

Hello, Thanks for prodding me again! I looked at MCA and you're right that it's been updated to remove the controversial pod stuff. That means I don't need to do this anymore :) I've ripped out the code that did this and will release a new version soon. On Fri Jul 05 15:29:55 2013, ETHER wrote: Show quoted text

> On 2010-05-14 11:31:50, APOCAL wrote:

> > Hello, > > > > Sorry it took me a while to get back to you about this. In the > > process > > of converting my dists to Dist::Zilla I decided to attack this > > problem. > > Your suggestion worked in tricking the kwalitee but I ended up with a > > different problem! Dist::Zilla automatically picks up your prereqs by > > scanning the perl code. > > > > Unfortunately, the kwalitee tests are now part of my prereqs! I found > > another solution to this that worked with dzil and the kwalitee tests > > :) > > > > require 'Test/NoWarnings.pm'; require 'Test/Pod.pm'; require > > 'Test/Pod/Coverage.pm'; > > > > This zany format worked, and I'll be suggesting it in an updated > > Test::Apocalypse that should be out *soon* heh... Thanks again for > > helping me with this!

> > > You can exclude specific matches from [AutoPrereqs]. Also, if you > remove [ExtraTests], your test won't be moved from xt/ to t/, so > AutoPrereqs won't even see it. > > Plus, the pod checks are being removed from kwalitee soon anyway. :)

-- ~Apocalypse

Fri Oct 24 18:58:28 2014 APOCAL [...] cpan.org - Status changed from 'open' to 'resolved'