[jboss-cvs] JBossAS SVN: r80715 - trunk/testsuite/src/main/org/jboss/test/cluster/invokerha.

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

Author: bstansberry at jboss.com
Date: 2008-11-08 23:56:26 -0500 (Sat, 08 Nov 2008)
New Revision: 80715

Modified:
   trunk/testsuite/src/main/org/jboss/test/cluster/invokerha/InvokerHaFailureType.java
Log:
Get rid of compiler warns

Modified: trunk/testsuite/src/main/org/jboss/test/cluster/invokerha/InvokerHaFailureType.java
===================================================================
--- trunk/testsuite/src/main/org/jboss/test/cluster/invokerha/InvokerHaFailureType.java	2008-11-09 04:48:53 UTC (rev 80714)
+++ trunk/testsuite/src/main/org/jboss/test/cluster/invokerha/InvokerHaFailureType.java	2008-11-09 04:56:26 UTC (rev 80715)
@@ -86,6 +86,9 @@
          case BEFORE_SERVER:
             log.debug("failing because of " + this);
             throw new IllegalStateException("see how you handle this!!");
+            
+         default:
+            break;
       }
    }
    
@@ -108,6 +111,9 @@
          case AFTER_SERVER_COMPLETED:
             log.debug("failing because of " + this);
             throw new GenericClusteringException(GenericClusteringException.COMPLETED_YES, this.toString());
+            
+         default:
+            break;
       }
    }
 
@@ -116,33 +122,36 @@
       switch(this)
       {
          case BEFORE_SERVER:
-            TestCase.assertEquals(0, failoverCounter);
+            TestCase.assertEquals(Integer.valueOf(0), failoverCounter);
             break;
             
          case AFTER_SERVER_NOT_COMPLETED_BUT_SUCCESS_AFTER:
-            TestCase.assertEquals(1, failoverCounter);
+            TestCase.assertEquals(Integer.valueOf(1), failoverCounter);
             break;
             
          case AFTER_SERVER_NOT_COMPLETED_BOTH_SERVERS:
             if (injectFailureIn1stCall)
             {
                /* Two failovers were attempted before finally giving up */
-               TestCase.assertEquals(2, failoverCounter);
+               TestCase.assertEquals(Integer.valueOf(2), failoverCounter);
             }
             else
             {
                /* It's 1 because after several calls, the transaction has 
                 * already reached the server, so that's good enough not to 
                 * succeed */
-               TestCase.assertEquals(1, failoverCounter);
+               TestCase.assertEquals(Integer.valueOf(1), failoverCounter);
             }
             break;
             
          case AFTER_SERVER_COMPLETED:
             /* failover counters are always 0 because there's no chance of 
              * calculating failover at all */
-            TestCase.assertEquals(0, failoverCounter);
+            TestCase.assertEquals(Integer.valueOf(0), failoverCounter);
             break;
+            
+         default:
+            break;
       }
    }