commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Luc Maisonobe <Luc.Maison...@free.fr>
Subject Re: [MATH] FastMathTest - split into 2 or 3 classes?
Date Sun, 23 Jan 2011 14:59:09 GMT
Hi Sebb,

Le 23/01/2011 15:51, sebb a écrit :
> I think it would be useful to split the FastMathTest class into 2 or 3 classes.
> 
> FastMathTest - individual method tests
> FastMathPerformanceTest - the performance test
> FastMathGenericTest - tests to compare against all StrictMath methods
> (the current reflection-based tests)
> 
> I think it would definitely be an improvement to put the performance
> tests in a separate class.
> At present the tests can only be run by editting the source (to remove
> the @Ignore annotation).
> This could be removed, and the test ignored in the Surefire
> configuration instead.
> One could still run the test independently using -Dtest=FastMathPerformanceTest.
> 
> I'm not so sure about splitting off the reflection tests.
> However the current class is getting quite big, and I want to use a
> suite to create individual tests for each method.
> So it would probably be simpler to have a separate class.

This is a very good idea.

Luc

> 
> ---------------------------------------------------------------------
> 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