You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by an...@apache.org on 2014/07/15 23:55:19 UTC

svn commit: r1610862 [26/44] - in /tomee/tomee/trunk/itests: ./ failover-ejb/ failover-ejb/src/main/java/org/apache/openejb/itest/failover/ejb/ failover/ failover/src/main/java/org/apache/openejb/itest/failover/ failover/src/main/java/org/apache/openej...

Modified: tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2HandleTests.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2HandleTests.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2HandleTests.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2HandleTests.java Tue Jul 15 21:55:09 2014
@@ -37,7 +37,7 @@ public class Cmp2HandleTests extends Bas
 
     protected void setUp() throws Exception {
         super.setUp();
-        Object obj = initialContext.lookup("client/tests/entity/cmp2/BasicCmpHome");
+        final Object obj = initialContext.lookup("client/tests/entity/cmp2/BasicCmpHome");
         ejbHome = (BasicCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, BasicCmpHome.class);
         ejbObject = ejbHome.createObject("Fifth Bean");
         ejbHandle = ejbObject.getHandle();
@@ -56,43 +56,43 @@ public class Cmp2HandleTests extends Bas
     public void test01_getEJBObject() {
 
         try {
-            EJBObject object = ejbHandle.getEJBObject();
+            final EJBObject object = ejbHandle.getEJBObject();
             assertNotNull("The EJBObject is null", object);
             // Wait until isIdentical is working.
             //assertTrue("EJBObjects are not identical", object.isIdentical(ejbObject));
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void Xtest02_copyHandleByMarshalledObject() {
         try {
-            MarshalledObject obj = new MarshalledObject(ejbHandle);
-            Handle copy = (Handle) obj.get();
+            final MarshalledObject obj = new MarshalledObject(ejbHandle);
+            final Handle copy = (Handle) obj.get();
 
-            EJBObject object = copy.getEJBObject();
+            final EJBObject object = copy.getEJBObject();
             assertNotNull("The EJBObject is null", object);
             assertTrue("EJBObjects are not identical", object.isIdentical(ejbObject));
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void Xtest03_copyHandleBySerialize() {
         try {
-            ByteArrayOutputStream baos = new ByteArrayOutputStream();
-            ObjectOutputStream oos = new ObjectOutputStream(baos);
+            final ByteArrayOutputStream baos = new ByteArrayOutputStream();
+            final ObjectOutputStream oos = new ObjectOutputStream(baos);
             oos.writeObject(ejbHandle);
             oos.flush();
             oos.close();
-            ByteArrayInputStream bais = new ByteArrayInputStream(baos.toByteArray());
-            ObjectInputStream ois = new ObjectInputStream(bais);
-            Handle copy = (Handle) ois.readObject();
+            final ByteArrayInputStream bais = new ByteArrayInputStream(baos.toByteArray());
+            final ObjectInputStream ois = new ObjectInputStream(bais);
+            final Handle copy = (Handle) ois.readObject();
 
-            EJBObject object = copy.getEJBObject();
+            final EJBObject object = copy.getEJBObject();
             assertNotNull("The EJBObject is null", object);
             assertTrue("EJBObjects are not identical", object.isIdentical(ejbObject));
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -108,11 +108,11 @@ public class Cmp2HandleTests extends Bas
             try {
                 ejbObject.businessMethod("Should throw an exception");
                 assertTrue("Calling business method after removing the EJBObject does not throw an exception", false);
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 assertTrue(true);
                 return;
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         } finally {
             ejbObject = null;

Modified: tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2HomeHandleTests.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2HomeHandleTests.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2HomeHandleTests.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2HomeHandleTests.java Tue Jul 15 21:55:09 2014
@@ -37,7 +37,7 @@ public class Cmp2HomeHandleTests extends
 
     protected void setUp() throws Exception {
         super.setUp();
-        Object obj = initialContext.lookup("client/tests/entity/cmp2/BasicCmpHome");
+        final Object obj = initialContext.lookup("client/tests/entity/cmp2/BasicCmpHome");
         ejbHome = (BasicCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, BasicCmpHome.class);
         ejbHomeHandle = ejbHome.getHomeHandle();
     }
@@ -47,41 +47,41 @@ public class Cmp2HomeHandleTests extends
     //
     public void test01_getEJBHome() {
         try {
-            EJBHome home = ejbHomeHandle.getEJBHome();
+            final EJBHome home = ejbHomeHandle.getEJBHome();
             assertNotNull("The EJBHome is null", home);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void Xtest02_copyHandleByMarshalledObject() {
         try {
-            MarshalledObject obj = new MarshalledObject(ejbHomeHandle);
-            HomeHandle copy = (HomeHandle) obj.get();
+            final MarshalledObject obj = new MarshalledObject(ejbHomeHandle);
+            final HomeHandle copy = (HomeHandle) obj.get();
 
             assertNotNull("The HomeHandle copy is null", copy);
-            EJBHome home = copy.getEJBHome();
+            final EJBHome home = copy.getEJBHome();
             assertNotNull("The EJBHome is null", home);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void Xtest03_copyHandleBySerialize() {
         try {
-            ByteArrayOutputStream baos = new ByteArrayOutputStream();
-            ObjectOutputStream oos = new ObjectOutputStream(baos);
+            final ByteArrayOutputStream baos = new ByteArrayOutputStream();
+            final ObjectOutputStream oos = new ObjectOutputStream(baos);
             oos.writeObject(ejbHomeHandle);
             oos.flush();
             oos.close();
-            ByteArrayInputStream bais = new ByteArrayInputStream(baos.toByteArray());
-            ObjectInputStream ois = new ObjectInputStream(bais);
-            HomeHandle copy = (HomeHandle) ois.readObject();
+            final ByteArrayInputStream bais = new ByteArrayInputStream(baos.toByteArray());
+            final ObjectInputStream ois = new ObjectInputStream(bais);
+            final HomeHandle copy = (HomeHandle) ois.readObject();
 
             assertNotNull("The HomeHandle copy is null", copy);
-            EJBHome home = copy.getEJBHome();
+            final EJBHome home = copy.getEJBHome();
             assertNotNull("The EJBHome is null", home);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }

Modified: tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2HomeIntfcTests.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2HomeIntfcTests.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2HomeIntfcTests.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2HomeIntfcTests.java Tue Jul 15 21:55:09 2014
@@ -31,7 +31,7 @@ public class Cmp2HomeIntfcTests extends 
 
     protected void setUp() throws Exception {
         super.setUp();
-        Object obj = initialContext.lookup("client/tests/entity/cmp2/BasicCmpHome");
+        final Object obj = initialContext.lookup("client/tests/entity/cmp2/BasicCmpHome");
         ejbHome = (BasicCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, BasicCmpHome.class);
     }
 
@@ -50,7 +50,7 @@ public class Cmp2HomeIntfcTests extends 
     }
 
     public void test03_findByLastName() throws Exception {
-        Integer[] keys = new Integer[3];
+        final Integer[] keys = new Integer[3];
         ejbObject = ejbHome.createObject("David Blevins");
         keys[0] = (Integer) ejbObject.getPrimaryKey();
 
@@ -60,10 +60,10 @@ public class Cmp2HomeIntfcTests extends 
         ejbObject = ejbHome.createObject("Claude Blevins");
         keys[2] = (Integer) ejbObject.getPrimaryKey();
 
-        java.util.Collection objects = ejbHome.findByLastName("Blevins");
+        final java.util.Collection objects = ejbHome.findByLastName("Blevins");
         assertNotNull("The Collection is null", objects);
         assertEquals("The Collection is not the right size.", keys.length, objects.size());
-        Object[] objs = objects.toArray();
+        final Object[] objs = objects.toArray();
         for (int i = 0; i < objs.length; i++) {
             ejbObject = (BasicCmpObject) javax.rmi.PortableRemoteObject.narrow(objs[i], BasicCmpObject.class);
             // This could be problematic, it assumes the order of the collection.
@@ -73,10 +73,10 @@ public class Cmp2HomeIntfcTests extends 
 
     public void test04_homeMethod() {
         try {
-            int expected = 8;
-            int actual = ejbHome.sum(5, 3);
+            final int expected = 8;
+            final int actual = ejbHome.sum(5, 3);
             assertEquals("home method returned wrong result", expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }

Modified: tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2JndiEncTests.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2JndiEncTests.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2JndiEncTests.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2JndiEncTests.java Tue Jul 15 21:55:09 2014
@@ -34,7 +34,7 @@ public class Cmp2JndiEncTests extends Cm
 
     protected void setUp() throws Exception {
         super.setUp();
-        Object obj = initialContext.lookup("client/tests/entity/cmp2/EncBean");
+        final Object obj = initialContext.lookup("client/tests/entity/cmp2/EncBean");
         ejbHome = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
         ejbObject = ejbHome.create("Enc Bean");
     }
@@ -42,7 +42,7 @@ public class Cmp2JndiEncTests extends Cm
     protected void tearDown() throws Exception {
         try {
             //ejbObject.remove();
-        } catch (Exception e) {
+        } catch (final Exception e) {
             throw e;
         } finally {
             super.tearDown();
@@ -52,9 +52,9 @@ public class Cmp2JndiEncTests extends Cm
     public void test01_lookupStringEntry() {
         try {
             ejbObject.lookupStringEntry();
-        } catch (TestFailureException e) {
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -62,9 +62,9 @@ public class Cmp2JndiEncTests extends Cm
     public void test02_lookupDoubleEntry() {
         try {
             ejbObject.lookupDoubleEntry();
-        } catch (TestFailureException e) {
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -72,9 +72,9 @@ public class Cmp2JndiEncTests extends Cm
     public void test03_lookupLongEntry() {
         try {
             ejbObject.lookupLongEntry();
-        } catch (TestFailureException e) {
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -82,9 +82,9 @@ public class Cmp2JndiEncTests extends Cm
     public void test04_lookupFloatEntry() {
         try {
             ejbObject.lookupFloatEntry();
-        } catch (TestFailureException e) {
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -92,9 +92,9 @@ public class Cmp2JndiEncTests extends Cm
     public void test05_lookupIntegerEntry() {
         try {
             ejbObject.lookupIntegerEntry();
-        } catch (TestFailureException e) {
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -102,9 +102,9 @@ public class Cmp2JndiEncTests extends Cm
     public void test06_lookupShortEntry() {
         try {
             ejbObject.lookupShortEntry();
-        } catch (TestFailureException e) {
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -112,9 +112,9 @@ public class Cmp2JndiEncTests extends Cm
     public void test07_lookupBooleanEntry() {
         try {
             ejbObject.lookupBooleanEntry();
-        } catch (TestFailureException e) {
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -122,9 +122,9 @@ public class Cmp2JndiEncTests extends Cm
     public void test08_lookupByteEntry() {
         try {
             ejbObject.lookupByteEntry();
-        } catch (TestFailureException e) {
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -132,9 +132,9 @@ public class Cmp2JndiEncTests extends Cm
     public void test09_lookupEntityBean() {
         try {
             ejbObject.lookupEntityBean();
-        } catch (TestFailureException e) {
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -142,9 +142,9 @@ public class Cmp2JndiEncTests extends Cm
     public void test10_lookupStatefulBean() {
         try {
             ejbObject.lookupStatefulBean();
-        } catch (TestFailureException e) {
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -152,9 +152,9 @@ public class Cmp2JndiEncTests extends Cm
     public void test11_lookupStatelessBean() {
         try {
             ejbObject.lookupStatelessBean();
-        } catch (TestFailureException e) {
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -162,80 +162,80 @@ public class Cmp2JndiEncTests extends Cm
     public void test12_lookupResource() {
         try {
             ejbObject.lookupResource();
-        } catch (TestFailureException e) {
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test13_lookupPersistenceUnit() {
-        try{
+        try {
             ejbObject.lookupPersistenceUnit();
-        } catch (TestFailureException e){
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e){
-            fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+        } catch (final Exception e) {
+            fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
-    
+
     public void test14_lookupPersistenceContext() {
-        try{
+        try {
             ejbObject.lookupPersistenceContext();
-        } catch (TestFailureException e){
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e){
-            fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+        } catch (final Exception e) {
+            fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test19_lookupStatelessBusinessLocal() {
-        try{
+        try {
             ejbObject.lookupStatelessBusinessLocal();
-        } catch (TestFailureException e){
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e){
-            fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+        } catch (final Exception e) {
+            fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test20_lookupStatelessBusinessRemote() {
-        try{
+        try {
             ejbObject.lookupStatelessBusinessRemote();
-        } catch (TestFailureException e){
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e){
-            fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+        } catch (final Exception e) {
+            fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test21_lookupStatefulBusinessLocal() {
-        try{
+        try {
             ejbObject.lookupStatefulBusinessLocal();
-        } catch (TestFailureException e){
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e){
-            fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+        } catch (final Exception e) {
+            fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test22_lookupStatefulBusinessRemote() {
-        try{
+        try {
             ejbObject.lookupStatefulBusinessRemote();
-        } catch (TestFailureException e){
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e){
-            fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+        } catch (final Exception e) {
+            fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
-    
+
     public void test23_lookupJMSConnectionFactory() {
-        try{
+        try {
             ejbObject.lookupJMSConnectionFactory();
-        } catch (TestFailureException e){
+        } catch (final TestFailureException e) {
             throw e.error;
-        } catch (Exception e){
-            fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+        } catch (final Exception e) {
+            fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 }

Modified: tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2JndiTests.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2JndiTests.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2JndiTests.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2JndiTests.java Tue Jul 15 21:55:09 2014
@@ -30,10 +30,10 @@ public class Cmp2JndiTests extends Basic
 
     public void test01_Jndi_lookupHome() {
         try {
-            Object obj = initialContext.lookup("client/tests/entity/cmp/BasicCmpHome");
+            final Object obj = initialContext.lookup("client/tests/entity/cmp/BasicCmpHome");
             ejbHome = (BasicCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, BasicCmpHome.class);
             assertNotNull("The EJBHome is null", ejbHome);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }

Modified: tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2RemoteIntfcTests.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2RemoteIntfcTests.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2RemoteIntfcTests.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2RemoteIntfcTests.java Tue Jul 15 21:55:09 2014
@@ -30,7 +30,7 @@ public class Cmp2RemoteIntfcTests extend
 
     protected void setUp() throws Exception {
         super.setUp();
-        Object obj = initialContext.lookup("client/tests/entity/cmp/BasicCmpHome");
+        final Object obj = initialContext.lookup("client/tests/entity/cmp/BasicCmpHome");
         ejbHome = (BasicCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, BasicCmpHome.class);
         ejbObject = ejbHome.createObject("Forth Bean");
     }
@@ -40,10 +40,10 @@ public class Cmp2RemoteIntfcTests extend
     //
     public void test01_businessMethod() {
         try {
-            String expected = "Success";
-            String actual = ejbObject.businessMethod("sseccuS");
+            final String expected = "Success";
+            final String actual = ejbObject.businessMethod("sseccuS");
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -55,10 +55,10 @@ public class Cmp2RemoteIntfcTests extend
     public void test02_throwApplicationException() {
         try {
             ejbObject.throwApplicationException();
-        } catch (org.apache.openejb.test.ApplicationException e) {
+        } catch (final org.apache.openejb.test.ApplicationException e) {
             //Good.  This is the correct behaviour
             return;
-        } catch (Throwable e) {
+        } catch (final Throwable e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
         fail("An ApplicationException should have been thrown.");
@@ -70,10 +70,10 @@ public class Cmp2RemoteIntfcTests extend
      */
     public void test03_invokeAfterApplicationException() {
         try {
-            String expected = "Success";
-            String actual = ejbObject.businessMethod("sseccuS");
+            final String expected = "Success";
+            final String actual = ejbObject.businessMethod("sseccuS");
             assertEquals(expected, actual);
-        } catch (Throwable e) {
+        } catch (final Throwable e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -81,13 +81,13 @@ public class Cmp2RemoteIntfcTests extend
     public void test04_throwSystemException() {
         try {
             ejbObject.throwSystemException_NullPointer();
-        } catch (java.rmi.RemoteException e) {
+        } catch (final java.rmi.RemoteException e) {
             //Good, so far.
-            Throwable n = e.detail;
+            final Throwable n = e.detail;
             assertNotNull("Nested exception should not be is null", n);
             assertTrue("Nested exception should be an instance of NullPointerException, but exception is " + n.getClass().getName(), (n instanceof NullPointerException));
             return;
-        } catch (Throwable e) {
+        } catch (final Throwable e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
         fail("A NullPointerException should have been thrown.");
@@ -103,9 +103,9 @@ public class Cmp2RemoteIntfcTests extend
         try {
             ejbObject.businessMethod("This refernce is invalid");
             fail("A java.rmi.NoSuchObjectException should have been thrown.");
-        } catch (java.rmi.NoSuchObjectException e) {
+        } catch (final java.rmi.NoSuchObjectException e) {
             // Good.
-        } catch (Throwable e) {
+        } catch (final Throwable e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }

Modified: tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2RmiIiopTests.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2RmiIiopTests.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2RmiIiopTests.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2RmiIiopTests.java Tue Jul 15 21:55:09 2014
@@ -40,7 +40,7 @@ public class Cmp2RmiIiopTests extends Cm
 
     protected void setUp() throws Exception {
         super.setUp();
-        Object obj = initialContext.lookup("client/tests/entity/cmp/RMI-over-IIOP/EJBHome");
+        final Object obj = initialContext.lookup("client/tests/entity/cmp/RMI-over-IIOP/EJBHome");
         ejbHome = (RmiIiopCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, RmiIiopCmpHome.class);
         ejbObject = ejbHome.create("RMI-IIOP TestBean");
     }
@@ -51,25 +51,25 @@ public class Cmp2RmiIiopTests extends Cm
 
     public void test01_returnStringObject() {
         try {
-            String expected = new String("1");
-            String actual = ejbObject.returnStringObject(expected);
+            final String expected = new String("1");
+            final String actual = ejbObject.returnStringObject(expected);
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test02_returnStringObjectArray() {
         try {
-            String[] expected = {"1", "2", "3"};
-            String[] actual = ejbObject.returnStringObjectArray(expected);
+            final String[] expected = {"1", "2", "3"};
+            final String[] actual = ejbObject.returnStringObjectArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -81,50 +81,50 @@ public class Cmp2RmiIiopTests extends Cm
 
     public void test03_returnCharacterObject() {
         try {
-            Character expected = new Character('1');
-            Character actual = ejbObject.returnCharacterObject(expected);
+            final Character expected = new Character('1');
+            final Character actual = ejbObject.returnCharacterObject(expected);
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test04_returnCharacterPrimitive() {
         try {
-            char expected = '1';
-            char actual = ejbObject.returnCharacterPrimitive(expected);
+            final char expected = '1';
+            final char actual = ejbObject.returnCharacterPrimitive(expected);
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test05_returnCharacterObjectArray() {
         try {
-            Character[] expected = {new Character('1'), new Character('2'), new Character('3')};
-            Character[] actual = ejbObject.returnCharacterObjectArray(expected);
+            final Character[] expected = {new Character('1'), new Character('2'), new Character('3')};
+            final Character[] actual = ejbObject.returnCharacterObjectArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test06_returnCharacterPrimitiveArray() {
         try {
-            char[] expected = {'1', '2', '3'};
-            char[] actual = ejbObject.returnCharacterPrimitiveArray(expected);
+            final char[] expected = {'1', '2', '3'};
+            final char[] actual = ejbObject.returnCharacterPrimitiveArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -136,50 +136,50 @@ public class Cmp2RmiIiopTests extends Cm
 
     public void test07_returnBooleanObject() {
         try {
-            Boolean expected = new Boolean(true);
-            Boolean actual = ejbObject.returnBooleanObject(expected);
+            final Boolean expected = new Boolean(true);
+            final Boolean actual = ejbObject.returnBooleanObject(expected);
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test08_returnBooleanPrimitive() {
         try {
-            boolean expected = true;
-            boolean actual = ejbObject.returnBooleanPrimitive(expected);
+            final boolean expected = true;
+            final boolean actual = ejbObject.returnBooleanPrimitive(expected);
             assertEquals("" + expected, "" + actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test09_returnBooleanObjectArray() {
         try {
-            Boolean[] expected = {new Boolean(true), new Boolean(false), new Boolean(true)};
-            Boolean[] actual = ejbObject.returnBooleanObjectArray(expected);
+            final Boolean[] expected = {new Boolean(true), new Boolean(false), new Boolean(true)};
+            final Boolean[] actual = ejbObject.returnBooleanObjectArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test10_returnBooleanPrimitiveArray() {
         try {
-            boolean[] expected = {false, true, true};
-            boolean[] actual = ejbObject.returnBooleanPrimitiveArray(expected);
+            final boolean[] expected = {false, true, true};
+            final boolean[] actual = ejbObject.returnBooleanPrimitiveArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -191,50 +191,50 @@ public class Cmp2RmiIiopTests extends Cm
 
     public void test11_returnByteObject() {
         try {
-            Byte expected = new Byte("1");
-            Byte actual = ejbObject.returnByteObject(expected);
+            final Byte expected = new Byte("1");
+            final Byte actual = ejbObject.returnByteObject(expected);
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test12_returnBytePrimitive() {
         try {
-            byte expected = (byte) 1;
-            byte actual = ejbObject.returnBytePrimitive(expected);
+            final byte expected = (byte) 1;
+            final byte actual = ejbObject.returnBytePrimitive(expected);
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test13_returnByteObjectArray() {
         try {
-            Byte[] expected = {new Byte("1"), new Byte("2"), new Byte("3")};
-            Byte[] actual = ejbObject.returnByteObjectArray(expected);
+            final Byte[] expected = {new Byte("1"), new Byte("2"), new Byte("3")};
+            final Byte[] actual = ejbObject.returnByteObjectArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test14_returnBytePrimitiveArray() {
         try {
-            byte[] expected = {(byte) 1, (byte) 2, (byte) 3};
-            byte[] actual = ejbObject.returnBytePrimitiveArray(expected);
+            final byte[] expected = {(byte) 1, (byte) 2, (byte) 3};
+            final byte[] actual = ejbObject.returnBytePrimitiveArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -246,50 +246,50 @@ public class Cmp2RmiIiopTests extends Cm
 
     public void test15_returnShortObject() {
         try {
-            Short expected = new Short("1");
-            Short actual = ejbObject.returnShortObject(expected);
+            final Short expected = new Short("1");
+            final Short actual = ejbObject.returnShortObject(expected);
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test16_returnShortPrimitive() {
         try {
-            short expected = (short) 1;
-            short actual = ejbObject.returnShortPrimitive(expected);
+            final short expected = (short) 1;
+            final short actual = ejbObject.returnShortPrimitive(expected);
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test17_returnShortObjectArray() {
         try {
-            Short[] expected = {new Short("1"), new Short("2"), new Short("3")};
-            Short[] actual = ejbObject.returnShortObjectArray(expected);
+            final Short[] expected = {new Short("1"), new Short("2"), new Short("3")};
+            final Short[] actual = ejbObject.returnShortObjectArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test18_returnShortPrimitiveArray() {
         try {
-            short[] expected = {(short) 1, (short) 2, (short) 3};
-            short[] actual = ejbObject.returnShortPrimitiveArray(expected);
+            final short[] expected = {(short) 1, (short) 2, (short) 3};
+            final short[] actual = ejbObject.returnShortPrimitiveArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -301,50 +301,50 @@ public class Cmp2RmiIiopTests extends Cm
 
     public void test19_returnIntegerObject() {
         try {
-            Integer expected = new Integer(1);
-            Integer actual = ejbObject.returnIntegerObject(expected);
+            final Integer expected = new Integer(1);
+            final Integer actual = ejbObject.returnIntegerObject(expected);
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test20_returnIntegerPrimitive() {
         try {
-            int expected = 1;
-            int actual = ejbObject.returnIntegerPrimitive(expected);
+            final int expected = 1;
+            final int actual = ejbObject.returnIntegerPrimitive(expected);
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test21_returnIntegerObjectArray() {
         try {
-            Integer[] expected = {new Integer(1), new Integer(2), new Integer(3)};
-            Integer[] actual = ejbObject.returnIntegerObjectArray(expected);
+            final Integer[] expected = {new Integer(1), new Integer(2), new Integer(3)};
+            final Integer[] actual = ejbObject.returnIntegerObjectArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test22_returnIntegerPrimitiveArray() {
         try {
-            int[] expected = {1, 2, 3};
-            int[] actual = ejbObject.returnIntegerPrimitiveArray(expected);
+            final int[] expected = {1, 2, 3};
+            final int[] actual = ejbObject.returnIntegerPrimitiveArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -356,50 +356,50 @@ public class Cmp2RmiIiopTests extends Cm
 
     public void test23_returnLongObject() {
         try {
-            Long expected = new Long("1");
-            Long actual = ejbObject.returnLongObject(expected);
+            final Long expected = new Long("1");
+            final Long actual = ejbObject.returnLongObject(expected);
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test24_returnLongPrimitive() {
         try {
-            long expected = 1;
-            long actual = ejbObject.returnLongPrimitive(expected);
+            final long expected = 1;
+            final long actual = ejbObject.returnLongPrimitive(expected);
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test25_returnLongObjectArray() {
         try {
-            Long[] expected = {new Long("1"), new Long("2"), new Long("3")};
-            Long[] actual = ejbObject.returnLongObjectArray(expected);
+            final Long[] expected = {new Long("1"), new Long("2"), new Long("3")};
+            final Long[] actual = ejbObject.returnLongObjectArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test26_returnLongPrimitiveArray() {
         try {
-            long[] expected = {1, 2, 3};
-            long[] actual = ejbObject.returnLongPrimitiveArray(expected);
+            final long[] expected = {1, 2, 3};
+            final long[] actual = ejbObject.returnLongPrimitiveArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -411,50 +411,50 @@ public class Cmp2RmiIiopTests extends Cm
 
     public void test27_returnFloatObject() {
         try {
-            Float expected = new Float("1.3");
-            Float actual = ejbObject.returnFloatObject(expected);
+            final Float expected = new Float("1.3");
+            final Float actual = ejbObject.returnFloatObject(expected);
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test28_returnFloatPrimitive() {
         try {
-            float expected = 1.2F;
-            float actual = ejbObject.returnFloatPrimitive(expected);
+            final float expected = 1.2F;
+            final float actual = ejbObject.returnFloatPrimitive(expected);
             assertEquals(expected, actual, 0.00D);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test29_returnFloatObjectArray() {
         try {
-            Float[] expected = {new Float("1.1"), new Float("2.2"), new Float("3.3")};
-            Float[] actual = ejbObject.returnFloatObjectArray(expected);
+            final Float[] expected = {new Float("1.1"), new Float("2.2"), new Float("3.3")};
+            final Float[] actual = ejbObject.returnFloatObjectArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test30_returnFloatPrimitiveArray() {
         try {
-            float[] expected = {1.2F, 2.3F, 3.4F};
-            float[] actual = ejbObject.returnFloatPrimitiveArray(expected);
+            final float[] expected = {1.2F, 2.3F, 3.4F};
+            final float[] actual = ejbObject.returnFloatPrimitiveArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i], 0.0D);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -466,50 +466,50 @@ public class Cmp2RmiIiopTests extends Cm
 
     public void test31_returnDoubleObject() {
         try {
-            Double expected = new Double("1.1");
-            Double actual = ejbObject.returnDoubleObject(expected);
+            final Double expected = new Double("1.1");
+            final Double actual = ejbObject.returnDoubleObject(expected);
             assertEquals(expected, actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test32_returnDoublePrimitive() {
         try {
-            double expected = 1.2;
-            double actual = ejbObject.returnDoublePrimitive(expected);
+            final double expected = 1.2;
+            final double actual = ejbObject.returnDoublePrimitive(expected);
             assertEquals(expected, actual, 0.0D);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test33_returnDoubleObjectArray() {
         try {
-            Double[] expected = {new Double("1.3"), new Double("2.4"), new Double("3.5")};
-            Double[] actual = ejbObject.returnDoubleObjectArray(expected);
+            final Double[] expected = {new Double("1.3"), new Double("2.4"), new Double("3.5")};
+            final Double[] actual = ejbObject.returnDoubleObjectArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i]);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test34_returnDoublePrimitiveArray() {
         try {
-            double[] expected = {1.4, 2.5, 3.6};
-            double[] actual = ejbObject.returnDoublePrimitiveArray(expected);
+            final double[] expected = {1.4, 2.5, 3.6};
+            final double[] actual = ejbObject.returnDoublePrimitiveArray(expected);
 
             assertNotNull("The array returned is null", actual);
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < actual.length; i++) {
                 assertEquals("Array values are not equal at index " + i, expected[i], actual[i], 0.0D);
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -521,52 +521,52 @@ public class Cmp2RmiIiopTests extends Cm
 
     public void test35_returnEJBHome() {
         try {
-            Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
-            EncCmpHome expected = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
+            final Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
+            final EncCmpHome expected = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
             assertNotNull("The EJBHome returned from JNDI is null", expected);
 
-            EncCmpHome actual = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(ejbObject.returnEJBHome(expected), EncCmpHome.class);
+            final EncCmpHome actual = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(ejbObject.returnEJBHome(expected), EncCmpHome.class);
             assertNotNull("The EJBHome returned is null", actual);
 
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test36_returnEJBHome2() {
         try {
-            EncCmpHome actual = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(ejbObject.returnEJBHome(), EncCmpHome.class);
+            final EncCmpHome actual = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(ejbObject.returnEJBHome(), EncCmpHome.class);
             assertNotNull("The EJBHome returned is null", actual);
 
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test37_returnNestedEJBHome() {
         try {
-            Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
-            EncCmpHome expected = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
+            final Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
+            final EncCmpHome expected = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
             assertNotNull("The EJBHome returned from JNDI is null", expected);
 
-            ObjectGraph graph = ejbObject.returnObjectGraph(new ObjectGraph(expected));
+            final ObjectGraph graph = ejbObject.returnObjectGraph(new ObjectGraph(expected));
             assertNotNull("The ObjectGraph is null", graph);
 
-            EncCmpHome actual = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(graph.getObject(), EncCmpHome.class);
+            final EncCmpHome actual = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(graph.getObject(), EncCmpHome.class);
             assertNotNull("The EJBHome returned is null", actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test38_returnNestedEJBHome2() {
         try {
-            ObjectGraph graph = ejbObject.returnNestedEJBHome();
+            final ObjectGraph graph = ejbObject.returnNestedEJBHome();
             assertNotNull("The ObjectGraph is null", graph);
 
-            EncCmpHome actual = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(graph.getObject(), EncCmpHome.class);
+            final EncCmpHome actual = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(graph.getObject(), EncCmpHome.class);
             assertNotNull("The EJBHome returned is null", actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -574,18 +574,18 @@ public class Cmp2RmiIiopTests extends Cm
     public void Xtest39_returnEJBHomeArray() {
         try {
 
-            EncCmpHome expected[] = new EncCmpHome[3];
+            final EncCmpHome[] expected = new EncCmpHome[3];
             for (int i = 0; i < expected.length; i++) {
-                Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
+                final Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
                 expected[i] = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
                 assertNotNull("The EJBHome returned from JNDI is null", expected[i]);
             }
 
-            EJBHome[] actual = ejbObject.returnEJBHomeArray(expected);
+            final EJBHome[] actual = ejbObject.returnEJBHomeArray(expected);
             assertNotNull("The EJBHome array returned is null", actual);
             assertEquals(expected.length, actual.length);
 
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -597,78 +597,78 @@ public class Cmp2RmiIiopTests extends Cm
 
     public void test40_returnEJBObject() {
         try {
-            Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
-            EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
+            final Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
+            final EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
             assertNotNull("The EJBHome returned from JNDI is null", home);
 
-            EncCmpObject expected = home.create("test_40 CmpBean");
+            final EncCmpObject expected = home.create("test_40 CmpBean");
             assertNotNull("The EJBObject created is null", expected);
 
-            EncCmpObject actual = (EncCmpObject) javax.rmi.PortableRemoteObject.narrow(ejbObject.returnEJBObject(expected), EncCmpObject.class);
+            final EncCmpObject actual = (EncCmpObject) javax.rmi.PortableRemoteObject.narrow(ejbObject.returnEJBObject(expected), EncCmpObject.class);
             assertNotNull("The EJBObject returned is null", actual);
 
             assertTrue("The EJBObejcts are not identical", expected.isIdentical(actual));
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test41_returnEJBObject2() {
         try {
-            EncCmpObject actual = (EncCmpObject) javax.rmi.PortableRemoteObject.narrow(ejbObject.returnEJBObject(), EncCmpObject.class);
+            final EncCmpObject actual = (EncCmpObject) javax.rmi.PortableRemoteObject.narrow(ejbObject.returnEJBObject(), EncCmpObject.class);
             assertNotNull("The EJBObject returned is null", actual);
 
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test42_returnNestedEJBObject() {
         try {
-            Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
-            EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
+            final Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
+            final EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
             assertNotNull("The EJBHome returned from JNDI is null", home);
 
-            EncCmpObject expected = home.create("test_42 CmpBean");
+            final EncCmpObject expected = home.create("test_42 CmpBean");
             assertNotNull("The EJBObject created is null", expected);
 
-            ObjectGraph graph = ejbObject.returnObjectGraph(new ObjectGraph(expected));
+            final ObjectGraph graph = ejbObject.returnObjectGraph(new ObjectGraph(expected));
             assertNotNull("The ObjectGraph is null", graph);
 
-            EncCmpObject actual = (EncCmpObject) javax.rmi.PortableRemoteObject.narrow(graph.getObject(), EncCmpObject.class);
+            final EncCmpObject actual = (EncCmpObject) javax.rmi.PortableRemoteObject.narrow(graph.getObject(), EncCmpObject.class);
             assertNotNull("The EJBObject returned is null", actual);
 
             assertTrue("The EJBObejcts are not identical", expected.isIdentical(actual));
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test43_returnNestedEJBObject2() {
         try {
-            ObjectGraph graph = ejbObject.returnNestedEJBObject();
+            final ObjectGraph graph = ejbObject.returnNestedEJBObject();
             assertNotNull("The ObjectGraph is null", graph);
 
-            EncCmpObject actual = (EncCmpObject) javax.rmi.PortableRemoteObject.narrow(graph.getObject(), EncCmpObject.class);
+            final EncCmpObject actual = (EncCmpObject) javax.rmi.PortableRemoteObject.narrow(graph.getObject(), EncCmpObject.class);
             assertNotNull("The EJBHome returned is null", actual);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void Xtest44_returnEJBObjectArray() {
         try {
-            Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
-            EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
+            final Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
+            final EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
             assertNotNull("The EJBHome returned from JNDI is null", home);
 
-            EncCmpObject expected[] = new EncCmpObject[3];
+            final EncCmpObject[] expected = new EncCmpObject[3];
             for (int i = 0; i < expected.length; i++) {
                 expected[i] = home.create("test_44 CmpBean");
                 assertNotNull("The EJBObject created is null", expected[i]);
             }
 
-            EJBObject[] actual = ejbObject.returnEJBObjectArray(expected);
+            final EJBObject[] actual = ejbObject.returnEJBObjectArray(expected);
             assertNotNull("The EJBObject array returned is null", actual);
             assertEquals(expected.length, actual.length);
 
@@ -676,7 +676,7 @@ public class Cmp2RmiIiopTests extends Cm
                 assertTrue("The EJBObejcts are not identical", expected[i].isIdentical(actual[i]));
             }
 
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -687,64 +687,64 @@ public class Cmp2RmiIiopTests extends Cm
 
     public void test45_returnEJBMetaData() {
         try {
-            Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
-            EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
+            final Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
+            final EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
             assertNotNull("The EJBHome returned from JNDI is null", home);
 
-            EJBMetaData expected = home.getEJBMetaData();
+            final EJBMetaData expected = home.getEJBMetaData();
             assertNotNull("The EJBMetaData returned is null", expected);
 
-            EJBMetaData actual = ejbObject.returnEJBMetaData(expected);
+            final EJBMetaData actual = ejbObject.returnEJBMetaData(expected);
             assertNotNull("The EJBMetaData returned is null", actual);
             assertEquals(expected.getHomeInterfaceClass(), actual.getHomeInterfaceClass());
             assertEquals(expected.getRemoteInterfaceClass(), actual.getRemoteInterfaceClass());
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test46_returnEJBMetaData() {
         try {
-            EJBMetaData actual = ejbObject.returnEJBMetaData();
+            final EJBMetaData actual = ejbObject.returnEJBMetaData();
             assertNotNull("The EJBMetaData returned is null", actual);
             assertEquals(actual.getHomeInterfaceClass(), actual.getHomeInterfaceClass());
             assertEquals(actual.getRemoteInterfaceClass(), actual.getRemoteInterfaceClass());
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test47_returnNestedEJBMetaData() {
         try {
-            Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
-            EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
+            final Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
+            final EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
             assertNotNull("The EJBHome returned from JNDI is null", home);
 
-            EJBMetaData expected = home.getEJBMetaData();
+            final EJBMetaData expected = home.getEJBMetaData();
             assertNotNull("The EJBMetaData returned is null", expected);
 
-            ObjectGraph graph = ejbObject.returnObjectGraph(new ObjectGraph(expected));
+            final ObjectGraph graph = ejbObject.returnObjectGraph(new ObjectGraph(expected));
             assertNotNull("The ObjectGraph is null", graph);
 
-            EJBMetaData actual = (EJBMetaData) graph.getObject();
+            final EJBMetaData actual = (EJBMetaData) graph.getObject();
             assertNotNull("The EJBMetaData returned is null", actual);
             assertEquals(expected.getHomeInterfaceClass(), actual.getHomeInterfaceClass());
             assertEquals(expected.getRemoteInterfaceClass(), actual.getRemoteInterfaceClass());
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test48_returnNestedEJBMetaData2() {
         try {
-            ObjectGraph graph = ejbObject.returnNestedEJBMetaData();
+            final ObjectGraph graph = ejbObject.returnNestedEJBMetaData();
             assertNotNull("The ObjectGraph is null", graph);
 
-            EJBMetaData actual = (EJBMetaData) graph.getObject();
+            final EJBMetaData actual = (EJBMetaData) graph.getObject();
             assertNotNull("The EJBMetaData returned is null", actual);
             assertNotNull("The home interface class of the EJBMetaData is null", actual.getHomeInterfaceClass());
             assertNotNull("The remote interface class of the EJBMetaData is null", actual.getRemoteInterfaceClass());
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -752,17 +752,17 @@ public class Cmp2RmiIiopTests extends Cm
     public void test49_returnEJBMetaDataArray() {
         try {
 
-            Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
-            EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
+            final Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
+            final EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
             assertNotNull("The EJBHome returned from JNDI is null", home);
 
-            EJBMetaData expected[] = new EJBMetaData[3];
+            final EJBMetaData[] expected = new EJBMetaData[3];
             for (int i = 0; i < expected.length; i++) {
                 expected[i] = home.getEJBMetaData();
                 assertNotNull("The EJBMetaData returned is null", expected[i]);
             }
 
-            EJBMetaData[] actual = ejbObject.returnEJBMetaDataArray(expected);
+            final EJBMetaData[] actual = ejbObject.returnEJBMetaDataArray(expected);
             assertNotNull("The EJBMetaData array returned is null", actual);
             assertEquals(expected.length, actual.length);
 
@@ -771,7 +771,7 @@ public class Cmp2RmiIiopTests extends Cm
                 assertEquals(expected[i].getHomeInterfaceClass(), actual[i].getHomeInterfaceClass());
                 assertEquals(expected[i].getRemoteInterfaceClass(), actual[i].getRemoteInterfaceClass());
             }
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -783,100 +783,100 @@ public class Cmp2RmiIiopTests extends Cm
 
     public void test50_returnHandle() {
         try {
-            Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
-            EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
+            final Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
+            final EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
             assertNotNull("The EJBHome returned from JNDI is null", home);
 
-            EncCmpObject object = home.create("test_50 CmpBean");
+            final EncCmpObject object = home.create("test_50 CmpBean");
             assertNotNull("The EJBObject created is null", object);
 
-            Handle expected = object.getHandle();
+            final Handle expected = object.getHandle();
             assertNotNull("The EJBObject Handle returned is null", expected);
             assertNotNull("The EJBObject in the Handle is null", expected.getEJBObject());
 
-            Handle actual = ejbObject.returnHandle(expected);
+            final Handle actual = ejbObject.returnHandle(expected);
             assertNotNull("The EJBObject Handle returned is null", actual);
             assertNotNull("The EJBObject in the Handle is null", actual.getEJBObject());
 
-            EJBObject exp = expected.getEJBObject();
-            EJBObject act = actual.getEJBObject();
+            final EJBObject exp = expected.getEJBObject();
+            final EJBObject act = actual.getEJBObject();
 
             assertTrue("The EJBObjects in the Handles are not identical", exp.isIdentical(act));
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test51_returnHandle() {
         try {
-            Handle actual = ejbObject.returnHandle();
+            final Handle actual = ejbObject.returnHandle();
             assertNotNull("The EJBObject Handle returned is null", actual);
             assertNotNull("The EJBObject in the Handle is null", actual.getEJBObject());
 
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test52_returnNestedHandle() {
         try {
-            Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
-            EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
+            final Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
+            final EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
             assertNotNull("The EJBHome returned from JNDI is null", home);
 
-            EncCmpObject object = home.create("test_52 CmpBean");
+            final EncCmpObject object = home.create("test_52 CmpBean");
             assertNotNull("The EJBObject created is null", object);
 
-            Handle expected = object.getHandle();
+            final Handle expected = object.getHandle();
             assertNotNull("The EJBObject Handle returned is null", expected);
             assertNotNull("The EJBObject in the Handle is null", expected.getEJBObject());
 
-            ObjectGraph graph = ejbObject.returnObjectGraph(new ObjectGraph(expected));
+            final ObjectGraph graph = ejbObject.returnObjectGraph(new ObjectGraph(expected));
             assertNotNull("The ObjectGraph is null", graph);
 
-            Handle actual = (Handle) graph.getObject();
+            final Handle actual = (Handle) graph.getObject();
             assertNotNull("The EJBObject Handle returned is null", actual);
             assertNotNull("The EJBObject in the Handle is null", actual.getEJBObject());
 
-            EJBObject exp = expected.getEJBObject();
-            EJBObject act = actual.getEJBObject();
+            final EJBObject exp = expected.getEJBObject();
+            final EJBObject act = actual.getEJBObject();
 
             assertTrue("The EJBObjects in the Handles are not identical", exp.isIdentical(act));
 
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test53_returnNestedHandle2() {
         try {
-            ObjectGraph graph = ejbObject.returnNestedHandle();
+            final ObjectGraph graph = ejbObject.returnNestedHandle();
             assertNotNull("The ObjectGraph is null", graph);
 
-            Handle actual = (Handle) graph.getObject();
+            final Handle actual = (Handle) graph.getObject();
             assertNotNull("The EJBObject Handle returned is null", actual);
             assertNotNull("The EJBObject in the Handle is null", actual.getEJBObject());
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test54_returnHandleArray() {
         try {
-            Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
-            EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
+            final Object obj = initialContext.lookup("client/tests/entity/cmp/EncBean");
+            final EncCmpHome home = (EncCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, EncCmpHome.class);
             assertNotNull("The EJBHome returned from JNDI is null", home);
 
-            EncCmpObject object = home.create("test_54 CmpBean");
+            final EncCmpObject object = home.create("test_54 CmpBean");
             assertNotNull("The EJBObject created is null", object);
 
-            Handle expected[] = new Handle[3];
+            final Handle[] expected = new Handle[3];
             for (int i = 0; i < expected.length; i++) {
                 expected[i] = object.getHandle();
                 assertNotNull("The EJBObject Handle returned is null", expected[i]);
             }
 
-            Handle[] actual = ejbObject.returnHandleArray(expected);
+            final Handle[] actual = ejbObject.returnHandleArray(expected);
             assertNotNull("The Handle array returned is null", actual);
             assertEquals(expected.length, actual.length);
 
@@ -886,7 +886,7 @@ public class Cmp2RmiIiopTests extends Cm
                 assertTrue("The EJBObjects in the Handles are not equal", expected[i].getEJBObject().isIdentical(actual[i].getEJBObject()));
             }
 
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -907,47 +907,47 @@ public class Cmp2RmiIiopTests extends Cm
 /*-------------------------------------------------*/
 
     public void test57_returnClass() {
-        Class[] primitives = {boolean.class, byte.class, char.class, short.class, int.class, long.class, float.class, double.class};
-        for (Class expected : primitives) {
+        final Class[] primitives = {boolean.class, byte.class, char.class, short.class, int.class, long.class, float.class, double.class};
+        for (final Class expected : primitives) {
             try {
-                Class actual = ejbObject.returnClass(expected);
+                final Class actual = ejbObject.returnClass(expected);
                 assertEquals(expected, actual);
-            } catch (Exception e) {
-                fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+            } catch (final Exception e) {
+                fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
         }
     }
 
     public void test58_returnClassArray() {
         try {
-            Class[] expected = {boolean.class, byte.class, char.class, short.class, int.class, long.class, float.class, double.class};
-            Class[] actual = ejbObject.returnClassArray(expected);
+            final Class[] expected = {boolean.class, byte.class, char.class, short.class, int.class, long.class, float.class, double.class};
+            final Class[] actual = ejbObject.returnClassArray(expected);
 
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < expected.length; i++) {
                 assertEquals(expected[i], actual[i]);
             }
-        } catch (RemoteException e) {
-            fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+        } catch (final RemoteException e) {
+            fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test59_returnClassObjectGraph() {
         try {
-            Class[] primitives = {boolean.class, byte.class, char.class, short.class, int.class, long.class, float.class, double.class};
-            ObjectGraph expectedGraph = new ObjectGraph(primitives);
+            final Class[] primitives = {boolean.class, byte.class, char.class, short.class, int.class, long.class, float.class, double.class};
+            final ObjectGraph expectedGraph = new ObjectGraph(primitives);
 
-            ObjectGraph actualGraph = ejbObject.returnObjectGraph(expectedGraph);
+            final ObjectGraph actualGraph = ejbObject.returnObjectGraph(expectedGraph);
 
-            Class[] expected = (Class[]) expectedGraph.getObject();
-            Class[] actual = (Class[]) actualGraph.getObject();
+            final Class[] expected = (Class[]) expectedGraph.getObject();
+            final Class[] actual = (Class[]) actualGraph.getObject();
 
             assertEquals(expected.length, actual.length);
             for (int i = 0; i < expected.length; i++) {
                 assertEquals(expected[i], actual[i]);
             }
-        } catch (RemoteException e) {
-            fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+        } catch (final RemoteException e) {
+            fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 

Modified: tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2TestClient.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2TestClient.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2TestClient.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2TestClient.java Tue Jul 15 21:55:09 2014
@@ -34,7 +34,7 @@ public abstract class Cmp2TestClient ext
     protected Handle ejbHandle;
     protected Object ejbPrimaryKey;
 
-    public Cmp2TestClient(String name) {
+    public Cmp2TestClient(final String name) {
         super("Entity.CMP." + name);
     }
 
@@ -44,7 +44,7 @@ public abstract class Cmp2TestClient ext
      */
     protected void setUp() throws Exception {
 
-        Properties properties = TestManager.getServer().getContextEnvironment();
+        final Properties properties = TestManager.getServer().getContextEnvironment();
         //properties.put(Context.SECURITY_PRINCIPAL, "ENTITY_TEST_CLIENT");
         //properties.put(Context.SECURITY_CREDENTIALS, "ENTITY_TEST_CLIENT");
 

Modified: tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2TestSuite.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2TestSuite.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2TestSuite.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Cmp2TestSuite.java Tue Jul 15 21:55:09 2014
@@ -67,7 +67,7 @@ public class Cmp2TestSuite extends org.a
      * This method is called before a test is executed.
      */
     protected void setUp() throws Exception {
-        Properties props = TestManager.getServer().getContextEnvironment();
+        final Properties props = TestManager.getServer().getContextEnvironment();
         //props.put(Context.SECURITY_PRINCIPAL, "ENTITY_TEST_CLIENT");
         //props.put(Context.SECURITY_CREDENTIALS, "ENTITY_TEST_CLIENT");
         new InitialContext(props);

Modified: tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Complex2EjbHomeTests.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Complex2EjbHomeTests.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Complex2EjbHomeTests.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-client/src/main/java/org/apache/openejb/test/entity/cmp2/Complex2EjbHomeTests.java Tue Jul 15 21:55:09 2014
@@ -34,7 +34,7 @@ public class Complex2EjbHomeTests extend
 
     protected void setUp() throws Exception {
         super.setUp();
-        Object obj = initialContext.lookup("client/tests/entity/cmp2/ComplexCmpHome");
+        final Object obj = initialContext.lookup("client/tests/entity/cmp2/ComplexCmpHome");
         ejbHome = (ComplexCmpHome) javax.rmi.PortableRemoteObject.narrow(obj, ComplexCmpHome.class);
         ejbObject = ejbHome.createObject("Second Bean");
         ejbPrimaryKey = ejbObject.getPrimaryKey();
@@ -45,9 +45,9 @@ public class Complex2EjbHomeTests extend
     //
     public void test01_getEJBMetaData() {
         try {
-            EJBMetaData ejbMetaData = ejbHome.getEJBMetaData();
+            final EJBMetaData ejbMetaData = ejbHome.getEJBMetaData();
             assertNotNull("The EJBMetaData is null", ejbMetaData);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
@@ -56,13 +56,13 @@ public class Complex2EjbHomeTests extend
         try {
             ejbHomeHandle = ejbHome.getHomeHandle();
             assertNotNull("The HomeHandle is null", ejbHomeHandle);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
-    public void test03_removeByPrimaryKey(){
-        try{
+    public void test03_removeByPrimaryKey() {
+        try {
             // remove the ejb
             ejbHome.remove(ejbPrimaryKey);
 
@@ -70,26 +70,26 @@ public class Complex2EjbHomeTests extend
             try {
                 ejbHome.findByPrimaryKey((ComplexCmpBeanPk) ejbPrimaryKey);
                 fail("Entity was not actually removed");
-            } catch (ObjectNotFoundException e) {
+            } catch (final ObjectNotFoundException e) {
             }
 
             // verify the proxy is dead
-            try{
+            try {
                 ejbObject.businessMethod("Should throw an exception");
-                assertTrue( "Calling business method after removing the EJBObject does not throw an exception", false );
-            } catch (Exception e){
+                assertTrue("Calling business method after removing the EJBObject does not throw an exception", false);
+            } catch (final Exception e) {
             }
 
             // create a new ejb for the next test
             ejbObject = ejbHome.createObject("Second Bean");
             ejbPrimaryKey = ejbObject.getPrimaryKey();
-        } catch (Exception e){
-            fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+        } catch (final Exception e) {
+            fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
-    public void test04_removeByPrimaryHandle(){
-        try{
+    public void test04_removeByPrimaryHandle() {
+        try {
             // remove the ejb
             ejbHome.remove(ejbObject.getHandle());
 
@@ -97,29 +97,29 @@ public class Complex2EjbHomeTests extend
             try {
                 ejbHome.findByPrimaryKey((ComplexCmpBeanPk) ejbPrimaryKey);
                 fail("Entity was not actually removed");
-            } catch (ObjectNotFoundException e) {
+            } catch (final ObjectNotFoundException e) {
             }
 
             // verify the proxy is dead
-            try{
+            try {
                 ejbObject.businessMethod("Should throw an exception");
-                assertTrue( "Calling business method after removing the EJBObject does not throw an exception", false );
-            } catch (Exception e){
+                assertTrue("Calling business method after removing the EJBObject does not throw an exception", false);
+            } catch (final Exception e) {
             }
 
             // create a new ejb for the next test
             ejbObject = ejbHome.createObject("Second Bean");
             ejbPrimaryKey = ejbObject.getPrimaryKey();
-        } catch (Exception e){
+        } catch (final Exception e) {
             e.printStackTrace();
-            fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+            fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }
     }
 
     public void test05_ejbHomeMethod() {
         try {
-            assertEquals(8+9, ejbHome.sum(8, 9));
-        } catch (Throwable e) {
+            assertEquals(8 + 9, ejbHome.sum(8, 9));
+        } catch (final Throwable e) {
             e.printStackTrace();
             fail("Received Exception " + e.getClass() + " : " + e.getMessage());
         }