I would like to request everybody to not do the following pattern on
tests. That means also look for this pattern on other people's pull
request (including mine.. I'm writing to myself also here).


Assert.assertNull(consumer.receive(ANY-TIMEOUT));


Instead, please use consumer.receiveNoWait();


some tests taking 2 seconds on a loop for each time the method is
called, being executed several time can amount for a few extra minutes
on the testsuite.


Even though I have seen this pattern in Artemis, this same request is
valid for ActiveMQ.


Thank you!


-- 
Clebert Suconic

Reply via email to