[hornetq-commits] JBoss hornetq SVN: r8286 - trunk/tests/src/org/hornetq/tests/integration/jms/consumer.

Author: timfox Date: 2009-11-15 05:35:38 -0500 (Sun, 15 Nov 2009) New Revision: 8286 Modified: trunk/tests/src/org/hornetq/tests/integration/jms/consumer/ConsumerTest.java Log: fixed bogus tests Modified: trunk/tests/src/org/hornetq/tests/integration/jms/consumer/ConsumerTest.java =================================================================== --- trunk/tests/src/org/hornetq/tests/integration/jms/consumer/ConsumerTest.java 2009-11-14 09:11:04 UTC (rev 8285) +++ trunk/tests/src/org/hornetq/tests/integration/jms/consumer/ConsumerTest.java 2009-11-15 10:35:38 UTC (rev 8286) @@ -137,8 +137,9 @@ Message m = consumer.receiveNoWait(); assertNull(m); - SimpleString queueName = new SimpleString(HornetQQueue.JMS_QUEUE_ADDRESS_PREFIX + Q_NAME); - assertEquals(0, ((Queue)server.getPostOffice().getBinding(queueName).getBindable()).getDeliveringCount()); + //Asserting delivering count is zero is bogus since messages might still be being delivered and expired at this point + //which can cause delivering count to flip to 1 + conn.close(); } @@ -164,8 +165,9 @@ Message m = consumer.receiveNoWait(); assertNull(m); - SimpleString queueName = new SimpleString(HornetQQueue.JMS_QUEUE_ADDRESS_PREFIX + Q_NAME); - assertEquals(0, ((Queue)server.getPostOffice().getBinding(queueName).getBindable()).getDeliveringCount()); + //Asserting delivering count is zero is bogus since messages might still be being delivered and expired at this point + //which can cause delivering count to flip to 1 + conn.close(); }