commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gary Gregory <GGreg...@seagullsoftware.com>
Subject RE: svn commit: r1022080 - /commons/proper/pool/trunk/src/test/org/apache/commons/pool/impl/TestCursorableLinkedList.java
Date Wed, 13 Oct 2010 16:06:29 GMT
+1 to JUnit 4! Welcome to the 21st century [IO]!

Gary Gregory
Senior Software Engineer
Rocket Software
3340 Peachtree Road, Suite 820 • Atlanta, GA 30326 • USA
Tel: +1.404.760.1560
Email: ggregory@seagullsoftware.com
Web: seagull.rocketsoftware.com  



> -----Original Message-----
> From: Simone Tripodi [mailto:simone.tripodi@gmail.com]
> Sent: Wednesday, October 13, 2010 08:33
> To: Commons Developers List
> Subject: Re: svn commit: r1022080 -
> /commons/proper/pool/trunk/src/test/org/apache/commons/pool/impl/TestCursorabl
> eLinkedList.java
> 
> Thanks for the feedback, I'm already in the process to convert them,
> previewing that junit4.x would have been accepted :P
> best,
> Simo
> 
> http://people.apache.org/~simonetripodi/
> http://www.99soft.org/
> 
> 
> 
> On Wed, Oct 13, 2010 at 5:17 PM, sebb <sebbaz@gmail.com> wrote:
> > On 13 October 2010 15:36, Simone Tripodi <simone.tripodi@gmail.com> wrote:
> >> Hi all guys,
> >> Pool 1.X was using junit 3.8.1 for unit tests because of the Java1.3
> >> retro compatibility, this test broke the build because junit 4.X is
> >> missing in the classpath, so I'd take advantage and migrate to junit
> >> 4, any abjection? just let me know,
> >
> > Oops! Done it again. I did test in Eclipse, but should have tested
> > using Maven as well.
> >
> > I agree - no need to keep the dependency on JUnit 3.8.1.
> >
> > JUnit 4.x still supports JUnit 3 tests (but not in the same class)
> >
> > It's not trivial to migrate existing tests to JUnit 4 (some will need
> > splitting), but I don't have any objections if you want to convert
> > them.
> >
> >> Simo
> >>
> >> http://people.apache.org/~simonetripodi/
> >> http://www.99soft.org/
> >>
> >>
> >>
> >> On Wed, Oct 13, 2010 at 2:46 PM,  <sebb@apache.org> wrote:
> >>> Author: sebb
> >>> Date: Wed Oct 13 12:46:01 2010
> >>> New Revision: 1022080
> >>>
> >>> URL: http://svn.apache.org/viewvc?rev=1022080&view=rev
> >>> Log:
> >>> Add basic tests for CursorableLinkedList
> >>>
> >>> Added:
> >
> > ---------------------------------------------------------------------
> > 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

Mime
View raw message