Skip Menu |

This queue is for tickets about the Params-Validate CPAN distribution.

Report information
The Basics
Id: 113063
Status: resolved
Priority: 0/
Queue: Params-Validate
People
Owner: Nobody in particular
Requestors: ether [...] cpan.org
Cc: KENTNL [...] cpan.org
AdminCc:
Bug Information
Severity: (no value)
Broken in: 1.24
Fixed in: (no value)

History Show all quoted text
  Tue Mar 15 19:44:30 2016 ether [...] cpan.org - Ticket created
Subject: avoid [ExtraTests]?
Would it be possible for you to remove [ExtraTests], either in this distribution or in your plugin bundle? Due to the number of pure perl xt/ tests in this distribution, there's a lot of overhead of t/release-* tests being loaded and then skipped, which adds noise to the build output. In my bundle I use [RunExtraTests] instead, which DTRT: adds xt/<subdir> to the list of test dirs for 'dzil test' and when using [TestRelease].
  Tue Mar 15 19:47:54 2016 KENTNL [...] cpan.org - Cc KENTNL added
  Tue Mar 15 19:50:48 2016 DROLSKY [...] cpan.org - Correspondence added
On Tue Mar 15 19:44:30 2016, ETHER wrote: Show quoted text
> Would it be possible for you to remove [ExtraTests], either in this > distribution or in your plugin bundle? Due to the number of pure perl > xt/ tests in this distribution, there's a lot of overhead of > t/release-* tests being loaded and then skipped, which adds noise to > the build output. > > In my bundle I use [RunExtraTests] instead, which DTRT: adds > xt/<subdir> to the list of test dirs for 'dzil test' and when using > [TestRelease].
I tried replacing [ExtraTests] with [RunExtraTests] but that doesn't do anything. I think I'd need to update my [PurePerlTests] plugin to write the tests to xt/release instead of as t/release-pp-*.t, right?
  Tue Mar 15 19:50:48 2016 The RT System itself - Status changed from 'new' to 'open'
  Tue Mar 15 20:55:10 2016 ether [...] cpan.org - Correspondence added
On 2016-03-15 16:50:48, DROLSKY wrote: Show quoted text
> I tried replacing [ExtraTests] with [RunExtraTests] but that doesn't > do anything. I think I'd need to update my [PurePerlTests] plugin to > write the tests to xt/release instead of as t/release-pp-*.t, right?
Ah, yes, I was assuming that you were creating the files in xt/release and letting [ExtraTests] move them for you (it also adds the skip_all unless $ENV{RELEASE_TESTING} header, I guess you were adding that manually too)? Since this is no longer a trivial change, I'll PR it up for you or bribe someone else to do so :)
  Sat Mar 19 21:02:08 2016 ether [...] cpan.org - Dependency on ticket #113187 added
  Sun May 08 18:49:45 2016 DROLSKY [...] cpan.org - Status changed from 'open' to 'resolved'
  Sun May 08 18:49:45 2016 DROLSKY [...] cpan.org - Broken in 1.24 added