[hornetq-commits] JBoss hornetq SVN: r12081 - trunk/tests/integration-tests/src/test/java/org/hornetq/tests/integration/jms/client.

[do-not-reply at jboss.org]

Author: borges
Date: 2012-02-03 08:27:53 -0500 (Fri, 03 Feb 2012)
New Revision: 12081

Modified:
   trunk/tests/integration-tests/src/test/java/org/hornetq/tests/integration/jms/client/StoreConfigTest.java
Log:
Restore: improved test code, only catch the expected exception

Modified: trunk/tests/integration-tests/src/test/java/org/hornetq/tests/integration/jms/client/StoreConfigTest.java
===================================================================
--- trunk/tests/integration-tests/src/test/java/org/hornetq/tests/integration/jms/client/StoreConfigTest.java	2012-02-03 13:27:36 UTC (rev 12080)
+++ trunk/tests/integration-tests/src/test/java/org/hornetq/tests/integration/jms/client/StoreConfigTest.java	2012-02-03 13:27:53 UTC (rev 12081)
@@ -74,17 +74,15 @@
       jmsServer.createConnectionFactory(false, nonPersisted, "/nonPersisted" );
       
       
-      boolean ex = false;
       try
       {
          jmsServer.addConnectionFactoryToJNDI("np", "/someCF");
+         fail("Failure expected and the API let duplicates");
       }
-      catch (Exception expected)
+      catch (HornetQException expected)
       {
-         ex = true;
+         // expected
       }
-      
-      assertTrue(ex);
 
 
       openCon("/someCF");
@@ -104,18 +102,15 @@
       jmsServer.start();
       
       jmsServer.addConnectionFactoryToJNDI("tst", "/newJNDI");
-      
-      ex = false;
       try
       {
          jmsServer.addConnectionFactoryToJNDI("tst", "/newJNDI");
+         fail("Failure expected and the API let duplicates");
       }
-      catch (Exception expected)
+      catch (HornetQException expected)
       {
-         ex = true;
+        // expected
       }
-      assertTrue(ex);
-      
       openCon("/someCF");
       openCon("/someCF2");
       openCon("/newJNDI");