You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by sebb <se...@gmail.com> on 2010/01/21 20:00:03 UTC
[POOL] get rid of suite() and TestAll classes?
I'd like to get rid of the suite() methods and TestAll classes in Pool
(as I did recently for DBCP).
There don't appear to be any missing tests currently, and the Maven
and Ant builds currently run the same number of tests, but using
TestAll and suite() is more effort to maintain.
WDYT?
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
RE: [POOL] get rid of suite() and TestAll classes?
Posted by Gary Gregory <GG...@seagullsoftware.com>.
+1. Since IDEs like Eclipse let you run all tests in a package or folder I do not think anything would be lost.
Gary
> -----Original Message-----
> From: sebb [mailto:sebbaz@gmail.com]
> Sent: Thursday, January 21, 2010 11:00
> To: Commons Developers List
> Subject: [POOL] get rid of suite() and TestAll classes?
>
> I'd like to get rid of the suite() methods and TestAll classes in Pool
> (as I did recently for DBCP).
>
> There don't appear to be any missing tests currently, and the Maven
> and Ant builds currently run the same number of tests, but using
> TestAll and suite() is more effort to maintain.
>
> WDYT?
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
Re: [CODEC] Get rid of suite() and TestAll classes?
Posted by sebb <se...@gmail.com>.
On 22/01/2010, Gary Gregory <GG...@seagullsoftware.com> wrote:
> Same idea for [CODEC] as the message below RE [POOL]
+1 from me (obviously).
Note that Surefire allows individual tests to be run by using the
-Dtest=TestClassName command-line qualifier, and similar functionality
can be implemented in Ant build files - see for example DBCP and POOL.
> Gary
>
> > -----Original Message-----
> > From: Phil Steitz [mailto:phil.steitz@gmail.com]
> > Sent: Thursday, January 21, 2010 15:06
> > To: Commons Developers List
> > Subject: Re: [POOL] get rid of suite() and TestAll classes?
> >
> > sebb wrote:
> > > I'd like to get rid of the suite() methods and TestAll classes in
> > Pool
> > > (as I did recently for DBCP).
> > >
> > > There don't appear to be any missing tests currently, and the Maven
> > > and Ant builds currently run the same number of tests, but using
> > > TestAll and suite() is more effort to maintain.
> >
> > +1
> >
> > Phil
> > >
> > > WDYT?
> > >
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > > For additional commands, e-mail: dev-help@commons.apache.org
> > >
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > For additional commands, e-mail: dev-help@commons.apache.org
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
[CODEC] Get rid of suite() and TestAll classes?
Posted by Gary Gregory <GG...@seagullsoftware.com>.
Same idea for [CODEC] as the message below RE [POOL]
Gary
> -----Original Message-----
> From: Phil Steitz [mailto:phil.steitz@gmail.com]
> Sent: Thursday, January 21, 2010 15:06
> To: Commons Developers List
> Subject: Re: [POOL] get rid of suite() and TestAll classes?
>
> sebb wrote:
> > I'd like to get rid of the suite() methods and TestAll classes in
> Pool
> > (as I did recently for DBCP).
> >
> > There don't appear to be any missing tests currently, and the Maven
> > and Ant builds currently run the same number of tests, but using
> > TestAll and suite() is more effort to maintain.
>
> +1
>
> Phil
> >
> > WDYT?
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> > For additional commands, e-mail: dev-help@commons.apache.org
> >
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
Re: [POOL] get rid of suite() and TestAll classes?
Posted by Phil Steitz <ph...@gmail.com>.
sebb wrote:
> I'd like to get rid of the suite() methods and TestAll classes in Pool
> (as I did recently for DBCP).
>
> There don't appear to be any missing tests currently, and the Maven
> and Ant builds currently run the same number of tests, but using
> TestAll and suite() is more effort to maintain.
+1
Phil
>
> WDYT?
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org
Re: [POOL] get rid of suite() and TestAll classes?
Posted by Jörg Schaible <jo...@gmx.de>.
sebb wrote at Donnerstag, 21. Januar 2010 20:00:
> I'd like to get rid of the suite() methods and TestAll classes in Pool
> (as I did recently for DBCP).
>
> There don't appear to be any missing tests currently, and the Maven
> and Ant builds currently run the same number of tests, but using
> TestAll and suite() is more effort to maintain.
>
> WDYT?
+1
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org