You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2009/05/31 19:33:46 UTC

svn commit: r780480 - /commons/proper/pool/trunk/src/test/org/apache/commons/pool/impl/TestGenericKeyedObjectPool.java

Author: sebb
Date: Sun May 31 17:33:46 2009
New Revision: 780480

URL: http://svn.apache.org/viewvc?rev=780480&view=rev
Log:
Tweak failure output

Modified:
    commons/proper/pool/trunk/src/test/org/apache/commons/pool/impl/TestGenericKeyedObjectPool.java

Modified: commons/proper/pool/trunk/src/test/org/apache/commons/pool/impl/TestGenericKeyedObjectPool.java
URL: http://svn.apache.org/viewvc/commons/proper/pool/trunk/src/test/org/apache/commons/pool/impl/TestGenericKeyedObjectPool.java?rev=780480&r1=780479&r2=780480&view=diff
==============================================================================
--- commons/proper/pool/trunk/src/test/org/apache/commons/pool/impl/TestGenericKeyedObjectPool.java (original)
+++ commons/proper/pool/trunk/src/test/org/apache/commons/pool/impl/TestGenericKeyedObjectPool.java Sun May 31 17:33:46 2009
@@ -899,7 +899,7 @@
         pool.setMaxIdle(-1);
         for (int i = 0; i < smallPrimes.length; i++) {
             pool.setNumTestsPerEvictionRun(smallPrimes[i]);
-            for (int j = 0; j < 5; j++) { // TODO why 5?
+            for (int j = 0; j < 5; j++) {// Try the tests a few times
                 pool.clear();
                 int zeroLength = 10 + random.nextInt(20);
                 for (int k = 0; k < zeroLength; k++) {
@@ -1481,13 +1481,13 @@
     private String formatSettings(String s1, int i1, String s2, int i2, String s3, int i3,
             String s4, int i4, String s5, int i5, String s6, int i6, String s7, int i7){
         StringBuffer sb = new StringBuffer();
-        sb.append(s1).append(' ').append(i1).append(' ');
-        sb.append(s2).append(' ').append(i2).append(' ');
-        sb.append(s3).append(' ').append(i3).append(' ');
-        sb.append(s4).append(' ').append(i4).append(' ');
-        sb.append(s5).append(' ').append(i5).append(' ');
-        sb.append(s6).append(' ').append(i6).append(' ');
-        sb.append(s7).append(' ').append(i7).append(' ');
+        sb.append(s1).append('=').append(i1).append(' ');
+        sb.append(s2).append('=').append(i2).append(' ');
+        sb.append(s3).append('=').append(i3).append(' ');
+        sb.append(s4).append('=').append(i4).append(' ');
+        sb.append(s5).append('=').append(i5).append(' ');
+        sb.append(s6).append('=').append(i6).append(' ');
+        sb.append(s7).append('=').append(i7).append(' ');
         return sb.toString();
     }