You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2020/04/15 11:05:07 UTC

[tomcat] branch 7.0.x updated: Additional logging to help debug CI failure

This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch 7.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/7.0.x by this push:
     new 4cc93c0  Additional logging to help debug CI failure
4cc93c0 is described below

commit 4cc93c0ce6c0da6cb68289f0a83e94c533b408b3
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Wed Apr 15 12:03:04 2020 +0100

    Additional logging to help debug CI failure
---
 .../websocket/TestWsWebSocketContainerGetOpenSessions.java   | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/test/org/apache/tomcat/websocket/TestWsWebSocketContainerGetOpenSessions.java b/test/org/apache/tomcat/websocket/TestWsWebSocketContainerGetOpenSessions.java
index e9af34f..68a1698 100644
--- a/test/org/apache/tomcat/websocket/TestWsWebSocketContainerGetOpenSessions.java
+++ b/test/org/apache/tomcat/websocket/TestWsWebSocketContainerGetOpenSessions.java
@@ -193,11 +193,11 @@ public class TestWsWebSocketContainerGetOpenSessions extends WebSocketBaseTest {
             delayCount++;
         }
 
-        Assert.assertTrue(Tracker.checkRecord("client1", client1Count));
-        Assert.assertTrue(Tracker.checkRecord("client2", client2Count));
+        Assert.assertTrue(Tracker.dump(), Tracker.checkRecord("client1", client1Count));
+        Assert.assertTrue(Tracker.dump(), Tracker.checkRecord("client2", client2Count));
         // Note: need to strip leading '/' from path
-        Assert.assertTrue(Tracker.checkRecord(server1.substring(1), server1Count));
-        Assert.assertTrue(Tracker.checkRecord(server2.substring(1), server2Count));
+        Assert.assertTrue(Tracker.dump(), Tracker.checkRecord(server1.substring(1), server1Count));
+        Assert.assertTrue(Tracker.dump(), Tracker.checkRecord(server2.substring(1), server2Count));
 
         sClient1Server1.close();
         sClient1Server2.close();
@@ -384,5 +384,9 @@ public class TestWsWebSocketContainerGetOpenSessions extends WebSocketBaseTest {
             records.clear();
             updateCount.set(0);
         }
+
+        public static String dump() {
+            return records.toString();
+        }
     }
 }


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