[activemq-artemis] branch master updated: ARTEMIS-2659 Fixing auto created queues leading to journal errors

[clebertsuconic]

This is an automated email from the ASF dual-hosted git repository.

clebertsuconic pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/activemq-artemis.git


The following commit(s) were added to refs/heads/master by this push:
     new ee29b4b  ARTEMIS-2659 Fixing auto created queues leading to journal 
errors
ee29b4b is described below

commit ee29b4be107527f4d1fa377b9e51845585ea2cda
Author: Clebert Suconic <clebertsuco...@apache.org>
AuthorDate: Wed Apr 8 12:42:36 2020 -0400

    ARTEMIS-2659 Fixing auto created queues leading to journal errors
    
    This was causing the queue to be removed right before the server being 
stopped, leading to exceptions about the journal being stopped.
---
 .../artemis/tests/integration/amqp/AmqpLargeMessageTest.java       | 4 ----
 .../activemq/artemis/tests/integration/amqp/TopicDurableTests.java | 7 -------
 2 files changed, 11 deletions(-)

diff --git 
a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/AmqpLargeMessageTest.java
 
b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/AmqpLargeMessageTest.java
index f20c933..304ce7c 100644
--- 
a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/AmqpLargeMessageTest.java
+++ 
b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/AmqpLargeMessageTest.java
@@ -87,10 +87,6 @@ public class AmqpLargeMessageTest extends 
AmqpClientTestSupport {
    }
 
    @Override
-   protected void createAddressAndQueues(ActiveMQServer server) throws 
Exception {
-   }
-
-   @Override
    protected void addAdditionalAcceptors(ActiveMQServer server) throws 
Exception {
       server.getConfiguration().addAcceptorConfiguration("tcp", 
"tcp://localhost:61616");
    }
diff --git 
a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/TopicDurableTests.java
 
b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/TopicDurableTests.java
index 8e7f463..e4a9066 100644
--- 
a/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/TopicDurableTests.java
+++ 
b/tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/TopicDurableTests.java
@@ -39,7 +39,6 @@ import java.util.concurrent.atomic.AtomicInteger;
 import java.util.stream.IntStream;
 
 import org.apache.activemq.artemis.api.core.SimpleString;
-import org.apache.activemq.artemis.core.server.ActiveMQServer;
 import org.apache.activemq.artemis.utils.Wait;
 import org.apache.qpid.jms.JmsConnectionFactory;
 import org.junit.Test;
@@ -48,12 +47,6 @@ import static org.hamcrest.CoreMatchers.is;
 
 public class TopicDurableTests extends JMSClientTestSupport {
 
-   @Override
-   protected void createAddressAndQueues(ActiveMQServer server) throws 
Exception {
-      // do not create unnecessary queues
-   }
-
-
    @Test
    public void testMessageDurableSubscription() throws Exception {
       JmsConnectionFactory connectionFactory = new 
JmsConnectionFactory(getBrokerQpidJMSConnectionURI() + 
"?jms.clientID=jmsTopicClient");