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);
     }
 
 }


Reply via email to