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