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. --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
