Skip Menu |
 

This queue is for tickets about the Test-Command CPAN distribution.

Report information
The Basics
Id: 43541
Status: open
Priority: 0/
Queue: Test-Command

People
Owner: danboo [...] cpan.org
Requestors: lanny [...] cisco.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Two requests for enhancement
Date: Mon, 23 Feb 2009 11:36:06 -0600
To: bug-Test-Command [...] rt.cpan.org
From: Lanny Ripple <lanny [...] cisco.com>
Download (untitled) / with headers
text/plain 954b
Hi Daniel, Thanks for writing Test::Command. I've written a module that uses it and it saved me a lot of time. (I never would have gotten exec and saving stdout, stderr as cleanly.) I have two requests for enhancement. The first is that you add a :internals export to expose your internals. Test::Command does not subclass well (you explicitly test $cmd refs as =~ 'Test::Command') and the internals are useful in their own right. I use full naming to get to them which works well enough but my code would look a bit cleaner without the full names. (Ok. This is a pretty low priority request.) The other request is that you make available the low order bit information currently thrown away from the system call. I have a long series of tests via Test::Class that uses Test::Command. If I want to quit the test run and type ^C the I-want-to-stop information is thrown away by Test::Command. Thanks again for such a useful module, -ljr
Download (untitled) / with headers
text/plain 188b
I've just uploaded 0.03 which adds signal support very much in the same way as exit is handled. I will have a look at the base class issues as well within a few days. Thanks, - danboo
Download (untitled) / with headers
text/plain 105b
In 0.07 I've clean up subclassing, and added a brief test. Let me know if you still experience problems.
Subject: Re: [rt.cpan.org #43541] Two requests for enhancement
Date: Tue, 12 May 2009 09:36:43 -0500
To: bug-Test-Command [...] rt.cpan.org
From: Lanny Ripple <lanny [...] cisco.com>
Download (untitled) / with headers
text/plain 229b
Thanks, Daniel. -ljr Daniel B. Boorstein via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=43541 > > > In 0.07 I've clean up subclassing, and added a brief test. Let me know > if you still experience problems.


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.