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

[do-not-reply at jboss.org]

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();
    }