Github user gemmellr commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/2084#discussion_r187566075
  
    --- Diff: 
tests/integration-tests/src/test/java/org/apache/activemq/artemis/tests/integration/amqp/JMSMessageProducerTest.java
 ---
    @@ -70,10 +70,31 @@ public void testAnonymousProducer() throws Exception {
        }
     
        @Test(timeout = 30000)
    -   public void testAnonymousProducerWithAutoCreation() throws Exception {
    -      Connection connection = createConnection();
    +   public void testAnonymousProducerWithQueueAutoCreation() throws 
Exception {
    +      try (Connection connection = createConnection()) {
    +         Session session = connection.createSession(false, 
Session.AUTO_ACKNOWLEDGE);
    +         Queue queue = session.createQueue(UUID.randomUUID().toString());
    +         MessageProducer p = session.createProducer(null);
     
    -      try {
    +         TextMessage message = session.createTextMessage();
    +         message.setText("hello");
    +         // this will auto-create the address
    +         p.send(queue, message);
    --- End diff --
    
    This send shouldn't be needed in this test since it would be a queue being 
created.


---

Reply via email to