Skip Menu |

This queue is for tickets about the Dist-Zilla-Plugin-Test-EOL CPAN distribution.

Report information
The Basics
Id: 116307
Status: rejected
Priority: 0/
Queue: Dist-Zilla-Plugin-Test-EOL

People
Owner: Nobody in particular
Requestors: jens [...] jebecs.de (daily)
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in: 0.18
Fixed in: (no value)



Subject: Missing runtime requirement Test::EOL
If I understood the whole dependency thing correctly then Test::EOL must to be a runtime requirement in this module. What I do: 1. I install Dist::Zilla::Plugin::Test::EOL via cpanm 2. I add [Test::EOL] to my dist.ini 3. I run "dzil test" What happens: complains because Test::EOL is not installed
On 2016-07-18 16:21:32, MAXHQ wrote: Show quoted text
> If I understood the whole dependency thing correctly then Test::EOL > must to be a runtime requirement in this module. > > What I do: > 1. I install Dist::Zilla::Plugin::Test::EOL via cpanm > 2. I add [Test::EOL] to my dist.ini > 3. I run "dzil test" > > What happens: complains because Test::EOL is not installed
A prereq is injected into the 'develop' phase, which is where it belongs, because the test is generated as xt/author/eol.t. This test is not run via 'dzil test'. It sounds like you are using the [ExtraTests] plugin, which moves tests from xt/ to t/. But it is still not appropriate to add Test::EOL as a test prerequisite, because [ExtraTests] adds guard code to all the files that it removes, to check for various *_TESTING environment variables, which are not set by normal users performing an installation. The bottom line is that if you, the author, want to run author tests, you should satisfy develop prereqs first.