[jboss-cvs] JBossAS SVN: r67633 - trunk/testsuite/src/main/org/jboss/test/cluster/multicfg/web/field/test.

[jboss-cvs-commits at lists.jboss.org]

Author: bstansberry at jboss.com
Date: 2007-11-29 18:43:59 -0500 (Thu, 29 Nov 2007)
New Revision: 67633

Modified:
   trunk/testsuite/src/main/org/jboss/test/cluster/multicfg/web/field/test/StateTransferTestCase.java
Log:
Make assertions clearer

Modified: trunk/testsuite/src/main/org/jboss/test/cluster/multicfg/web/field/test/StateTransferTestCase.java
===================================================================
--- trunk/testsuite/src/main/org/jboss/test/cluster/multicfg/web/field/test/StateTransferTestCase.java	2007-11-29 23:27:25 UTC (rev 67632)
+++ trunk/testsuite/src/main/org/jboss/test/cluster/multicfg/web/field/test/StateTransferTestCase.java	2007-11-29 23:43:59 UTC (rev 67633)
@@ -147,6 +147,9 @@
 
       getLog().debug(setUrl_ + ":::::::" + getUrl_);
       
+      Set sessions = SessionTestUtil.getSessionIds(adaptor0_, warFqn_);
+      assertEquals("server0 has no cached sessions", 0, sessions.size());
+      
       // Stop the war on server1
       undeploy(adaptor1_, getWarName() + ".war");
 
@@ -169,9 +172,9 @@
       getLog().debug("Sessions created");
       
       // Confirm there are no sessions in the server1 cache
-      Set sessions = SessionTestUtil.getSessionIds(adaptor1_, warFqn_);
+      sessions = SessionTestUtil.getSessionIds(adaptor1_, warFqn_);
       
-      assertTrue("server1 has no cached sessions", sessions.size() == 0);
+      assertEquals("server1 has no cached sessions", 0, sessions.size());
       
       getLog().debug("Server1 has no cached sessions");
       
@@ -205,7 +208,7 @@
       // Confirm there are no sessions in the server0 cache
       sessions = SessionTestUtil.getSessionIds(adaptor0_, warFqn_);
       
-      assertTrue("server0 has no cached sessions", sessions.size() == 0);
+      assertEquals("server0 has no cached sessions", 0, sessions.size());
       
       getLog().debug("Server0 has no cached sessions");
    }