Author: rjung
Date: Tue Feb 12 16:13:40 2013
New Revision: 1445231
URL: http://svn.apache.org/r1445231
Log:
Stop ServerContainerImpl in contextDestroyed
by basing the test listeners on WsListener.
ServerContainerImpl.stop() is protected so not
directly accessable.
stop() needs to be called in order to stop
websocket timeout threads.
Modified:
tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java
tomcat/trunk/test/org/apache/tomcat/websocket/TesterEchoServer.java
Modified:
tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java?rev=1445231&r1=1445230&r2=1445231&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java
(original)
+++ tomcat/trunk/test/org/apache/tomcat/websocket/TestWsWebSocketContainer.java
Tue Feb 12 16:13:40 2013
@@ -403,7 +403,7 @@ public class TestWsWebSocketContainer ex
}
- public static class BlockingConfig implements ServletContextListener {
+ public static class BlockingConfig extends WsListener {
public static final String PATH = "/block";
@@ -415,7 +415,7 @@ public class TestWsWebSocketContainer ex
@Override
public void contextDestroyed(ServletContextEvent sce) {
- // NO-OP
+ super.contextDestroyed(sce);
}
}
@@ -524,14 +524,14 @@ public class TestWsWebSocketContainer ex
}
- public static class ConstantTxConfig implements ServletContextListener {
+ public static class ConstantTxConfig extends WsListener {
private static final String PATH = "/test";
@Override
public void contextInitialized(ServletContextEvent sce) {
+ super.contextInitialized(sce);
ServerContainerImpl sc = ServerContainerImpl.getServerContainer();
- sc.setServletContext(sce.getServletContext());
try {
sc.publishServer(ConstantTxEndpoint.class, PATH,
DefaultServerConfiguration.class);
@@ -545,7 +545,7 @@ public class TestWsWebSocketContainer ex
@Override
public void contextDestroyed(ServletContextEvent sce) {
- // NO-OP
+ super.contextDestroyed(sce);
}
}
}
Modified: tomcat/trunk/test/org/apache/tomcat/websocket/TesterEchoServer.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/tomcat/websocket/TesterEchoServer.java?rev=1445231&r1=1445230&r2=1445231&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/tomcat/websocket/TesterEchoServer.java
(original)
+++ tomcat/trunk/test/org/apache/tomcat/websocket/TesterEchoServer.java Tue Feb
12 16:13:40 2013
@@ -25,10 +25,11 @@ import javax.websocket.Session;
import javax.websocket.WebSocketMessage;
import org.apache.tomcat.websocket.server.ServerContainerImpl;
+import org.apache.tomcat.websocket.server.WsListener;
public class TesterEchoServer {
- public static class Config implements ServletContextListener {
+ public static class Config extends WsListener {
public static final String PATH_ASYNC = "/echoAsync";
public static final String PATH_BASIC = "/echoBasic";
@@ -43,7 +44,7 @@ public class TesterEchoServer {
@Override
public void contextDestroyed(ServletContextEvent sce) {
- // NO-OP
+ super.contextDestroyed(sce);
}
}
@@ -106,4 +107,4 @@ public class TesterEchoServer {
}
}
}
-}
\ No newline at end of file
+}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
