Author: celestin Date: Thu Jun 7 05:12:46 2012 New Revision: 1347393 URL: http://svn.apache.org/viewvc?rev=1347393&view=rev Log: In TestUtils, improved handling of error messages for - assertEquals(String, double[], RealVector, double), - assertEquals(String, RealVector, RealVector, double).
Modified: commons/proper/math/trunk/src/test/java/org/apache/commons/math3/TestUtils.java Modified: commons/proper/math/trunk/src/test/java/org/apache/commons/math3/TestUtils.java URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/test/java/org/apache/commons/math3/TestUtils.java?rev=1347393&r1=1347392&r2=1347393&view=diff ============================================================================== --- commons/proper/math/trunk/src/test/java/org/apache/commons/math3/TestUtils.java (original) +++ commons/proper/math/trunk/src/test/java/org/apache/commons/math3/TestUtils.java Thu Jun 7 05:12:46 2012 @@ -253,10 +253,11 @@ public class TestUtils { */ public static void assertEquals(final String message, final double[] expected, final RealVector actual, final double delta) { - Assert.assertEquals(message + ", dimension", expected.length, + final String msgAndSep = message.equals("") ? "" : message + ", "; + Assert.assertEquals(msgAndSep + "dimension", expected.length, actual.getDimension()); for (int i = 0; i < expected.length; i++) { - Assert.assertEquals(message + ", entry #" + i, expected[i], + Assert.assertEquals(msgAndSep + "entry #" + i, expected[i], actual.getEntry(i), delta); } } @@ -274,11 +275,12 @@ public class TestUtils { */ public static void assertEquals(final String message, final RealVector expected, final RealVector actual, final double delta) { - Assert.assertEquals(message + ", dimension", expected.getDimension(), + final String msgAndSep = message.equals("") ? "" : message + ", "; + Assert.assertEquals(msgAndSep + "dimension", expected.getDimension(), actual.getDimension()); final int dim = expected.getDimension(); for (int i = 0; i < dim; i++) { - Assert.assertEquals(message + ", entry #" + i, + Assert.assertEquals(msgAndSep + "entry #" + i, expected.getEntry(i), actual.getEntry(i), delta); } }