Bug #105637 for Error-Pure: Test::Pod, Test::Pod::Coverage should not be in test-requires

Thu Jul 02 17:44:30 2015 ether [...] cpan.org - Ticket created

Subject:

Test::Pod, Test::Pod::Coverage should not be in test-requires Download (untitled) / with headers text/plain 280b

These modules are only used by author tests; it should not be listed in TEST_REQUIRES. It looks like those pod tests can simply be moved to xt/, which will also make cpants happier (it will complain about undeclared prereqs used by t/ files, not understanding the skip_all bit.)

Thu Jul 02 17:52:29 2015 ether [...] cpan.org - Subject changed from 'Test::Pod, Test::Pod::Coverage should not be in test-requires Download (untitled) / with headers text/plain 280b' to 'Test::Pod, Test::Pod::Coverage should not be in test-requires'

Tue Sep 15 16:00:07 2015 SKIM [...] cpan.org - Correspondence added

Dne Čt 02.čec.2015 17:44:30, ETHER napsal(a): Hi, Show quoted text

> These modules are only used by author tests; it should not be listed > in TEST_REQUIRES. > > It looks like those pod tests can simply be moved to xt/, which will > also make cpants happier (it will complain about undeclared prereqs > used by t/ files, not understanding the skip_all bit.)

I am thinking about it. But i don't know how to create it now. My concept of tests is simple: Each class/module has own test directory. Then i can create xt/ with this structure. But i don't see concept for test it by Module::Install, which i am using :-) btw: Is xt/ directory concept described somewhere? I am a bit confused :-) Kind regards, Michal Špaček

Tue Sep 15 16:00:07 2015 The RT System itself - Status changed from 'new' to 'open'

Tue Sep 15 17:57:30 2015 ether [...] cpan.org - Correspondence added

On 2015-09-15 13:00:07, SKIM wrote: Show quoted text

> Dne Čt 02.čec.2015 17:44:30, ETHER napsal(a): > Hi, >

> > These modules are only used by author tests; it should not be listed > > in TEST_REQUIRES. > > > > It looks like those pod tests can simply be moved to xt/, which will > > also make cpants happier (it will complain about undeclared prereqs > > used by t/ files, not understanding the skip_all bit.)

> > I am thinking about it. But i don't know how to create it now. > My concept of tests is simple: > Each class/module has own test directory. Then i can create xt/ with > this structure. > But i don't see concept for test it by Module::Install, which i am > using :-) > > btw: Is xt/ directory concept described somewhere? I am a bit confused > :-)

I think this is what you're looking for: https://metacpan.org/pod/Module::Install::AuthorTests

Tue Sep 15 18:58:03 2015 SKIM [...] cpan.org - Correspondence added

Show quoted text

> Dne Čt 02.čec.2015 17:44:30, ETHER napsal(a): > I think this is what you're looking for: > https://metacpan.org/pod/Module::Install::AuthorTests

Ohh. Thanks. I released 0.23 version. :-) Kind Regards, M.

Tue Sep 15 19:21:21 2015 SKIM [...] cpan.org - Correspondence added

Dne Út 15.zář.2015 18:58:03, SKIM napsal(a): Show quoted text

> > Dne Čt 02.čec.2015 17:44:30, ETHER napsal(a): > > I think this is what you're looking for: > > https://metacpan.org/pod/Module::Install::AuthorTests

> > Ohh. Thanks. > I released 0.23 version. :-)

Hm, one problem is 'test_requires'. That's bad. M.

Tue Sep 15 20:16:11 2015 ether [...] cpan.org - Correspondence added

On 2015-09-15 16:21:21, SKIM wrote: Show quoted text

> Dne Út 15.zář.2015 18:58:03, SKIM napsal(a):

> > > Dne Čt 02.čec.2015 17:44:30, ETHER napsal(a): > > > I think this is what you're looking for: > > > https://metacpan.org/pod/Module::Install::AuthorTests

> > > > Ohh. Thanks. > > I released 0.23 version. :-)

> > Hm, one problem is 'test_requires'. That's bad.

https://metacpan.org/pod/Module::Install::AuthorRequires :)

Wed Sep 16 03:58:16 2015 SKIM [...] cpan.org - Correspondence added

Dne Út 15.zář.2015 20:16:11, ETHER napsal(a): Show quoted text

> https://metacpan.org/pod/Module::Install::AuthorRequires :)

Nice, thanks :-) Released in 0.24 version. M.

Wed Sep 16 13:57:33 2015 SKIM [...] cpan.org - Status changed from 'open' to 'resolved'

Wed Sep 16 13:57:34 2015 SKIM [...] cpan.org - Taken

Wed Sep 16 13:57:34 2015 SKIM [...] cpan.org - Fixed in 0.24 added