From 9f28d26cffdb6466bddcc53eb8b6a868ca860212 Mon Sep 17 00:00:00 2001 From: Joakim Erdfelt Date: Wed, 23 Aug 2017 15:58:22 -0700 Subject: [PATCH] Fixed #1766 - Fixing test case results --- .../jsr356/server/DelayedStartClientOnServerTest.java | 2 +- .../org/eclipse/jetty/websocket/client/WebSocketClient.java | 6 +----- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/jetty-websocket/javax-websocket-server-impl/src/test/java/org/eclipse/jetty/websocket/jsr356/server/DelayedStartClientOnServerTest.java b/jetty-websocket/javax-websocket-server-impl/src/test/java/org/eclipse/jetty/websocket/jsr356/server/DelayedStartClientOnServerTest.java index 9da6b911cb1..602e8920a6d 100644 --- a/jetty-websocket/javax-websocket-server-impl/src/test/java/org/eclipse/jetty/websocket/jsr356/server/DelayedStartClientOnServerTest.java +++ b/jetty-websocket/javax-websocket-server-impl/src/test/java/org/eclipse/jetty/websocket/jsr356/server/DelayedStartClientOnServerTest.java @@ -247,7 +247,7 @@ public class DelayedStartClientOnServerTest assertThat("Response", response, startsWith("Connected to ws://")); List threadNames = getThreadNames(server); assertNoHttpClientPoolThreads(threadNames); - assertThat("Threads", threadNames, hasItem(containsString("Jsr356Client@"))); + assertThat("Threads", threadNames, hasItem(containsString("WebSocketContainer@"))); } finally { diff --git a/jetty-websocket/websocket-client/src/main/java/org/eclipse/jetty/websocket/client/WebSocketClient.java b/jetty-websocket/websocket-client/src/main/java/org/eclipse/jetty/websocket/client/WebSocketClient.java index 04d6e64a543..8f4149ddd03 100644 --- a/jetty-websocket/websocket-client/src/main/java/org/eclipse/jetty/websocket/client/WebSocketClient.java +++ b/jetty-websocket/websocket-client/src/main/java/org/eclipse/jetty/websocket/client/WebSocketClient.java @@ -392,11 +392,7 @@ public class WebSocketClient extends ContainerLifeCycle implements WebSocketCont if (LOG.isDebugEnabled()) LOG.debug("Stopping {}",this); - - if (ShutdownThread.isRegistered(this)) - { - ShutdownThread.deregister(this); - } + ShutdownThread.deregister(this); super.doStop();