You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by kw...@apache.org on 2016/03/07 08:51:57 UTC

svn commit: r1733873 - /qpid/java/trunk/systests/src/test/java/org/apache/qpid/test/client/CloseOnNoRouteForMandatoryMessageTest.java

Author: kwall
Date: Mon Mar  7 07:51:56 2016
New Revision: 1733873

URL: http://svn.apache.org/viewvc?rev=1733873&view=rev
Log:
QPID-7122: Prevent sporadic failure of CloseOnNoRouteForMandatoryMessageTest

Modified:
    qpid/java/trunk/systests/src/test/java/org/apache/qpid/test/client/CloseOnNoRouteForMandatoryMessageTest.java

Modified: qpid/java/trunk/systests/src/test/java/org/apache/qpid/test/client/CloseOnNoRouteForMandatoryMessageTest.java
URL: http://svn.apache.org/viewvc/qpid/java/trunk/systests/src/test/java/org/apache/qpid/test/client/CloseOnNoRouteForMandatoryMessageTest.java?rev=1733873&r1=1733872&r2=1733873&view=diff
==============================================================================
--- qpid/java/trunk/systests/src/test/java/org/apache/qpid/test/client/CloseOnNoRouteForMandatoryMessageTest.java (original)
+++ qpid/java/trunk/systests/src/test/java/org/apache/qpid/test/client/CloseOnNoRouteForMandatoryMessageTest.java Mon Mar  7 07:51:56 2016
@@ -73,6 +73,9 @@ public class CloseOnNoRouteForMandatoryM
 
         Message message = transactedSession.createMessage();
         mandatoryProducer.send(message);
+
+        _testExceptionListener.assertReceivedNoRoute(testQueueName);
+
         try
         {
             transactedSession.commit();
@@ -84,12 +87,6 @@ public class CloseOnNoRouteForMandatoryM
             //The session was marked closed even before we had a chance to call commit on it
             assertTrue("ISE did not indicate closure", ise.getMessage().contains("closed"));
         }
-        catch(JMSException e)
-        {
-            _logger.debug("Caught exception", e);
-            _testExceptionListener.assertNoRoute(e, testQueueName);
-        }
-        _testExceptionListener.assertReceivedNoRoute(testQueueName);
 
         forgetConnection(_connection);
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org