You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pdfbox.apache.org by ti...@apache.org on 2016/01/06 18:34:51 UTC

svn commit: r1723389 - /pdfbox/branches/1.8/pdfbox/src/test/java/org/apache/pdfbox/util/TestQuickSort.java

Author: tilman
Date: Wed Jan  6 17:34:51 2016
New Revision: 1723389

URL: http://svn.apache.org/viewvc?rev=1723389&view=rev
Log:
PDFBOX-2996: fix bug that the test compared two identical arrays

Modified:
    pdfbox/branches/1.8/pdfbox/src/test/java/org/apache/pdfbox/util/TestQuickSort.java

Modified: pdfbox/branches/1.8/pdfbox/src/test/java/org/apache/pdfbox/util/TestQuickSort.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/1.8/pdfbox/src/test/java/org/apache/pdfbox/util/TestQuickSort.java?rev=1723389&r1=1723388&r2=1723389&view=diff
==============================================================================
--- pdfbox/branches/1.8/pdfbox/src/test/java/org/apache/pdfbox/util/TestQuickSort.java (original)
+++ pdfbox/branches/1.8/pdfbox/src/test/java/org/apache/pdfbox/util/TestQuickSort.java Wed Jan  6 17:34:51 2016
@@ -32,7 +32,7 @@ public class TestQuickSort extends TestC
         List<T> list = Arrays.asList(input);
         QuickSort.sort(list);
 
-        boolean equal = Arrays.equals(list.toArray(new Object[input.length]), input);
+        boolean equal = Arrays.equals(list.toArray(new Object[input.length]), expected);
 
         assertTrue(equal);
     }