Skip Menu |
 

This queue is for tickets about the Module-Build CPAN distribution.

Report information
The Basics
Id: 1624
Status: resolved
Priority: 0/
Queue: Module-Build

People
Owner: Nobody in particular
Requestors: spoon [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: 0.01
Fixed in: 0.12



Date: Tue, 8 Oct 2002 00:34:26 +1000
From: Iain Truskett <spoon [...] cpan.org>
To: bug-module-build [...] rt.cpan.org
Subject: Test order
Download (untitled) / with headers
text/plain 869b
Heyo, Minor patch. I actually like having my tests in a particular order, thus an addition of 'sort' to the runtests call is a good thing imho. I suspect a proper patch really should have included some method for the author to provide their own sorting (à la Test::Manifest), but this does it for me =) Also, I'm yet to figure out how to get $self->{properties}{verbose} to be true (for verbose tests). --- Base.pm.orig Fri Aug 23 18:45:11 2002 +++ Base.pm.new Mon Oct 7 23:00:46 2002 @@ -581,7 +581,7 @@ if (@tests) { # Work around a Test::Harness bug that loses the particular perl we're running under local $^X = $self->{config}{perlpath} unless $Test::Harness::VERSION gt '2.01'; - Test::Harness::runtests(@tests); + Test::Harness::runtests(sort @tests); } else { print("No tests defined.\n"); } cheers, -- Iain.
Date: Wed, 16 Oct 2002 03:30:23 +1000
Subject: Re: [cpan #1624] Test order
From: Ken Williams <ken [...] mathforum.org>
To: bug-Module-Build [...] rt.cpan.org
RT-Send-Cc:
Download (untitled) / with headers
text/plain 1.2k
I agree with this whole synopsis, including patch and caveats. Applied. -Ken On Tuesday, October 8, 2002, at 12:36 AM, via RT wrote: Show quoted text
> > > This message about Module-Build was sent to you by SPOON via rt.cpan.org > > Full context and any attached attachments can be found at: > <URL: https://rt.cpan.org/Ticket/Display.html?id=1624 > > > Heyo, > > Minor patch. I actually like having my tests in a particular order, thus > an addition of 'sort' to the runtests call is a good thing imho. I > suspect a proper patch really should have included some method for > the author to provide their own sorting (‡ la Test::Manifest), but this > does it for me =) > > Also, I'm yet to figure out how to get $self->{properties}{verbose} to > be true (for verbose tests). > > > --- Base.pm.orig Fri Aug 23 18:45:11 2002 > +++ Base.pm.new Mon Oct 7 23:00:46 2002 > @@ -581,7 +581,7 @@ > if (@tests) { > # Work around a Test::Harness bug that loses the particular perl > we're running under > local $^X = $self->{config}{perlpath} unless > $Test::Harness::VERSION gt '2.01'; > - Test::Harness::runtests(@tests); > + Test::Harness::runtests(sort @tests); > } else { > print("No tests defined.\n"); > } > > > cheers, > -- > Iain. >


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.