You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by tv...@apache.org on 2014/04/02 14:26:57 UTC

svn commit: r1583995 - in /commons/proper/jcs/trunk/src/test/org/apache/commons/jcs/auxiliary: lateral/socket/tcp/LateralTCPIssueRemoveOnPutUnitTest.java remote/RemoteUtilsUnitTest.java

Author: tv
Date: Wed Apr  2 12:26:57 2014
New Revision: 1583995

URL: http://svn.apache.org/r1583995
Log:
Made tests less chatty

Modified:
    commons/proper/jcs/trunk/src/test/org/apache/commons/jcs/auxiliary/lateral/socket/tcp/LateralTCPIssueRemoveOnPutUnitTest.java
    commons/proper/jcs/trunk/src/test/org/apache/commons/jcs/auxiliary/remote/RemoteUtilsUnitTest.java

Modified: commons/proper/jcs/trunk/src/test/org/apache/commons/jcs/auxiliary/lateral/socket/tcp/LateralTCPIssueRemoveOnPutUnitTest.java
URL: http://svn.apache.org/viewvc/commons/proper/jcs/trunk/src/test/org/apache/commons/jcs/auxiliary/lateral/socket/tcp/LateralTCPIssueRemoveOnPutUnitTest.java?rev=1583995&r1=1583994&r2=1583995&view=diff
==============================================================================
--- commons/proper/jcs/trunk/src/test/org/apache/commons/jcs/auxiliary/lateral/socket/tcp/LateralTCPIssueRemoveOnPutUnitTest.java (original)
+++ commons/proper/jcs/trunk/src/test/org/apache/commons/jcs/auxiliary/lateral/socket/tcp/LateralTCPIssueRemoveOnPutUnitTest.java Wed Apr  2 12:26:57 2014
@@ -36,7 +36,7 @@ public class LateralTCPIssueRemoveOnPutU
     extends TestCase
 {
     /** Should log data go to system out. */
-    private static boolean isSysOut = true;
+    private static boolean isSysOut = false;
 
     /** The port the server will listen to. */
     private final int serverPort = 1118;
@@ -171,7 +171,7 @@ public class LateralTCPIssueRemoveOnPutU
                     p( "put " + key );
                 }
 
-                if ( i % 100 == 0 )
+                if (show && i % 100 == 0 )
                 {
                     System.out.println( cache.getStats() );
                 }

Modified: commons/proper/jcs/trunk/src/test/org/apache/commons/jcs/auxiliary/remote/RemoteUtilsUnitTest.java
URL: http://svn.apache.org/viewvc/commons/proper/jcs/trunk/src/test/org/apache/commons/jcs/auxiliary/remote/RemoteUtilsUnitTest.java?rev=1583995&r1=1583994&r2=1583995&view=diff
==============================================================================
--- commons/proper/jcs/trunk/src/test/org/apache/commons/jcs/auxiliary/remote/RemoteUtilsUnitTest.java (original)
+++ commons/proper/jcs/trunk/src/test/org/apache/commons/jcs/auxiliary/remote/RemoteUtilsUnitTest.java Wed Apr  2 12:26:57 2014
@@ -44,19 +44,17 @@ public class RemoteUtilsUnitTest
         }
         catch ( RemoteException e )
         {
-            // TODO Auto-generated catch block
-            e.printStackTrace();
+            fail("RemoteException unexpected " + e);
         }
 
         try
         {
             RemoteUtils.createRegistry( 1102 );
+            fail("Second RemoteException expected");
         }
         catch ( Exception e )
         {
-            // TODO Auto-generated catch block
-            e.printStackTrace();
+            // Expected
         }
-
     }
 }