Croway commented on code in PR #9210:
URL: https://github.com/apache/camel/pull/9210#discussion_r1087004598
##########
components/camel-atmosphere-websocket/src/test/java/org/apache/camel/component/atmosphere/websocket/WebsocketRouteWithInitParamTest.java:
##########
@@ -110,18 +114,18 @@ void
testWebsocketSingleClientBroadcastMultipleClientsGuaranteeDelivery() throws
final int awaitTime = 5;
connectionKeyUserMap.clear();
- WebsocketTestClient wsclient1 = new
WebsocketTestClient("ws://localhost:" + PORT + "/hola3", 2);
- WebsocketTestClient wsclient2 = new
WebsocketTestClient("ws://localhost:" + PORT + "/hola3", 2);
- WebsocketTestClient wsclient3 = new
WebsocketTestClient("ws://localhost:" + PORT + "/hola3", 2);
+ WebsocketTestClient wsclient1 = new
WebsocketTestClient("ws://localhost:" + PORT + "/guarantee", 2);
+ WebsocketTestClient wsclient2 = new
WebsocketTestClient("ws://localhost:" + PORT + "/guarantee", 2);
+ WebsocketTestClient wsclient3 = new
WebsocketTestClient("ws://localhost:" + PORT + "/guarantee", 2);
wsclient1.connect();
- wsclient1.await(awaitTime);
+ wsclient1.await(1);
Review Comment:
in this way the test runs faster, I noticed that when connecting a high
await time is not useful, moreover, there are 3 consequent await
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
