Bug #113187 for Dist-Zilla-Plugin-PurePerlTests: allow putting the new tests in xt/

Sat Mar 19 21:01:49 2016 ether [...] cpan.org - Ticket created

CC:	KENTNL [...] cpan.org
Subject:	allow putting the new tests in xt/

This ticket describes the work needed to fulfill https://rt.cpan.org/Ticket/Display.html?id=113063. - add a config option to specify the target directory of the new files: - t - xt/release - add a config option for whether to add the "skip_all unless RELEASE_TESTING" preamble: default value can be yes if target dir is t, no if target dir is anything inside xt/.

Sat Mar 19 21:02:08 2016 ether [...] cpan.org - Dependency by ticket #113063 added

Sat Mar 26 12:31:07 2016 DROLSKY [...] cpan.org - Correspondence added

On Sat Mar 19 21:01:49 2016, ETHER wrote: Show quoted text

> This ticket describes the work needed to fulfill > https://rt.cpan.org/Ticket/Display.html?id=113063. > > - add a config option to specify the target directory of the new > files: > - t > - xt/release > > - add a config option for whether to add the "skip_all unless > RELEASE_TESTING" preamble: > default value can be yes if target dir is t, no if target dir is > anything inside xt/.

I implemented this, more or less. The new version puts tests under "xt/author" and does not add its own RELEASE_TESTING preamble. I'm not sure there's any reason to make this configurable. I'm the only user of this plugin, AFAICT.

Sat Mar 26 12:31:08 2016 The RT System itself - Status changed from 'new' to 'open'

Sat Mar 26 14:01:50 2016 DROLSKY [...] cpan.org - Status changed from 'open' to 'resolved'

Sat Mar 26 18:55:07 2016 ether [...] cpan.org - Correspondence added

RT-Send-CC:

KENTNL [...] cpan.org

On 2016-03-26 09:31:07, DROLSKY wrote: Show quoted text

> I'm not sure there's any reason to make this configurable. I'm the > only user of this plugin, AFAICT.

Sounds good to me! [ExtraTests] can do the preamble addition, if desired, so there's no need to reimplement it here. thanks!