This commit is contained in:
Clebert Suconic 2018-05-03 16:44:32 -04:00
commit 3c0f3cf52f
3 changed files with 16 additions and 2 deletions

View File

@ -249,7 +249,7 @@ public class AMQSession implements SessionCallback {
routingTypeToUse = as.getDefaultAddressRoutingType(); routingTypeToUse = as.getDefaultAddressRoutingType();
} }
} }
coreSession.createQueue(addressToUse, queueNameToUse, routingTypeToUse, null, isTemporary, true); coreSession.createQueue(addressToUse, queueNameToUse, routingTypeToUse, null, isTemporary, true, true);
connection.addKnownDestination(queueName); connection.addKnownDestination(queueName);
} else { } else {
hasQueue = false; hasQueue = false;

View File

@ -419,6 +419,7 @@ public class SimpleStringTest extends Assert {
final SimpleString expectedPooled = pool.getOrCreate(bb); final SimpleString expectedPooled = pool.getOrCreate(bb);
bb.resetReaderIndex(); bb.resetReaderIndex();
Assert.assertSame(expectedPooled, pool.getOrCreate(bb)); Assert.assertSame(expectedPooled, pool.getOrCreate(bb));
bb.resetReaderIndex();
} }
} }

View File

@ -919,10 +919,12 @@ public class SimpleOpenWireTest extends BasicOpenWireTest {
} }
@Test @Test
public void testAutoDestinationCreationOnConsumer() throws JMSException { public void testAutoDestinationCreationAndDeletionOnConsumer() throws Exception {
AddressSettings addressSetting = new AddressSettings(); AddressSettings addressSetting = new AddressSettings();
addressSetting.setAutoCreateQueues(true); addressSetting.setAutoCreateQueues(true);
addressSetting.setAutoCreateAddresses(true); addressSetting.setAutoCreateAddresses(true);
addressSetting.setAutoDeleteQueues(true);
addressSetting.setAutoDeleteAddresses(true);
String address = "foo"; String address = "foo";
server.getAddressSettingsRepository().addMatch(address, addressSetting); server.getAddressSettingsRepository().addMatch(address, addressSetting);
@ -935,11 +937,22 @@ public class SimpleOpenWireTest extends BasicOpenWireTest {
MessageConsumer consumer = session.createConsumer(queue); MessageConsumer consumer = session.createConsumer(queue);
assertTrue(Wait.waitFor(() -> (server.locateQueue(SimpleString.toSimpleString("foo")) != null), 2000, 100));
assertTrue(Wait.waitFor(() -> (server.getAddressInfo(SimpleString.toSimpleString("foo")) != null), 2000, 100));
MessageProducer producer = session.createProducer(null); MessageProducer producer = session.createProducer(null);
producer.send(queue, message); producer.send(queue, message);
TextMessage message1 = (TextMessage) consumer.receive(1000); TextMessage message1 = (TextMessage) consumer.receive(1000);
assertTrue(message1.getText().equals(message.getText())); assertTrue(message1.getText().equals(message.getText()));
assertNotNull(server.locateQueue(SimpleString.toSimpleString("foo")));
consumer.close();
connection.close();
assertTrue(Wait.waitFor(() -> (server.locateQueue(SimpleString.toSimpleString("foo")) == null), 2000, 100));
assertTrue(Wait.waitFor(() -> (server.getAddressInfo(SimpleString.toSimpleString("foo")) == null), 2000, 100));
} }
@Test @Test