On Sun, Jan 23, 2011 at 9:59 AM, Luc Maisonobe <luc.maison...@free.fr> wrote: > 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.
+1 Phil > > Luc > >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org >> For additional commands, e-mail: dev-h...@commons.apache.org >> > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
