From 9b017fdb391ae1c5ef890e3824a77d63a490dc11 Mon Sep 17 00:00:00 2001 From: Timothy Bish Date: Mon, 18 Aug 2014 17:58:58 -0400 Subject: [PATCH] Ensure all resources shut down. --- .../transport/ws/WSTransportTest.java | 19 ++++++++++++++----- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/activemq-http/src/test/java/org/apache/activemq/transport/ws/WSTransportTest.java b/activemq-http/src/test/java/org/apache/activemq/transport/ws/WSTransportTest.java index 6c99328fd2..b133e71319 100644 --- a/activemq-http/src/test/java/org/apache/activemq/transport/ws/WSTransportTest.java +++ b/activemq-http/src/test/java/org/apache/activemq/transport/ws/WSTransportTest.java @@ -17,6 +17,8 @@ package org.apache.activemq.transport.ws; +import static org.junit.Assert.assertTrue; + import java.io.File; import java.io.IOException; import java.net.Socket; @@ -47,8 +49,6 @@ import org.openqa.selenium.firefox.FirefoxProfile; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import static org.junit.Assert.assertTrue; - public class WSTransportTest { private static final Logger LOG = LoggerFactory.getLogger(WSTransportTest.class); @@ -137,13 +137,21 @@ public class WSTransportTest { } catch(Exception e) { // Some tests explicitly disconnect from stomp so can ignore } finally { - stopBroker(); + try { + stopBroker(); + } catch (Exception e) { + LOG.warn("Error on Broker stop."); + } if (driver != null) { - driver.quit(); + try { + driver.quit(); + } catch (Exception e) {} driver = null; } if (server != null) { - server.stop(); + try { + server.stop(); + } catch (Exception e) {} } } } @@ -250,6 +258,7 @@ public class WSTransportTest { assertTrue("Should have disconnected", Wait.waitFor(new Wait.Condition() { + @Override public boolean isSatisified() throws Exception { return webStatus.getText().equals("Disconnected"); }