Bug #33723 for Curses-UI: t/99template.t has hard coded exit 1

Fri Feb 29 18:37:57 2008 mschwern [...] cpan.org - Ticket created

Subject:

t/99template.t has hard coded exit 1

t/99template.t fails because it exits with status 1. #### REMOVE #### exit 1; A non-zero exit code results in a failed test. It should instead say use Test::More skip_all => "incomplete test";

Sat Mar 01 23:51:16 2008 mdxi [...] cpan.org - Correspondence added

On Fri Feb 29 18:37:57 2008, MSCHWERN wrote: Show quoted text

> t/99template.t fails because it exits with status 1. > > #### REMOVE #### > exit 1; > > A non-zero exit code results in a failed test. It should instead say > > use Test::More skip_all => "incomplete test";

Sorry; in my excavations, I hadn't dug down into C::UI's testing suite yet so I didn't catch this null test file and its odd behavior. I've gotten 3 FAILs from CPAN testers in the past 24 hours due to this, which is the really odd part (that it worked just fine until now). Version 0.9602 has been released, removing the file entirely.

Sat Mar 01 23:51:17 2008 The RT System itself - Status changed from 'new' to 'open'

Sat Mar 01 23:51:18 2008 mdxi [...] cpan.org - Status changed from 'open' to 'resolved'

Sun Mar 02 00:31:03 2008 mschwern [...] cpan.org - Correspondence added

On Sat Mar 01 23:51:16 2008, MDXI wrote: Show quoted text

> Sorry; in my excavations, I hadn't dug down into C::UI's testing suite > yet so I didn't catch this null test file and its odd behavior. I've > gotten 3 FAILs from CPAN testers in the past 24 hours due to this, which > is the really odd part (that it worked just fine until now).

Test::More just recently fixed a bug which could mask the test's real exit status. That was in 0.76_01.

Sun Mar 02 00:31:04 2008 The RT System itself - Status changed from 'resolved' to 'open'

Sun Mar 02 03:44:56 2008 mdxi [...] cpan.org - Status changed from 'open' to 'resolved'