Author: orudyy
Date: Mon May 12 11:33:56 2014
New Revision: 1593930

URL: http://svn.apache.org/r1593930
Log:
QPID-5761: Fix BDBHAVirtualHostNodeRestTest test failure caused by premature 
call to virtual host rest service before opening of virtual host

Patch Supplied by Andrew MacBean <[email protected]>.

Modified:
    
qpid/trunk/qpid/java/bdbstore/systests/src/main/java/org/apache/qpid/server/store/berkeleydb/BDBHAVirtualHostNodeRestTest.java

Modified: 
qpid/trunk/qpid/java/bdbstore/systests/src/main/java/org/apache/qpid/server/store/berkeleydb/BDBHAVirtualHostNodeRestTest.java
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/bdbstore/systests/src/main/java/org/apache/qpid/server/store/berkeleydb/BDBHAVirtualHostNodeRestTest.java?rev=1593930&r1=1593929&r2=1593930&view=diff
==============================================================================
--- 
qpid/trunk/qpid/java/bdbstore/systests/src/main/java/org/apache/qpid/server/store/berkeleydb/BDBHAVirtualHostNodeRestTest.java
 (original)
+++ 
qpid/trunk/qpid/java/bdbstore/systests/src/main/java/org/apache/qpid/server/store/berkeleydb/BDBHAVirtualHostNodeRestTest.java
 Mon May 12 11:33:56 2014
@@ -28,6 +28,7 @@ import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 
+import org.apache.qpid.server.model.State;
 import org.apache.qpid.server.model.VirtualHost;
 import org.apache.qpid.server.model.VirtualHostNode;
 import 
org.apache.qpid.server.store.berkeleydb.replication.ReplicatedEnvironmentFacade;
@@ -132,13 +133,12 @@ public class BDBHAVirtualHostNodeRestTes
         assertEquals("Unexpected path", new File(_storeBaseDir, 
nodeName).getPath(), nodeData.get(BDBHAVirtualHostNode.STORE_PATH));
         assertEquals("Unexpected address", "localhost:" + nodePort, 
nodeData.get(BDBHAVirtualHostNode.ADDRESS));
         assertEquals("Unexpected helper address", "localhost:" + 
nodeHelperPort, nodeData.get(BDBHAVirtualHostNode.HELPER_ADDRESS));
-        assertEquals("Unexpected group name", _hostName, 
nodeData.get(BDBHAVirtualHostNode.GROUP_NAME));;
+        assertEquals("Unexpected group name", _hostName, 
nodeData.get(BDBHAVirtualHostNode.GROUP_NAME));
         assertEquals("Unexpected role", expectedRole, 
nodeData.get(BDBHAVirtualHostNode.ROLE));
 
         if (isMaster)
         {
-            Map<String, Object> hostData = 
getRestTestHelper().getJsonAsSingletonList("virtualhost/" + masterNode + "/" + 
_hostName + "?depth=0");
-            assertEquals("Unexpected host name", _hostName, 
hostData.get(VirtualHost.NAME));
+            waitForAttributeChanged("virtualhost/" + masterNode + "/" + 
_hostName + "?depth=0", VirtualHost.STATE, State.ACTIVE.name());
         }
 
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to