You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by ng...@apache.org on 2008/11/08 06:50:03 UTC

svn commit: r712353 - /mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/clienttests/DataTransferTimeoutTest.java

Author: ngn
Date: Fri Nov  7 21:50:03 2008
New Revision: 712353

URL: http://svn.apache.org/viewvc?rev=712353&view=rev
Log:
Test still failing in the CI build, adding a check

Modified:
    mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/clienttests/DataTransferTimeoutTest.java

Modified: mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/clienttests/DataTransferTimeoutTest.java
URL: http://svn.apache.org/viewvc/mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/clienttests/DataTransferTimeoutTest.java?rev=712353&r1=712352&r2=712353&view=diff
==============================================================================
--- mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/clienttests/DataTransferTimeoutTest.java (original)
+++ mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/clienttests/DataTransferTimeoutTest.java Fri Nov  7 21:50:03 2008
@@ -96,11 +96,16 @@
         TestUtil.writeDataToFile(TEST_FILE, testData);
         InputStream is = client.retrieveFileStream(TEST_FILENAME);
 
-        // read one buffer size at a time, trying to trigger IODataConnection to update
+        // read ten buffer sizes at a time, trying to trigger IODataConnection to update
         // the session timeout for each read
         for(int i = 0; i<100; i++) {
+            long startTime = System.currentTimeMillis();
             Thread.sleep(20);
-            is.read(buffer);   
+            is.read(buffer);
+            
+            if((System.currentTimeMillis() - startTime) > 500 ) {
+                fail("Read took to long, test not safe");
+            }
         }
         
         is.close();