Improve resilience of some tests

acogoluegnes · acogoluegnes · commit caba8b8ffb64 · 2016-09-06T09:26:09.000+02:00
Those regularly fail on CI.
diff --git a/src/test/java/com/rabbitmq/client/test/functional/DeadLetterExchange.java b/src/test/java/com/rabbitmq/client/test/functional/DeadLetterExchange.java
@@ -583,7 +583,7 @@ private void sleep(long millis) {
        publication time + TTL + latency */
     private void checkPromptArrival(QueueingConsumer c,
                                     int count, long latency) throws Exception {
-        long epsilon = TTL / 20;
+        long epsilon = TTL / 10;
         for (int i = 0; i < count; i++) {
             Delivery d = c.nextDelivery(TTL + TTL + latency + epsilon);
             assertNotNull("message #" + i + " did not expire", d);
diff --git a/src/test/java/com/rabbitmq/client/test/functional/TTLHandling.java b/src/test/java/com/rabbitmq/client/test/functional/TTLHandling.java
@@ -156,7 +156,7 @@ protected void releaseResources() throws IOException {
         closeChannel();
         openChannel();
 
-        Thread.sleep(150);
+        Thread.sleep(110);
         expectBodyAndRemainingMessages(MSG[1], 1);
         expectBodyAndRemainingMessages(MSG[2], 0);
     }
diff --git a/src/test/java/com/rabbitmq/client/test/server/XDeathHeaderGrowth.java b/src/test/java/com/rabbitmq/client/test/server/XDeathHeaderGrowth.java
@@ -28,6 +28,7 @@
 import java.util.UUID;
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
+import java.util.concurrent.atomic.AtomicLong;
 
 import org.junit.Test;
 
@@ -40,24 +41,35 @@
 class RejectingConsumer extends DefaultConsumer {
     private CountDownLatch latch;
     private Map<String, Object> headers;
+    private AtomicLong counter;
 
     public RejectingConsumer(Channel channel, CountDownLatch latch) {
         super(channel);
         this.latch = latch;
+        this.counter = new AtomicLong(latch.getCount());
     }
 
     @Override
     public void handleDelivery(String consumerTag, Envelope envelope,
                                AMQP.BasicProperties properties, byte[] body)
             throws IOException {
+
         if(this.latch.getCount() > 0) {
             this.getChannel().basicReject(envelope.getDeliveryTag(), false);
         } else {
             if(this.getChannel().isOpen()) {
                 this.getChannel().basicAck(envelope.getDeliveryTag(), false);
             }
         }
-        this.headers = properties.getHeaders();
+        if(this.counter.decrementAndGet() == 0) {
+            // get headers only when the message has been redelivered
+            // the expected number of times.
+            // it looks like the message can be redelivered because
+            // of the reject when handleDelivery isn't done yet or
+            // before the latch releases the main thread. There's then
+            // an additional delivery and the checks (transiently) fail.
+            this.headers = properties.getHeaders();
+        }
         latch.countDown();
     }
 
@@ -88,7 +100,7 @@ public class XDeathHeaderGrowth extends BrokerTestCase {
         this.channel.queueBind(q3, x2, "");
 
         final String qz = "issues.rabbitmq-server-78.destination";
-        declareTransientQueue(qz, argumentsForDeadLetteringTo(x3));
+        declareTransientQueue(qz, argumentsForDeadLetteringWithoutTtlTo(x3));
         this.channel.queueBind(qz, x3, "");
 
         CountDownLatch latch = new CountDownLatch(10);
@@ -143,7 +155,7 @@ private void cleanUpQueues(String... qs) throws IOException {
         this.channel.queueBind(q2, x1, "");
 
         final String qz = "issues.rabbitmq-server-152.destination";
-        declareTransientQueue(qz, argumentsForDeadLetteringTo(x2));
+        declareTransientQueue(qz, argumentsForDeadLetteringWithoutTtlTo(x2));
         this.channel.queueBind(qz, x2, "");
 
         CountDownLatch latch = new CountDownLatch(10);
@@ -215,11 +227,17 @@ private Map<String, Object> argumentsForDeadLetteringTo(String dlx) {
         return argumentsForDeadLetteringTo(dlx, 1);
     }
 
+    private Map<String, Object> argumentsForDeadLetteringWithoutTtlTo(String dlx) {
+        return argumentsForDeadLetteringTo(dlx, -1);
+    }
+
     private Map<String, Object> argumentsForDeadLetteringTo(String dlx, int ttl) {
         Map<String, Object> m = new HashMap<String, Object>();
         m.put("x-dead-letter-exchange", dlx);
         m.put("x-dead-letter-routing-key", "some-routing-key");
-        m.put("x-message-ttl", ttl);
+        if(ttl > 0) {
+            m.put("x-message-ttl", ttl);
+        }
         return m;
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -156,7 +156,7 @@ protected void releaseResources() throws IOException {`
`156`	`156`	`closeChannel();`
`157`	`157`	`openChannel();`
`158`	`158`
`159`		`- Thread.sleep(150);`
	`159`	`+ Thread.sleep(110);`
`160`	`160`	`expectBodyAndRemainingMessages(MSG[1], 1);`
`161`	`161`	`expectBodyAndRemainingMessages(MSG[2], 0);`
`162`	`162`	`}`