Author: markt
Date: Mon Jun 25 10:24:31 2018
New Revision: 1834295

URL: http://svn.apache.org/viewvc?rev=1834295&view=rev
Log:
Improve debug messages when test fails

Modified:
    tomcat/trunk/test/org/apache/tomcat/jni/TestSocketServer.java

Modified: tomcat/trunk/test/org/apache/tomcat/jni/TestSocketServer.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/tomcat/jni/TestSocketServer.java?rev=1834295&r1=1834294&r2=1834295&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/tomcat/jni/TestSocketServer.java (original)
+++ tomcat/trunk/test/org/apache/tomcat/jni/TestSocketServer.java Mon Jun 25 
10:24:31 2018
@@ -108,9 +108,9 @@ public class TestSocketServer extends Ab
         while (Socket.recv(clientSocket, buf, 0, 1) == 1) {
         }
         long wait = System.nanoTime() - start;
-        Assert.assertFalse("Socket.timeoutSet failed (<1s) [" + wait + "]",
+        Assert.assertFalse("Socket.timeoutSet failed (<1s) [" + wait + "] +-[" 
+ ERROR_MARGIN + "]",
                 wait < 1000000000 - ERROR_MARGIN);
-        Assert.assertFalse("Socket.timeoutSet failed (>2s) [" + wait + "]",
+        Assert.assertFalse("Socket.timeoutSet failed (>2s) [" + wait + "] +-[" 
+ ERROR_MARGIN + "]",
                 wait > 2000000000 + ERROR_MARGIN);
 
         client.countDown();
@@ -137,8 +137,8 @@ public class TestSocketServer extends Ab
         while (Socket.recv(clientSocket, buf, 0, 1) == 1) {
         }
         long wait = System.nanoTime() - start;
-        Assert.assertFalse("non_blocking client Socket.APR_SO_NONBLOCK failed 
(>2ms)",
-                wait > 2000000 + ERROR_MARGIN);
+        Assert.assertFalse("non_blocking client Socket.APR_SO_NONBLOCK failed 
(>2ms) [" + wait +
+                "] +-[" + ERROR_MARGIN + "]", wait > 2000000 + ERROR_MARGIN);
 
         client.countDown();
         client.join();
@@ -162,8 +162,8 @@ public class TestSocketServer extends Ab
         while (Socket.recv(clientSocket, buf, 0, 1) == 1) {
         }
         long wait = System.nanoTime() - start;
-        Assert.assertFalse("non_blocking client Socket.APR_SO_NONBLOCK failed 
(>1ms)",
-                wait > 1000000 + ERROR_MARGIN);
+        Assert.assertFalse("non_blocking client Socket.APR_SO_NONBLOCK failed 
(>1ms) [" + wait +
+                "] +-[" + ERROR_MARGIN + "]", wait > 1000000 + ERROR_MARGIN);
 
         /* Configure for blocking */
         Socket.optSet(clientSocket, Socket.APR_SO_NONBLOCK, 0);
@@ -172,8 +172,8 @@ public class TestSocketServer extends Ab
         while (Socket.recv(clientSocket, buf, 0, 1) == 1) {
         }
         wait = System.nanoTime() - start;
-        Assert.assertFalse("non_blocking client Socket.APR_SO_NONBLOCK false 
failed",
-                wait < 1000000 - ERROR_MARGIN);
+        Assert.assertFalse("non_blocking client Socket.APR_SO_NONBLOCK false 
failed (<1ms) [" +
+                wait + "] +-[" + ERROR_MARGIN + "]", wait < 1000000 - 
ERROR_MARGIN);
 
         client.countDown();
         client.join();
@@ -195,8 +195,8 @@ public class TestSocketServer extends Ab
         }
         long wait = System.nanoTime() - start;
         Assert.assertTrue("Timeout failed", ok);
-        Assert.assertFalse("non_blocking accept Socket.APR_SO_NONBLOCK failed 
([" + wait + "]>1ms)",
-                wait > 1000000 + ERROR_MARGIN);
+        Assert.assertFalse("non_blocking accept Socket.APR_SO_NONBLOCK failed 
(>1ms) [" + wait +
+                "] +-[" + ERROR_MARGIN + "]", wait > 1000000 + ERROR_MARGIN);
     }
 
 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to