This queue is for tickets about the TermReadKey CPAN distribution.

Report information
The Basics
Id:
25396
Status:
resolved
Priority:
Low/Low
Queue:

People
Owner:
jns [...] gellyfish.co.uk
Requestors:
CHORNY [...] cpan.org
Cc:
AdminCc:

BugTracker
Severity:
Wishlist
Broken in:
2.30
Fixed in:
(no value)



Subject: test.pl
Tests from test.pl are not properly reported to perl.cpan.testers. Can you move tests from test.pl to t/*.t ? -- Alexandr Ciornii, http://chorny.net
Surely this is a bug in something else then rather than in Term::ReadKey (and the other couple of thousand modules that use a test.pl) you probably want to determine where exactly that bug lies and make a proper report. I'm disinclined to alter a module simply to work around a bug in someone else's tools.
test.pl requires different way to report that some tests failed - setting exit status. -- Alexandr Ciornii, http://chorny.net
On Tue Mar 13 05:25:59 2007, JSTOWE wrote:
Show quoted text
> Surely this is a bug in something else then rather than in Term::ReadKey

This bug is legit.  The output of test.pl is not parsed and the only measure is its exit code.  This has been since way back, there is no specification for the output of test.pl.  Your test.pl is a perfect example, it outputs lots of things which are not TAP.  DBI is an example which uses t/*.t for tests and test.pl to output some benchmarking.

Other distributions which use test.pl are in the same boat.  If they use Test::More it takes care of setting the exit code on failure.

I would suggest moving the beginning of test.pl which uses TAP up to "exit(0) unless $interactive;" into t/basic.t or something.  The rest can remain in test.pl.
On Thu Apr 04 13:10:48 2013, MSCHWERN wrote:
Show quoted text
> On Tue Mar 13 05:25:59 2007, JSTOWE wrote:
> > Surely this is a bug in something else then rather than in > > Term::ReadKey
> > This bug is legit. The output of test.pl is not parsed and the only > measure is > its exit code. This has been since way back, there is no specification > for the > output of test.pl. Your test.pl is a perfect example, it outputs lots > of things > which are not TAP. DBI is an example which uses t/*.t for tests and > test.pl to > output some benchmarking. > > Other distributions which use test.pl are in the same boat. If they > use > Test::More it takes care of setting the exit code on failure. > > I would suggest moving the beginning of test.pl which uses TAP up to > "exit(0) > unless $interactive;" into t/basic.t or something. The rest can remain > in > test.pl.
Okay, Having thought about it for six years I've decided to do just that, after the next release I'll bump the requirements to use Test::More and leave behind anything before 5.6.2. The next release will just cover easily fixed issues :)


This service runs on Request Tracker, is sponsored by The Perl Foundation, and maintained by Best Practical Solutions.

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