svn commit: r583154 - /incubator/qpid/branches/M2.1/java/client/src/test/java/org/apache/qpid/test/unit/topic/DurableSubscriptionTest.java

[ritchiem]

Author: ritchiem
Date: Tue Oct  9 06:19:37 2007
New Revision: 583154

URL: http://svn.apache.org/viewvc?rev=583154&view=rev
Log:
Qpid-627 Patch provided by Aidan Skinner to address hangs in 
DurableSubscriptionTest.

Modified:
    
incubator/qpid/branches/M2.1/java/client/src/test/java/org/apache/qpid/test/unit/topic/DurableSubscriptionTest.java

Modified: 
incubator/qpid/branches/M2.1/java/client/src/test/java/org/apache/qpid/test/unit/topic/DurableSubscriptionTest.java
URL: 
http://svn.apache.org/viewvc/incubator/qpid/branches/M2.1/java/client/src/test/java/org/apache/qpid/test/unit/topic/DurableSubscriptionTest.java?rev=583154&r1=583153&r2=583154&view=diff
==============================================================================
--- 
incubator/qpid/branches/M2.1/java/client/src/test/java/org/apache/qpid/test/unit/topic/DurableSubscriptionTest.java
 (original)
+++ 
incubator/qpid/branches/M2.1/java/client/src/test/java/org/apache/qpid/test/unit/topic/DurableSubscriptionTest.java
 Tue Oct  9 06:19:37 2007
@@ -143,17 +143,17 @@
         producer.send(session1.createTextMessage("A"));
 
         Message msg;
-        msg = consumer1.receive(100);
+        msg = consumer1.receive(500);
         assertNotNull("Message should be available", msg);
         assertEquals("Message Text doesn't match", "A", ((TextMessage) 
msg).getText());
 
-        msg = consumer1.receive(100);
+        msg = consumer1.receive(500);
         assertNull("There should be no more messages for consumption on 
consumer1.", msg);
 
         msg = consumer2.receive();
         assertNotNull(msg);
         assertEquals("Consumer 2 should also received the first msg.", "A", 
((TextMessage) msg).getText());
-        msg = consumer2.receive(100);
+        msg = consumer2.receive(500);
         assertNull("There should be no more messages for consumption on 
consumer2.", msg);
 
         consumer2.close();
@@ -164,21 +164,24 @@
         producer.send(session1.createTextMessage("B"));
 
         _logger.info("Receive message on consumer 1 :expecting B");
-        msg = consumer1.receive(100);
+        msg = consumer1.receive(500);
         assertNotNull("Consumer 1 should get message 'B'.", msg);
         assertEquals("Incorrect Message recevied on consumer1.", "B", 
((TextMessage) msg).getText());
         _logger.info("Receive message on consumer 1 :expecting null");
-        msg = consumer1.receive(100);
+        msg = consumer1.receive(500);
         assertNull("There should be no more messages for consumption on 
consumer1.", msg);
 
         _logger.info("Receive message on consumer 3 :expecting B");
-        msg = consumer3.receive(100);
+        msg = consumer3.receive(500);
         assertNotNull("Consumer 3 should get message 'B'.", msg);
         assertEquals("Incorrect Message recevied on consumer4.", "B", 
((TextMessage) msg).getText());
         _logger.info("Receive message on consumer 3 :expecting null");
-        msg = consumer3.receive(100);
+        msg = consumer3.receive(500);
         assertNull("There should be no more messages for consumption on 
consumer3.", msg);
-
+        
+        consumer1.close();
+        consumer3.close();
+        
         con.close();
     }