https://issues.apache.org/jira/browse/AMQ-5621

Turn off JMX, switch to JUnit 4 style test with timeout, actually
shutdown the running broker and clean up in tear down method.

activemq-unit-tests/src/test/java/org/apache/activemq/bugs/AMQ3779Test.java

@@ -17,29 +17,40 @@
 
 package org.apache.activemq.bugs;
 
+import static org.junit.Assert.assertTrue;
+
 import java.util.concurrent.atomic.AtomicBoolean;
+
 import javax.jms.Connection;
 import javax.jms.DeliveryMode;
 import javax.jms.MessageProducer;
 import javax.jms.Session;
+
 import org.apache.activemq.ActiveMQConnectionFactory;
-import org.apache.activemq.AutoFailTestSupport;
 import org.apache.activemq.broker.BrokerService;
 import org.apache.activemq.broker.util.LoggingBrokerPlugin;
 import org.apache.activemq.util.DefaultTestAppender;
 import org.apache.log4j.Appender;
 import org.apache.log4j.Logger;
 import org.apache.log4j.spi.LoggingEvent;
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
 
-public class AMQ3779Test extends AutoFailTestSupport {
+public class AMQ3779Test {
 
-    private static final Logger logger = Logger.getLogger(AMQ3779Test.class);
+    private static final Logger LOG = Logger.getLogger(AMQ3779Test.class);
     private static final String qName = "QNameToFind";
 
-    public void testLogPerDest() throws Exception {
+    private BrokerService brokerService;
+    private Appender appender;
+    private final AtomicBoolean ok = new AtomicBoolean(false);
 
-        final AtomicBoolean ok = new AtomicBoolean(false);
+    @Before
-        Appender appender = new DefaultTestAppender() {
+    public void setUp() throws Exception {
+        ok.set(false);
+
+        appender = new DefaultTestAppender() {
             @Override
             public void doAppend(LoggingEvent event) {
                 if (event.getLoggerName().toString().contains(qName)) {
@@ -47,19 +58,39 @@ public class AMQ3779Test extends AutoFailTestSupport {
                 }
             }
         };
-        logger.getRootLogger().addAppender(appender);
+
+        Logger.getRootLogger().addAppender(appender);
 
         try {
-
+            brokerService = new BrokerService();
-            BrokerService broker = new BrokerService();
             LoggingBrokerPlugin loggingBrokerPlugin = new LoggingBrokerPlugin();
             loggingBrokerPlugin.setPerDestinationLogger(true);
             loggingBrokerPlugin.setLogAll(true);
-            broker.setPlugins(new LoggingBrokerPlugin[]{loggingBrokerPlugin});
+            brokerService.setPlugins(new LoggingBrokerPlugin[]{loggingBrokerPlugin});
-            broker.start();
 
+            brokerService.setPersistent(false);
+            brokerService.setUseJmx(false);
+            brokerService.start();
+        } finally {
+            LOG.removeAppender(appender);
+        }
+    }
 
-            Connection connection = new ActiveMQConnectionFactory(broker.getVmConnectorURI()).createConnection();
+    @After
+    public void tearDown() throws Exception {
+        try {
+            if (brokerService != null) {
+                brokerService.stop();
+                brokerService.waitUntilStopped();
+            }
+        } finally {
+            LOG.removeAppender(appender);
+        }
+    }
+
+    @Test(timeout = 60000)
+    public void testLogPerDest() throws Exception {
+        Connection connection = new ActiveMQConnectionFactory(brokerService.getVmConnectorURI()).createConnection();
         Session session = connection.createSession(false, Session.AUTO_ACKNOWLEDGE);
         MessageProducer messageProducer = session.createProducer(session.createQueue(qName));
         messageProducer.setDeliveryMode(DeliveryMode.PERSISTENT);
@@ -69,8 +100,5 @@ public class AMQ3779Test extends AutoFailTestSupport {
         connection.close();
 
         assertTrue("got expected log message", ok.get());
-        } finally {
-            logger.removeAppender(appender);
-        }
     }
 }