QPID-2120 : Updated SimpleAMQQueue to ensure that the sub.isSuspended() check is done before any attempt to retrieve the getLastSeenNode(). This and the commits for QPID-1871,QPID-2116 are tested by RollbackOrderTest that now can be enabled for the Java profile.

Additional changes were done to SAMQQ to improve readability and add some debug loggging. Performance testing should be done to identify any penality for this additional logging.

git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@820320 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 6 insertions, 0 deletions

diff --git a/qpid/java/systests/src/main/java/org/apache/qpid/test/client/RollbackOrderTest.java b/qpid/java/systests/src/main/java/org/apache/qpid/test/client/RollbackOrderTest.java
index 2efe93eed8..ff766c907d 100644
--- a/qpid/java/systests/src/main/java/org/apache/qpid/test/client/RollbackOrderTest.java
+++ b/qpid/java/systests/src/main/java/org/apache/qpid/test/client/RollbackOrderTest.java
@@ -144,6 +144,7 @@ public class RollbackOrderTest extends QpidTestCase
                 }
                 catch (JMSException e)
                 {
+                    System.out.println("Error:" + e.getMessage());
                     exceptions[(int)count.getCount()] = e;
                 }
                 catch (AssertionFailedError cf)
@@ -154,6 +155,7 @@ public class RollbackOrderTest extends QpidTestCase
                         count.countDown();
                     }
 
+                    System.out.println("Error:" + cf.getMessage());
                     System.err.println(cf.getMessage());
                     cf.printStackTrace();
                     failed.set(true);
@@ -175,11 +177,15 @@ public class RollbackOrderTest extends QpidTestCase
             }
         }
 
+//        _consumer.close();
+        _connection.close();
+        
         assertFalse("Exceptions thrown during test run, Check Std.err.", failed.get());
     }
 
     @Override public void tearDown() throws Exception
     {
+
         drainQueue(_queue);
 
         super.tearDown();