Author: britter Date: Sat Dec 27 21:31:56 2014 New Revision: 1648117 URL: http://svn.apache.org/r1648117 Log: Use assertArrayEquals for comparing arrays
Modified: commons/proper/imaging/trunk/src/test/java/org/apache/commons/imaging/formats/png/PngWriteReadTest.java Modified: commons/proper/imaging/trunk/src/test/java/org/apache/commons/imaging/formats/png/PngWriteReadTest.java URL: http://svn.apache.org/viewvc/commons/proper/imaging/trunk/src/test/java/org/apache/commons/imaging/formats/png/PngWriteReadTest.java?rev=1648117&r1=1648116&r2=1648117&view=diff ============================================================================== --- commons/proper/imaging/trunk/src/test/java/org/apache/commons/imaging/formats/png/PngWriteReadTest.java (original) +++ commons/proper/imaging/trunk/src/test/java/org/apache/commons/imaging/formats/png/PngWriteReadTest.java Sat Dec 27 21:31:56 2014 @@ -17,6 +17,7 @@ package org.apache.commons.imaging.formats.png; +import static org.junit.Assert.assertArrayEquals; import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertTrue; @@ -157,22 +158,7 @@ public class PngWriteReadTest extends Im assertTrue(srcImage.getHeight() == dstImage.getHeight()); final int dstData[][] = bufferedImageToImageData(dstImage); - compare(rawData, dstData); - } - - private void compare(final int[][] a, final int[][] b) { - assertNotNull(a); - assertNotNull(b); - assertTrue(a.length == b.length); - - for (int y = 0; y < a.length; y++) { - assertTrue(a[y].length == b[y].length); - // make sure row lengths consistent. - assertTrue(a[0].length == b[y].length); - for (int x = 0; x < a[y].length; x++) { - assertTrue(a[y][x] == b[y][x]); - } - } + assertArrayEquals(rawData, dstData); } }