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 [4/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/openejb...

Modified: tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/EncBmpBean.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/EncBmpBean.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/EncBmpBean.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/EncBmpBean.java Tue Jul 15 21:55:09 2014
@@ -47,7 +47,7 @@ import org.apache.openejb.test.stateless
 import org.apache.openejb.test.stateless.BasicStatelessBusinessLocal;
 import org.apache.openejb.test.stateless.BasicStatelessBusinessRemote;
 
-public class EncBmpBean implements javax.ejb.EntityBean{
+public class EncBmpBean implements javax.ejb.EntityBean {
 
     private int primaryKey;
     private String firstName;
@@ -61,44 +61,44 @@ public class EncBmpBean implements javax
 
     /**
      * Maps to EncBmpHome.findEmptyCollection
-     * 
+     *
      * @return
-     * @exception javax.ejb.FinderException
+     * @throws javax.ejb.FinderException
      * @see EncBmpHome#findEmptyCollection()
      */
     public java.util.Collection ejbFindEmptyCollection()
-    throws javax.ejb.FinderException, java.rmi.RemoteException {
+        throws javax.ejb.FinderException, java.rmi.RemoteException {
         return new java.util.Vector();
     }
 
     /**
      * Maps to EncBmpHome.findByPrimaryKey
-     * 
+     *
      * @param primaryKey
      * @return
-     * @exception javax.ejb.FinderException
+     * @throws javax.ejb.FinderException
      * @see EncBmpHome#findByPrimaryKey(Integer)
      */
-    public Integer ejbFindByPrimaryKey(Integer primaryKey)
-    throws javax.ejb.FinderException{
+    public Integer ejbFindByPrimaryKey(final Integer primaryKey)
+        throws javax.ejb.FinderException {
         return new Integer(-1);
     }
 
     /**
      * Maps to EncBmpHome.create
-     * 
+     *
      * @param name
      * @return
-     * @exception javax.ejb.CreateException
+     * @throws javax.ejb.CreateException
      * @see EncBmpHome#create
      */
-    public Integer ejbCreate(String name)
-    throws javax.ejb.CreateException{
+    public Integer ejbCreate(final String name)
+        throws javax.ejb.CreateException {
         return new Integer(-1);
     }
 
-    public void ejbPostCreate(String name)
-    throws javax.ejb.CreateException{
+    public void ejbPostCreate(final String name)
+        throws javax.ejb.CreateException {
     }
 
 
@@ -112,397 +112,397 @@ public class EncBmpBean implements javax
     //    
 
 
-    public void lookupEntityBean() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupEntityBean() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            BasicBmpHome home = (BasicBmpHome) javax.rmi.PortableRemoteObject.narrow( ctx.lookup("java:comp/env/entity/bmp/beanReferences/bmp_entity"), BasicBmpHome.class );
-            Assert.assertNotNull("The EJBHome looked up is null",home);
+                final BasicBmpHome home = (BasicBmpHome) javax.rmi.PortableRemoteObject.narrow(ctx.lookup("java:comp/env/entity/bmp/beanReferences/bmp_entity"), BasicBmpHome.class);
+                Assert.assertNotNull("The EJBHome looked up is null", home);
 
-            BasicBmpObject object = home.createObject("Enc Bean");
-            Assert.assertNotNull("The EJBObject is null", object );
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+                final BasicBmpObject object = home.createObject("Enc Bean");
+                Assert.assertNotNull("The EJBObject is null", object);
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupStatefulBean() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupStatefulBean() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            BasicStatefulHome home = (BasicStatefulHome) javax.rmi.PortableRemoteObject.narrow( ctx.lookup("java:comp/env/entity/bmp/beanReferences/stateful"), BasicStatefulHome.class );
-            Assert.assertNotNull("The EJBHome looked up is null",home);
+                final BasicStatefulHome home = (BasicStatefulHome) javax.rmi.PortableRemoteObject.narrow(ctx.lookup("java:comp/env/entity/bmp/beanReferences/stateful"), BasicStatefulHome.class);
+                Assert.assertNotNull("The EJBHome looked up is null", home);
 
-            BasicStatefulObject object = home.createObject("Enc Bean");
-            Assert.assertNotNull("The EJBObject is null", object );
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+                final BasicStatefulObject object = home.createObject("Enc Bean");
+                Assert.assertNotNull("The EJBObject is null", object);
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupStatelessBean() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupStatelessBean() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            BasicStatelessHome home = (BasicStatelessHome) javax.rmi.PortableRemoteObject.narrow( ctx.lookup("java:comp/env/entity/bmp/beanReferences/stateless"), BasicStatelessHome.class );
-            Assert.assertNotNull("The EJBHome looked up is null",home);
+                final BasicStatelessHome home = (BasicStatelessHome) javax.rmi.PortableRemoteObject.narrow(ctx.lookup("java:comp/env/entity/bmp/beanReferences/stateless"), BasicStatelessHome.class);
+                Assert.assertNotNull("The EJBHome looked up is null", home);
 
-            BasicStatelessObject object = home.createObject();
-            Assert.assertNotNull("The EJBObject is null", object );
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+                final BasicStatelessObject object = home.createObject();
+                Assert.assertNotNull("The EJBObject is null", object);
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupStatelessBusinessLocal() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupStatelessBusinessLocal() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-                Object o = ctx.lookup("java:comp/env/entity/bmp/beanReferences/stateless-business-local");
-                BasicStatelessBusinessLocal object = (BasicStatelessBusinessLocal) o;
-            Assert.assertNotNull("The EJB BusinessLocal is null", object );
-            } catch (Exception e){
+                final Object o = ctx.lookup("java:comp/env/entity/bmp/beanReferences/stateless-business-local");
+                final BasicStatelessBusinessLocal object = (BasicStatelessBusinessLocal) o;
+                Assert.assertNotNull("The EJB BusinessLocal is null", object);
+            } catch (final Exception e) {
                 e.printStackTrace();
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupStatelessBusinessRemote() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupStatelessBusinessRemote() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            BasicStatelessBusinessRemote object = (BasicStatelessBusinessRemote) javax.rmi.PortableRemoteObject.narrow( ctx.lookup("java:comp/env/entity/bmp/beanReferences/stateless-business-remote"), BasicStatelessBusinessRemote.class );
-            Assert.assertNotNull("The EJB BusinessRemote is null", object );
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+                final BasicStatelessBusinessRemote object = (BasicStatelessBusinessRemote) javax.rmi.PortableRemoteObject.narrow(ctx.lookup("java:comp/env/entity/bmp/beanReferences/stateless-business-remote"), BasicStatelessBusinessRemote.class);
+                Assert.assertNotNull("The EJB BusinessRemote is null", object);
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupStatefulBusinessLocal() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupStatefulBusinessLocal() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            BasicStatefulBusinessLocal object = (BasicStatefulBusinessLocal) javax.rmi.PortableRemoteObject.narrow( ctx.lookup("java:comp/env/entity/bmp/beanReferences/stateful-business-local"), BasicStatefulBusinessLocal.class );
-            Assert.assertNotNull("The EJB BusinessLocal is null", object );
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+                final BasicStatefulBusinessLocal object = (BasicStatefulBusinessLocal) javax.rmi.PortableRemoteObject.narrow(ctx.lookup("java:comp/env/entity/bmp/beanReferences/stateful-business-local"), BasicStatefulBusinessLocal.class);
+                Assert.assertNotNull("The EJB BusinessLocal is null", object);
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupStatefulBusinessRemote() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupStatefulBusinessRemote() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            BasicStatefulBusinessRemote object = (BasicStatefulBusinessRemote) javax.rmi.PortableRemoteObject.narrow( ctx.lookup("java:comp/env/entity/bmp/beanReferences/stateful-business-remote"), BasicStatefulBusinessRemote.class );
-            Assert.assertNotNull("The EJB BusinessRemote is null", object );
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+                final BasicStatefulBusinessRemote object = (BasicStatefulBusinessRemote) javax.rmi.PortableRemoteObject.narrow(ctx.lookup("java:comp/env/entity/bmp/beanReferences/stateful-business-remote"), BasicStatefulBusinessRemote.class);
+                Assert.assertNotNull("The EJB BusinessRemote is null", object);
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupStringEntry() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupStringEntry() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            String expected = new String("1");
-            String actual   = (String)ctx.lookup("java:comp/env/entity/bmp/references/String");
+                final String expected = new String("1");
+                final String actual = (String) ctx.lookup("java:comp/env/entity/bmp/references/String");
 
-            Assert.assertNotNull("The String looked up is null", actual );
-            Assert.assertEquals(expected, actual );
+                Assert.assertNotNull("The String looked up is null", actual);
+                Assert.assertEquals(expected, actual);
 
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupDoubleEntry() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupDoubleEntry() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            Double expected = new Double(1.0D);
-            Double actual   = (Double)ctx.lookup("java:comp/env/entity/bmp/references/Double");
+                final Double expected = new Double(1.0D);
+                final Double actual = (Double) ctx.lookup("java:comp/env/entity/bmp/references/Double");
 
-            Assert.assertNotNull("The Double looked up is null", actual );
-            Assert.assertEquals(expected, actual );
+                Assert.assertNotNull("The Double looked up is null", actual);
+                Assert.assertEquals(expected, actual);
 
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupLongEntry() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupLongEntry() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            Long expected = new Long(1L);
-            Long actual   = (Long)ctx.lookup("java:comp/env/entity/bmp/references/Long");
+                final Long expected = new Long(1L);
+                final Long actual = (Long) ctx.lookup("java:comp/env/entity/bmp/references/Long");
 
-            Assert.assertNotNull("The Long looked up is null", actual );
-            Assert.assertEquals(expected, actual );
+                Assert.assertNotNull("The Long looked up is null", actual);
+                Assert.assertEquals(expected, actual);
 
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupFloatEntry() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupFloatEntry() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            Float expected = new Float(1.0F);
-            Float actual   = (Float)ctx.lookup("java:comp/env/entity/bmp/references/Float");
+                final Float expected = new Float(1.0F);
+                final Float actual = (Float) ctx.lookup("java:comp/env/entity/bmp/references/Float");
 
-            Assert.assertNotNull("The Float looked up is null", actual );
-            Assert.assertEquals(expected, actual );
+                Assert.assertNotNull("The Float looked up is null", actual);
+                Assert.assertEquals(expected, actual);
 
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupIntegerEntry() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupIntegerEntry() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            Integer expected = new Integer(1);
-            Integer actual   = (Integer)ctx.lookup("java:comp/env/entity/bmp/references/Integer");
+                final Integer expected = new Integer(1);
+                final Integer actual = (Integer) ctx.lookup("java:comp/env/entity/bmp/references/Integer");
 
-            Assert.assertNotNull("The Integer looked up is null", actual );
-            Assert.assertEquals(expected, actual );
+                Assert.assertNotNull("The Integer looked up is null", actual);
+                Assert.assertEquals(expected, actual);
 
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupShortEntry() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupShortEntry() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            Short expected = new Short((short)1);
-            Short actual   = (Short)ctx.lookup("java:comp/env/entity/bmp/references/Short");
+                final Short expected = new Short((short) 1);
+                final Short actual = (Short) ctx.lookup("java:comp/env/entity/bmp/references/Short");
 
-            Assert.assertNotNull("The Short looked up is null", actual );
-            Assert.assertEquals(expected, actual );
+                Assert.assertNotNull("The Short looked up is null", actual);
+                Assert.assertEquals(expected, actual);
 
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupBooleanEntry() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupBooleanEntry() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            Boolean expected = new Boolean(true);
-            Boolean actual = (Boolean)ctx.lookup("java:comp/env/entity/bmp/references/Boolean");
+                final Boolean expected = new Boolean(true);
+                final Boolean actual = (Boolean) ctx.lookup("java:comp/env/entity/bmp/references/Boolean");
 
-            Assert.assertNotNull("The Boolean looked up is null", actual );
-            Assert.assertEquals(expected, actual );
+                Assert.assertNotNull("The Boolean looked up is null", actual);
+                Assert.assertEquals(expected, actual);
 
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupByteEntry() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupByteEntry() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            Byte expected = new Byte((byte)1);
-            Byte actual   = (Byte)ctx.lookup("java:comp/env/entity/bmp/references/Byte");
+                final Byte expected = new Byte((byte) 1);
+                final Byte actual = (Byte) ctx.lookup("java:comp/env/entity/bmp/references/Byte");
 
-            Assert.assertNotNull("The Byte looked up is null", actual );
-            Assert.assertEquals(expected, actual );
+                Assert.assertNotNull("The Byte looked up is null", actual);
+                Assert.assertEquals(expected, actual);
 
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
 
-    public void lookupCharacterEntry() throws TestFailureException{
-        try{
-            try{
-            InitialContext ctx = new InitialContext();
-            Assert.assertNotNull("The InitialContext is null", ctx );
+    public void lookupCharacterEntry() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
+                Assert.assertNotNull("The InitialContext is null", ctx);
 
-            Character expected = new Character('D');
-            Character actual   = (Character)ctx.lookup("java:comp/env/entity/bmp/references/Character");
+                final Character expected = new Character('D');
+                final Character actual = (Character) ctx.lookup("java:comp/env/entity/bmp/references/Character");
 
-            Assert.assertNotNull("The Character looked up is null", actual );
-            Assert.assertEquals(expected, actual );
+                Assert.assertNotNull("The Character looked up is null", actual);
+                Assert.assertEquals(expected, actual);
 
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupResource() throws TestFailureException{
-        try{
-            try{
-                InitialContext ctx = new InitialContext();
+    public void lookupResource() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
                 Assert.assertNotNull("The InitialContext is null", ctx);
-                Object obj = ctx.lookup("java:comp/env/datasource");
+                final Object obj = ctx.lookup("java:comp/env/datasource");
                 Assert.assertNotNull("The DataSource is null", obj);
                 Assert.assertTrue("Not an instance of DataSource", obj instanceof DataSource);
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupJMSConnectionFactory() throws TestFailureException{
-        try{
-            try{
-                InitialContext ctx = new InitialContext();
+    public void lookupJMSConnectionFactory() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
                 Assert.assertNotNull("The InitialContext is null", ctx);
                 Object obj = ctx.lookup("java:comp/env/jms");
                 Assert.assertNotNull("The JMS ConnectionFactory is null", obj);
                 Assert.assertTrue("Not an instance of ConnectionFactory", obj instanceof ConnectionFactory);
-                ConnectionFactory connectionFactory = (ConnectionFactory) obj;
+                final ConnectionFactory connectionFactory = (ConnectionFactory) obj;
                 testJmsConnection(connectionFactory.createConnection());
 
                 obj = ctx.lookup("java:comp/env/TopicCF");
                 Assert.assertNotNull("The JMS TopicConnectionFactory is null", obj);
                 Assert.assertTrue("Not an instance of TopicConnectionFactory", obj instanceof TopicConnectionFactory);
-                TopicConnectionFactory topicConnectionFactory = (TopicConnectionFactory) obj;
+                final TopicConnectionFactory topicConnectionFactory = (TopicConnectionFactory) obj;
                 testJmsConnection(topicConnectionFactory.createConnection());
 
                 obj = ctx.lookup("java:comp/env/QueueCF");
                 Assert.assertNotNull("The JMS QueueConnectionFactory is null", obj);
                 Assert.assertTrue("Not an instance of QueueConnectionFactory", obj instanceof QueueConnectionFactory);
-                QueueConnectionFactory queueConnectionFactory = (QueueConnectionFactory) obj;
+                final QueueConnectionFactory queueConnectionFactory = (QueueConnectionFactory) obj;
                 testJmsConnection(queueConnectionFactory.createConnection());
-            } catch (Exception e){
+            } catch (final Exception e) {
                 e.printStackTrace();
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    private void testJmsConnection(Connection connection) throws JMSException {
-        Session session = connection.createSession(false, Session.DUPS_OK_ACKNOWLEDGE);
-        Topic topic = session.createTopic("test");
-        MessageProducer producer = session.createProducer(topic);
+    private void testJmsConnection(final Connection connection) throws JMSException {
+        final Session session = connection.createSession(false, Session.DUPS_OK_ACKNOWLEDGE);
+        final Topic topic = session.createTopic("test");
+        final MessageProducer producer = session.createProducer(topic);
         producer.send(session.createMessage());
         producer.close();
         session.close();
         connection.close();
     }
 
-    public void lookupPersistenceUnit() throws TestFailureException{
-        try{
-            try{
-                InitialContext ctx = new InitialContext();
+    public void lookupPersistenceUnit() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
                 Assert.assertNotNull("The InitialContext is null", ctx);
-                EntityManagerFactory emf = (EntityManagerFactory)ctx.lookup("java:comp/env/persistence/TestUnit");
-                Assert.assertNotNull("The EntityManagerFactory is null", emf );
+                final EntityManagerFactory emf = (EntityManagerFactory) ctx.lookup("java:comp/env/persistence/TestUnit");
+                Assert.assertNotNull("The EntityManagerFactory is null", emf);
 
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
 
-    public void lookupPersistenceContext() throws TestFailureException{
-        try{
-            try{
-                InitialContext ctx = new InitialContext();
+    public void lookupPersistenceContext() throws TestFailureException {
+        try {
+            try {
+                final InitialContext ctx = new InitialContext();
                 Assert.assertNotNull("The InitialContext is null", ctx);
-                EntityManager em = (EntityManager)ctx.lookup("java:comp/env/persistence/TestContext");
+                final EntityManager em = (EntityManager) ctx.lookup("java:comp/env/persistence/TestContext");
                 Assert.assertNotNull("The EntityManager is null", em);
 
                 // call a do nothing method to assure entity manager actually exists
                 em.getFlushMode();
-            } catch (Exception e){
-                Assert.fail("Received Exception "+e.getClass()+ " : "+e.getMessage());
+            } catch (final Exception e) {
+                Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe){
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -521,14 +521,14 @@ public class EncBmpBean implements javax
      * instance to synchronize its state by loading it state from the
      * underlying database.
      */
-    public void ejbLoad() throws EJBException,RemoteException {
+    public void ejbLoad() throws EJBException, RemoteException {
     }
 
     /**
      * Set the associated entity context. The container invokes this method
      * on an instance after the instance has been created.
      */
-    public void setEntityContext(EntityContext ctx) throws EJBException,RemoteException {
+    public void setEntityContext(final EntityContext ctx) throws EJBException, RemoteException {
         ejbContext = ctx;
     }
 
@@ -536,7 +536,7 @@ public class EncBmpBean implements javax
      * Unset the associated entity context. The container calls this method
      * before removing the instance.
      */
-    public void unsetEntityContext() throws EJBException,RemoteException {
+    public void unsetEntityContext() throws EJBException, RemoteException {
     }
 
     /**
@@ -544,7 +544,7 @@ public class EncBmpBean implements javax
      * instance to synchronize its state by storing it to the underlying
      * database.
      */
-    public void ejbStore() throws EJBException,RemoteException {
+    public void ejbStore() throws EJBException, RemoteException {
     }
 
     /**
@@ -555,7 +555,7 @@ public class EncBmpBean implements javax
      * This method transitions the instance from the ready state to the pool
      * of available instances.
      */
-    public void ejbRemove() throws RemoveException,EJBException,RemoteException {
+    public void ejbRemove() throws RemoveException, EJBException, RemoteException {
     }
 
     /**
@@ -564,7 +564,7 @@ public class EncBmpBean implements javax
      * with a specific EJB object. This method transitions the instance to
      * the ready state.
      */
-    public void ejbActivate() throws EJBException,RemoteException {
+    public void ejbActivate() throws EJBException, RemoteException {
     }
 
     /**
@@ -573,7 +573,7 @@ public class EncBmpBean implements javax
      * completes, the container will place the instance into the pool of
      * available instances.
      */
-    public void ejbPassivate() throws EJBException,RemoteException {
+    public void ejbPassivate() throws EJBException, RemoteException {
     }
     //    
     // EntityBean interface methods

Modified: tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/EncBmpHome.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/EncBmpHome.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/EncBmpHome.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/EncBmpHome.java Tue Jul 15 21:55:09 2014
@@ -21,12 +21,12 @@ public interface EncBmpHome extends java
 
 
     public EncBmpObject create(String name)
-    throws javax.ejb.CreateException, java.rmi.RemoteException;
-    
+        throws javax.ejb.CreateException, java.rmi.RemoteException;
+
     public EncBmpObject findByPrimaryKey(Integer primarykey)
-    throws javax.ejb.FinderException, java.rmi.RemoteException;
-    
+        throws javax.ejb.FinderException, java.rmi.RemoteException;
+
     public java.util.Collection findEmptyCollection()
-    throws javax.ejb.FinderException, java.rmi.RemoteException;
-    
+        throws javax.ejb.FinderException, java.rmi.RemoteException;
+
 }

Modified: tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/EncBmpObject.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/EncBmpObject.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/EncBmpObject.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/EncBmpObject.java Tue Jul 15 21:55:09 2014
@@ -20,30 +20,46 @@ import java.rmi.RemoteException;
 
 import org.apache.openejb.test.TestFailureException;
 
-public interface EncBmpObject extends javax.ejb.EJBObject{
-    
-    public void lookupEntityBean()    throws TestFailureException, RemoteException;
-    public void lookupStatefulBean()  throws TestFailureException, RemoteException;
+public interface EncBmpObject extends javax.ejb.EJBObject {
+
+    public void lookupEntityBean() throws TestFailureException, RemoteException;
+
+    public void lookupStatefulBean() throws TestFailureException, RemoteException;
+
     public void lookupStatelessBean() throws TestFailureException, RemoteException;
 
     public void lookupStatelessBusinessLocal() throws TestFailureException, RemoteException;
+
     public void lookupStatelessBusinessRemote() throws TestFailureException, RemoteException;
+
     public void lookupStatefulBusinessLocal() throws TestFailureException, RemoteException;
+
     public void lookupStatefulBusinessRemote() throws TestFailureException, RemoteException;
 
     public void lookupResource() throws TestFailureException, RemoteException;
+
     public void lookupJMSConnectionFactory() throws TestFailureException, RemoteException;
+
     public void lookupPersistenceUnit() throws TestFailureException, RemoteException;
+
     public void lookupPersistenceContext() throws TestFailureException, RemoteException;
 
-    public void lookupStringEntry()  throws TestFailureException, RemoteException;
-    public void lookupDoubleEntry()  throws TestFailureException, RemoteException;
-    public void lookupLongEntry()    throws TestFailureException, RemoteException;
-    public void lookupFloatEntry()   throws TestFailureException, RemoteException;
+    public void lookupStringEntry() throws TestFailureException, RemoteException;
+
+    public void lookupDoubleEntry() throws TestFailureException, RemoteException;
+
+    public void lookupLongEntry() throws TestFailureException, RemoteException;
+
+    public void lookupFloatEntry() throws TestFailureException, RemoteException;
+
     public void lookupIntegerEntry() throws TestFailureException, RemoteException;
-    public void lookupShortEntry()   throws TestFailureException, RemoteException;
+
+    public void lookupShortEntry() throws TestFailureException, RemoteException;
+
     public void lookupBooleanEntry() throws TestFailureException, RemoteException;
-    public void lookupByteEntry()    throws TestFailureException, RemoteException;
-    public void lookupCharacterEntry()    throws TestFailureException, RemoteException;
+
+    public void lookupByteEntry() throws TestFailureException, RemoteException;
+
+    public void lookupCharacterEntry() throws TestFailureException, RemoteException;
 
 }

Modified: tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/RmiIiopBmpBean.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/RmiIiopBmpBean.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/RmiIiopBmpBean.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/RmiIiopBmpBean.java Tue Jul 15 21:55:09 2014
@@ -29,65 +29,65 @@ import javax.naming.InitialContext;
 
 import org.apache.openejb.test.object.ObjectGraph;
 
-public class RmiIiopBmpBean implements javax.ejb.EntityBean{
+public class RmiIiopBmpBean implements javax.ejb.EntityBean {
     private int primaryKey;
     private String firstName;
     private String lastName;
     private EntityContext ejbContext;
-    
-    
+
+
     //=============================
     // Home interface methods
     //    
-    
+
     /**
      * Maps to RmiIiopBmpHome.findEmptyCollection
-     * 
+     *
      * @param primaryKey
-     * @return 
-     * @exception javax.ejb.FinderException
+     * @return
+     * @throws javax.ejb.FinderException
      * @see RmiIiopBmpHome#sum
      */
     public java.util.Collection ejbFindEmptyCollection()
-    throws javax.ejb.FinderException, java.rmi.RemoteException {
+        throws javax.ejb.FinderException, java.rmi.RemoteException {
         return new java.util.Vector();
     }
 
     /**
      * Maps to RmiIiopBmpHome.findByPrimaryKey
-     * 
+     *
      * @param primaryKey
-     * @return 
-     * @exception javax.ejb.FinderException
+     * @return
+     * @throws javax.ejb.FinderException
      * @see RmiIiopBmpHome#sum
      */
-    public Integer ejbFindByPrimaryKey(Integer primaryKey)
-    throws javax.ejb.FinderException{
+    public Integer ejbFindByPrimaryKey(final Integer primaryKey)
+        throws javax.ejb.FinderException {
         return new Integer(-1);
     }
 
     /**
      * Maps to RmiIiopBmpHome.create
-     * 
+     *
      * @param name
-     * @return 
-     * @exception javax.ejb.CreateException
+     * @return
+     * @throws javax.ejb.CreateException
      * @see RmiIiopBmpHome#create
      */
-    public Integer ejbCreate(String name)
-    throws javax.ejb.CreateException{
+    public Integer ejbCreate(final String name)
+        throws javax.ejb.CreateException {
         return new Integer(-1);
     }
-    
-    public void ejbPostCreate(String name)
-    throws javax.ejb.CreateException{
+
+    public void ejbPostCreate(final String name)
+        throws javax.ejb.CreateException {
     }
-    
-    
+
+
     //    
     // Home interface methods
     //=============================
-    
+
 
     //=============================
     // Remote interface methods
@@ -95,172 +95,172 @@ public class RmiIiopBmpBean implements j
     /*-------------------------------------------------*/
     /*  String                                         */  
     /*-------------------------------------------------*/
-    
-    public String returnStringObject(String data) {
+
+    public String returnStringObject(final String data) {
         return data;
     }
-    
-    public String[] returnStringObjectArray(String[] data) {
+
+    public String[] returnStringObjectArray(final String[] data) {
         return data;
     }
     
     /*-------------------------------------------------*/
     /*  Character                                      */  
     /*-------------------------------------------------*/
-    
-    public Character returnCharacterObject(Character data) {
+
+    public Character returnCharacterObject(final Character data) {
         return data;
     }
-    
-    public char returnCharacterPrimitive(char data) {
+
+    public char returnCharacterPrimitive(final char data) {
         return data;
     }
-    
-    public Character[] returnCharacterObjectArray(Character[] data) {
+
+    public Character[] returnCharacterObjectArray(final Character[] data) {
         return data;
     }
-    
-    public char[] returnCharacterPrimitiveArray(char[] data) {
+
+    public char[] returnCharacterPrimitiveArray(final char[] data) {
         return data;
     }
 
     /*-------------------------------------------------*/
     /*  Boolean                                        */  
     /*-------------------------------------------------*/
-    
-    public Boolean returnBooleanObject(Boolean data) {
+
+    public Boolean returnBooleanObject(final Boolean data) {
         return data;
     }
-    
-    public boolean returnBooleanPrimitive(boolean data) {
+
+    public boolean returnBooleanPrimitive(final boolean data) {
         return data;
     }
-    
-    public Boolean[] returnBooleanObjectArray(Boolean[] data) {
+
+    public Boolean[] returnBooleanObjectArray(final Boolean[] data) {
         return data;
     }
-    
-    public boolean[] returnBooleanPrimitiveArray(boolean[] data) {
+
+    public boolean[] returnBooleanPrimitiveArray(final boolean[] data) {
         return data;
     }
     
     /*-------------------------------------------------*/
     /*  Byte                                           */  
     /*-------------------------------------------------*/
-    
-    public Byte returnByteObject(Byte data) {
+
+    public Byte returnByteObject(final Byte data) {
         return data;
     }
-    
-    public byte returnBytePrimitive(byte data) {
+
+    public byte returnBytePrimitive(final byte data) {
         return data;
     }
-    
-    public Byte[] returnByteObjectArray(Byte[] data) {
+
+    public Byte[] returnByteObjectArray(final Byte[] data) {
         return data;
     }
-    
-    public byte[] returnBytePrimitiveArray(byte[] data) {
+
+    public byte[] returnBytePrimitiveArray(final byte[] data) {
         return data;
     }
     
     /*-------------------------------------------------*/
     /*  Short                                          */  
     /*-------------------------------------------------*/
-    
-    public Short returnShortObject(Short data) {
+
+    public Short returnShortObject(final Short data) {
         return data;
     }
-    
-    public short returnShortPrimitive(short data) {
+
+    public short returnShortPrimitive(final short data) {
         return data;
     }
-    
-    public Short[] returnShortObjectArray(Short[] data) {
+
+    public Short[] returnShortObjectArray(final Short[] data) {
         return data;
     }
-    
-    public short[] returnShortPrimitiveArray(short[] data) {
+
+    public short[] returnShortPrimitiveArray(final short[] data) {
         return data;
     }
     
     /*-------------------------------------------------*/
     /*  Integer                                        */  
     /*-------------------------------------------------*/
-    
-    public Integer returnIntegerObject(Integer data) {
+
+    public Integer returnIntegerObject(final Integer data) {
         return data;
     }
-    
-    public int returnIntegerPrimitive(int data) {
+
+    public int returnIntegerPrimitive(final int data) {
         return data;
     }
-    
-    public Integer[] returnIntegerObjectArray(Integer[] data) {
+
+    public Integer[] returnIntegerObjectArray(final Integer[] data) {
         return data;
     }
-    
-    public int[] returnIntegerPrimitiveArray(int[] data) {
+
+    public int[] returnIntegerPrimitiveArray(final int[] data) {
         return data;
     }
     
     /*-------------------------------------------------*/
     /*  Long                                           */  
     /*-------------------------------------------------*/
-    
-    public Long returnLongObject(Long data) {
+
+    public Long returnLongObject(final Long data) {
         return data;
     }
-    
-    public long returnLongPrimitive(long data) {
+
+    public long returnLongPrimitive(final long data) {
         return data;
     }
-    
-    public Long[] returnLongObjectArray(Long[] data) {
+
+    public Long[] returnLongObjectArray(final Long[] data) {
         return data;
     }
-    
-    public long[] returnLongPrimitiveArray(long[] data) {
+
+    public long[] returnLongPrimitiveArray(final long[] data) {
         return data;
     }
     
     /*-------------------------------------------------*/
     /*  Float                                          */  
     /*-------------------------------------------------*/
-    
-    public Float returnFloatObject(Float data) {
+
+    public Float returnFloatObject(final Float data) {
         return data;
     }
-    
-    public float returnFloatPrimitive(float data) {
+
+    public float returnFloatPrimitive(final float data) {
         return data;
     }
-    
-    public Float[] returnFloatObjectArray(Float[] data) {
+
+    public Float[] returnFloatObjectArray(final Float[] data) {
         return data;
     }
-    
-    public float[] returnFloatPrimitiveArray(float[] data) {
+
+    public float[] returnFloatPrimitiveArray(final float[] data) {
         return data;
     }
     
     /*-------------------------------------------------*/
     /*  Double                                         */  
     /*-------------------------------------------------*/
-    
-    public Double returnDoubleObject(Double data) {
+
+    public Double returnDoubleObject(final Double data) {
         return data;
     }
-    
-    public double returnDoublePrimitive(double data) {
+
+    public double returnDoublePrimitive(final double data) {
         return data;
     }
-    
-    public Double[] returnDoubleObjectArray(Double[] data) {
+
+    public Double[] returnDoubleObjectArray(final Double[] data) {
         return data;
     }
-    
-    public double[] returnDoublePrimitiveArray(double[] data) {
+
+    public double[] returnDoublePrimitiveArray(final double[] data) {
         return data;
     }
     
@@ -268,172 +268,172 @@ public class RmiIiopBmpBean implements j
     /*-------------------------------------------------*/
     /*  EJBHome                                         */  
     /*-------------------------------------------------*/
-    
-    public EJBHome returnEJBHome(EJBHome data) {
+
+    public EJBHome returnEJBHome(final EJBHome data) {
         return data;
     }
-    
-    public EJBHome returnEJBHome() throws javax.ejb.EJBException{
+
+    public EJBHome returnEJBHome() throws javax.ejb.EJBException {
         EJBHome data = null;
 
-        try{
-        InitialContext ctx = new InitialContext();
+        try {
+            final InitialContext ctx = new InitialContext();
 
-        data = (EJBHome)ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
+            data = (EJBHome) ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
 
-        } catch (Exception e){
+        } catch (final Exception e) {
             e.printStackTrace();
             throw new javax.ejb.EJBException(e);
         }
         return data;
     }
 
-    public ObjectGraph returnNestedEJBHome() throws javax.ejb.EJBException{
-        ObjectGraph data = null; 
+    public ObjectGraph returnNestedEJBHome() throws javax.ejb.EJBException {
+        ObjectGraph data = null;
 
-        try{
-        InitialContext ctx = new InitialContext();
+        try {
+            final InitialContext ctx = new InitialContext();
 
-        Object object = ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
-        data = new ObjectGraph(object);
+            final Object object = ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
+            data = new ObjectGraph(object);
 
-        } catch (Exception e){
+        } catch (final Exception e) {
             throw new javax.ejb.EJBException(e);
         }
         return data;
     }
 
-    public EJBHome[] returnEJBHomeArray(EJBHome[] data) {
+    public EJBHome[] returnEJBHomeArray(final EJBHome[] data) {
         return data;
     }
     
     /*-------------------------------------------------*/
     /*  EJBObject                                      */  
     /*-------------------------------------------------*/
-    
-    public EJBObject returnEJBObject(EJBObject data) {
+
+    public EJBObject returnEJBObject(final EJBObject data) {
         return data;
     }
-    
-    public EJBObject returnEJBObject() throws javax.ejb.EJBException{
+
+    public EJBObject returnEJBObject() throws javax.ejb.EJBException {
         EncBmpObject data = null;
 
-        try{
-        InitialContext ctx = new InitialContext();
+        try {
+            final InitialContext ctx = new InitialContext();
 
-        EncBmpHome home = (EncBmpHome)ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
-        data = home.create("Test01 BmpBean");
+            final EncBmpHome home = (EncBmpHome) ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
+            data = home.create("Test01 BmpBean");
 
-        } catch (Exception e){
+        } catch (final Exception e) {
             throw new javax.ejb.EJBException(e);
         }
         return data;
     }
-    
-    public ObjectGraph returnNestedEJBObject() throws javax.ejb.EJBException{
+
+    public ObjectGraph returnNestedEJBObject() throws javax.ejb.EJBException {
         ObjectGraph data = null;
 
-        try{
-        InitialContext ctx = new InitialContext();
+        try {
+            final InitialContext ctx = new InitialContext();
 
-        EncBmpHome home = (EncBmpHome)ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
-        EncBmpObject object = home.create("Test02 BmpBean");
-        data = new ObjectGraph(object);
+            final EncBmpHome home = (EncBmpHome) ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
+            final EncBmpObject object = home.create("Test02 BmpBean");
+            data = new ObjectGraph(object);
 
-        } catch (Exception e){
+        } catch (final Exception e) {
             throw new javax.ejb.EJBException(e);
         }
         return data;
     }
-    
-    public EJBObject[] returnEJBObjectArray(EJBObject[] data) {
+
+    public EJBObject[] returnEJBObjectArray(final EJBObject[] data) {
         return data;
     }
     
     /*-------------------------------------------------*/
     /*  EJBMetaData                                         */  
     /*-------------------------------------------------*/
-    
-    public EJBMetaData returnEJBMetaData(EJBMetaData data) {
+
+    public EJBMetaData returnEJBMetaData(final EJBMetaData data) {
         return data;
     }
-    
-    public EJBMetaData returnEJBMetaData() throws javax.ejb.EJBException{
+
+    public EJBMetaData returnEJBMetaData() throws javax.ejb.EJBException {
         EJBMetaData data = null;
 
-        try{
-        InitialContext ctx = new InitialContext();
+        try {
+            final InitialContext ctx = new InitialContext();
 
-        EncBmpHome home = (EncBmpHome)ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
-        data = home.getEJBMetaData();
+            final EncBmpHome home = (EncBmpHome) ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
+            data = home.getEJBMetaData();
 
-        } catch (Exception e){
+        } catch (final Exception e) {
             throw new javax.ejb.EJBException(e);
         }
         return data;
     }
-    
-    public ObjectGraph returnNestedEJBMetaData() throws javax.ejb.EJBException{
+
+    public ObjectGraph returnNestedEJBMetaData() throws javax.ejb.EJBException {
         ObjectGraph data = null;
 
-        try{
-        InitialContext ctx = new InitialContext();
+        try {
+            final InitialContext ctx = new InitialContext();
 
-        EncBmpHome home = (EncBmpHome)ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
-        EJBMetaData object = home.getEJBMetaData();
-        data = new ObjectGraph(object);
+            final EncBmpHome home = (EncBmpHome) ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
+            final EJBMetaData object = home.getEJBMetaData();
+            data = new ObjectGraph(object);
 
-        } catch (Exception e){
+        } catch (final Exception e) {
             throw new javax.ejb.EJBException(e);
         }
         return data;
     }
-    
-    public EJBMetaData[] returnEJBMetaDataArray(EJBMetaData[] data) {
+
+    public EJBMetaData[] returnEJBMetaDataArray(final EJBMetaData[] data) {
         return data;
     }
     
     /*-------------------------------------------------*/
     /*  Handle                                         */  
     /*-------------------------------------------------*/
-    
-    public Handle returnHandle(Handle data) {
+
+    public Handle returnHandle(final Handle data) {
         return data;
     }
-    
-    public Handle returnHandle() throws javax.ejb.EJBException{
+
+    public Handle returnHandle() throws javax.ejb.EJBException {
         Handle data = null;
 
-        try{
-        InitialContext ctx = new InitialContext();
+        try {
+            final InitialContext ctx = new InitialContext();
 
-        EncBmpHome home = (EncBmpHome)ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
-        EncBmpObject object = home.create("Test03 BmpBean");
-        data = object.getHandle();
+            final EncBmpHome home = (EncBmpHome) ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
+            final EncBmpObject object = home.create("Test03 BmpBean");
+            data = object.getHandle();
 
-        } catch (Exception e){
+        } catch (final Exception e) {
             throw new javax.ejb.EJBException(e);
         }
         return data;
     }
-    
-    public ObjectGraph returnNestedHandle() throws javax.ejb.EJBException{
+
+    public ObjectGraph returnNestedHandle() throws javax.ejb.EJBException {
         ObjectGraph data = null;
 
-        try{
-        InitialContext ctx = new InitialContext();
+        try {
+            final InitialContext ctx = new InitialContext();
 
-        EncBmpHome home = (EncBmpHome)ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
-        EncBmpObject object = home.create("Test04 BmpBean");
-        data = new ObjectGraph(object.getHandle());
+            final EncBmpHome home = (EncBmpHome) ctx.lookup("java:comp/env/bmp/rmi-iiop/home");
+            final EncBmpObject object = home.create("Test04 BmpBean");
+            data = new ObjectGraph(object.getHandle());
 
-        } catch (Exception e){
+        } catch (final Exception e) {
             throw new javax.ejb.EJBException(e);
         }
         return data;
     }
-    
-    public Handle[] returnHandleArray(Handle[] data) {
+
+    public Handle[] returnHandleArray(final Handle[] data) {
         return data;
     }
     
@@ -442,23 +442,23 @@ public class RmiIiopBmpBean implements j
     /*  Class                                         */
     /*-------------------------------------------------*/
 
-    public Class returnClass(Class data) {
+    public Class returnClass(final Class data) {
         return data;
     }
 
-    public Class[] returnClassArray(Class[] data) {
+    public Class[] returnClassArray(final Class[] data) {
         return data;
     }
 
     /*-------------------------------------------------*/
     /*  ObjectGraph                                         */  
     /*-------------------------------------------------*/
-    
-    public ObjectGraph returnObjectGraph(ObjectGraph data) {
+
+    public ObjectGraph returnObjectGraph(final ObjectGraph data) {
         return data;
     }
-    
-    public ObjectGraph[] returnObjectGraphArray(ObjectGraph[] data) {
+
+    public ObjectGraph[] returnObjectGraphArray(final ObjectGraph[] data) {
         return data;
     }
     //    
@@ -469,38 +469,38 @@ public class RmiIiopBmpBean implements j
     //================================
     // EntityBean interface methods
     //    
-    
+
     /**
      * A container invokes this method to instruct the
      * instance to synchronize its state by loading it state from the
      * underlying database.
      */
-    public void ejbLoad() throws EJBException,RemoteException {
+    public void ejbLoad() throws EJBException, RemoteException {
     }
-    
+
     /**
      * Set the associated entity context. The container invokes this method
      * on an instance after the instance has been created.
      */
-    public void setEntityContext(EntityContext ctx) throws EJBException,RemoteException {
+    public void setEntityContext(final EntityContext ctx) throws EJBException, RemoteException {
         ejbContext = ctx;
     }
-    
+
     /**
      * Unset the associated entity context. The container calls this method
      * before removing the instance.
      */
-    public void unsetEntityContext() throws EJBException,RemoteException {
+    public void unsetEntityContext() throws EJBException, RemoteException {
     }
-    
+
     /**
      * A container invokes this method to instruct the
      * instance to synchronize its state by storing it to the underlying
      * database.
      */
-    public void ejbStore() throws EJBException,RemoteException {
+    public void ejbStore() throws EJBException, RemoteException {
     }
-    
+
     /**
      * A container invokes this method before it removes the EJB object
      * that is currently associated with the instance. This method
@@ -509,25 +509,25 @@ public class RmiIiopBmpBean implements j
      * This method transitions the instance from the ready state to the pool
      * of available instances.
      */
-    public void ejbRemove() throws RemoveException,EJBException,RemoteException {
+    public void ejbRemove() throws RemoveException, EJBException, RemoteException {
     }
-    
+
     /**
      * A container invokes this method when the instance
      * is taken out of the pool of available instances to become associated
      * with a specific EJB object. This method transitions the instance to
      * the ready state.
      */
-    public void ejbActivate() throws EJBException,RemoteException {
+    public void ejbActivate() throws EJBException, RemoteException {
     }
-    
+
     /**
      * A container invokes this method on an instance before the instance
      * becomes disassociated with a specific EJB object. After this method
      * completes, the container will place the instance into the pool of
      * available instances.
      */
-    public void ejbPassivate() throws EJBException,RemoteException {
+    public void ejbPassivate() throws EJBException, RemoteException {
     }
 
     //    

Modified: tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/RmiIiopBmpHome.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/RmiIiopBmpHome.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/RmiIiopBmpHome.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/RmiIiopBmpHome.java Tue Jul 15 21:55:09 2014
@@ -20,12 +20,12 @@ package org.apache.openejb.test.entity.b
 public interface RmiIiopBmpHome extends javax.ejb.EJBHome {
 
     public RmiIiopBmpObject create(String name)
-    throws javax.ejb.CreateException, java.rmi.RemoteException;
-    
+        throws javax.ejb.CreateException, java.rmi.RemoteException;
+
     public RmiIiopBmpObject findByPrimaryKey(Integer primarykey)
-    throws javax.ejb.FinderException, java.rmi.RemoteException;
-    
+        throws javax.ejb.FinderException, java.rmi.RemoteException;
+
     public java.util.Collection findEmptyCollection()
-    throws javax.ejb.FinderException, java.rmi.RemoteException;
-    
+        throws javax.ejb.FinderException, java.rmi.RemoteException;
+
 }

Modified: tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/RmiIiopBmpObject.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/RmiIiopBmpObject.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/RmiIiopBmpObject.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/bmp/RmiIiopBmpObject.java Tue Jul 15 21:55:09 2014
@@ -25,111 +25,111 @@ import javax.ejb.Handle;
 
 import org.apache.openejb.test.object.ObjectGraph;
 
-public interface RmiIiopBmpObject extends javax.ejb.EJBObject{
-    
-    public String returnStringObject(String data) throws RemoteException; 
+public interface RmiIiopBmpObject extends javax.ejb.EJBObject {
 
-    public String[] returnStringObjectArray(String[] data) throws RemoteException; 
+    public String returnStringObject(String data) throws RemoteException;
 
-    public Character returnCharacterObject(Character data) throws RemoteException; 
+    public String[] returnStringObjectArray(String[] data) throws RemoteException;
 
-    public char returnCharacterPrimitive(char data) throws RemoteException; 
+    public Character returnCharacterObject(Character data) throws RemoteException;
 
-    public Character[] returnCharacterObjectArray(Character[] data) throws RemoteException; 
+    public char returnCharacterPrimitive(char data) throws RemoteException;
 
-    public char[] returnCharacterPrimitiveArray(char[] data) throws RemoteException; 
+    public Character[] returnCharacterObjectArray(Character[] data) throws RemoteException;
 
-    public Boolean returnBooleanObject(Boolean data) throws RemoteException; 
+    public char[] returnCharacterPrimitiveArray(char[] data) throws RemoteException;
 
-    public boolean returnBooleanPrimitive(boolean data) throws RemoteException; 
+    public Boolean returnBooleanObject(Boolean data) throws RemoteException;
 
-    public Boolean[] returnBooleanObjectArray(Boolean[] data) throws RemoteException; 
+    public boolean returnBooleanPrimitive(boolean data) throws RemoteException;
 
-    public boolean[] returnBooleanPrimitiveArray(boolean[] data) throws RemoteException; 
+    public Boolean[] returnBooleanObjectArray(Boolean[] data) throws RemoteException;
 
-    public Byte returnByteObject(Byte data) throws RemoteException; 
+    public boolean[] returnBooleanPrimitiveArray(boolean[] data) throws RemoteException;
 
-    public byte returnBytePrimitive(byte data) throws RemoteException; 
+    public Byte returnByteObject(Byte data) throws RemoteException;
 
-    public Byte[] returnByteObjectArray(Byte[] data) throws RemoteException; 
+    public byte returnBytePrimitive(byte data) throws RemoteException;
 
-    public byte[] returnBytePrimitiveArray(byte[] data) throws RemoteException; 
+    public Byte[] returnByteObjectArray(Byte[] data) throws RemoteException;
 
-    public Short returnShortObject(Short data) throws RemoteException; 
+    public byte[] returnBytePrimitiveArray(byte[] data) throws RemoteException;
 
-    public short returnShortPrimitive(short data) throws RemoteException; 
+    public Short returnShortObject(Short data) throws RemoteException;
 
-    public Short[] returnShortObjectArray(Short[] data) throws RemoteException; 
+    public short returnShortPrimitive(short data) throws RemoteException;
 
-    public short[] returnShortPrimitiveArray(short[] data) throws RemoteException; 
+    public Short[] returnShortObjectArray(Short[] data) throws RemoteException;
 
-    public Integer returnIntegerObject(Integer data) throws RemoteException; 
+    public short[] returnShortPrimitiveArray(short[] data) throws RemoteException;
 
-    public int returnIntegerPrimitive(int data) throws RemoteException; 
+    public Integer returnIntegerObject(Integer data) throws RemoteException;
 
-    public Integer[] returnIntegerObjectArray(Integer[] data) throws RemoteException; 
+    public int returnIntegerPrimitive(int data) throws RemoteException;
 
-    public int[] returnIntegerPrimitiveArray(int[] data) throws RemoteException; 
+    public Integer[] returnIntegerObjectArray(Integer[] data) throws RemoteException;
 
-    public Long returnLongObject(Long data) throws RemoteException; 
+    public int[] returnIntegerPrimitiveArray(int[] data) throws RemoteException;
 
-    public long returnLongPrimitive(long data) throws RemoteException; 
+    public Long returnLongObject(Long data) throws RemoteException;
 
-    public Long[] returnLongObjectArray(Long[] data) throws RemoteException; 
+    public long returnLongPrimitive(long data) throws RemoteException;
 
-    public long[] returnLongPrimitiveArray(long[] data) throws RemoteException; 
+    public Long[] returnLongObjectArray(Long[] data) throws RemoteException;
 
-    public Float returnFloatObject(Float data) throws RemoteException; 
+    public long[] returnLongPrimitiveArray(long[] data) throws RemoteException;
 
-    public float returnFloatPrimitive(float data) throws RemoteException; 
+    public Float returnFloatObject(Float data) throws RemoteException;
 
-    public Float[] returnFloatObjectArray(Float[] data) throws RemoteException; 
+    public float returnFloatPrimitive(float data) throws RemoteException;
 
-    public float[] returnFloatPrimitiveArray(float[] data) throws RemoteException; 
+    public Float[] returnFloatObjectArray(Float[] data) throws RemoteException;
 
-    public Double returnDoubleObject(Double data) throws RemoteException; 
+    public float[] returnFloatPrimitiveArray(float[] data) throws RemoteException;
 
-    public double returnDoublePrimitive(double data) throws RemoteException; 
+    public Double returnDoubleObject(Double data) throws RemoteException;
 
-    public Double[] returnDoubleObjectArray(Double[] data) throws RemoteException; 
+    public double returnDoublePrimitive(double data) throws RemoteException;
 
-    public double[] returnDoublePrimitiveArray(double[] data) throws RemoteException; 
+    public Double[] returnDoubleObjectArray(Double[] data) throws RemoteException;
+
+    public double[] returnDoublePrimitiveArray(double[] data) throws RemoteException;
 
     public EJBHome returnEJBHome(EJBHome data) throws RemoteException;
 
-    public EJBHome returnEJBHome() throws RemoteException; 
+    public EJBHome returnEJBHome() throws RemoteException;
 
-    public ObjectGraph returnNestedEJBHome() throws RemoteException; 
+    public ObjectGraph returnNestedEJBHome() throws RemoteException;
 
-    public EJBHome[] returnEJBHomeArray(EJBHome[] data) throws RemoteException; 
+    public EJBHome[] returnEJBHomeArray(EJBHome[] data) throws RemoteException;
 
-    public EJBObject returnEJBObject(EJBObject data) throws RemoteException; 
+    public EJBObject returnEJBObject(EJBObject data) throws RemoteException;
 
-    public EJBObject returnEJBObject() throws RemoteException; 
+    public EJBObject returnEJBObject() throws RemoteException;
 
-    public ObjectGraph returnNestedEJBObject() throws RemoteException; 
+    public ObjectGraph returnNestedEJBObject() throws RemoteException;
 
-    public EJBObject[] returnEJBObjectArray(EJBObject[] data) throws RemoteException; 
+    public EJBObject[] returnEJBObjectArray(EJBObject[] data) throws RemoteException;
 
-    public EJBMetaData returnEJBMetaData(EJBMetaData data) throws RemoteException; 
+    public EJBMetaData returnEJBMetaData(EJBMetaData data) throws RemoteException;
 
-    public EJBMetaData returnEJBMetaData() throws RemoteException; 
+    public EJBMetaData returnEJBMetaData() throws RemoteException;
 
-    public ObjectGraph returnNestedEJBMetaData() throws RemoteException; 
+    public ObjectGraph returnNestedEJBMetaData() throws RemoteException;
 
-    public EJBMetaData[] returnEJBMetaDataArray(EJBMetaData[] data) throws RemoteException; 
+    public EJBMetaData[] returnEJBMetaDataArray(EJBMetaData[] data) throws RemoteException;
 
-    public Handle returnHandle(Handle data) throws RemoteException; 
+    public Handle returnHandle(Handle data) throws RemoteException;
 
-    public Handle returnHandle() throws RemoteException; 
+    public Handle returnHandle() throws RemoteException;
 
-    public ObjectGraph returnNestedHandle() throws RemoteException; 
+    public ObjectGraph returnNestedHandle() throws RemoteException;
 
-    public Handle[] returnHandleArray(Handle[] data) throws RemoteException; 
+    public Handle[] returnHandleArray(Handle[] data) throws RemoteException;
 
-    public ObjectGraph returnObjectGraph(ObjectGraph data) throws RemoteException; 
+    public ObjectGraph returnObjectGraph(ObjectGraph data) throws RemoteException;
 
-    public ObjectGraph[] returnObjectGraphArray(ObjectGraph[] data) throws RemoteException; 
+    public ObjectGraph[] returnObjectGraphArray(ObjectGraph[] data) throws RemoteException;
 
     public Class returnClass(Class data) throws RemoteException;
 

Modified: tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmp/AllowedOperationsCmp2Bean.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmp/AllowedOperationsCmp2Bean.java?rev=1610862&r1=1610861&r2=1610862&view=diff
==============================================================================
--- tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmp/AllowedOperationsCmp2Bean.java (original)
+++ tomee/tomee/trunk/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmp/AllowedOperationsCmp2Bean.java Tue Jul 15 21:55:09 2014
@@ -32,7 +32,7 @@ import org.apache.openejb.test.object.Op
 public abstract class AllowedOperationsCmp2Bean implements EntityBean {
     private static int nextId;
     public EntityContext ejbContext;
-    public static final Map<String,OperationsPolicy> allowedOperationsTable = new TreeMap<String,OperationsPolicy>();
+    public static final Map<String, OperationsPolicy> allowedOperationsTable = new TreeMap<String, OperationsPolicy>();
 
     public abstract Integer getId();
 
@@ -57,10 +57,10 @@ public abstract class AllowedOperationsC
 
     /**
      * Maps to BasicCmpHome.sum
-     *
+     * <p/>
      * Adds x and y and returns the result.
      */
-    public int ejbHomeSum(int x, int y) {
+    public int ejbHomeSum(final int x, final int y) {
         testAllowedOperations("ejbHome");
         return x + y;
     }
@@ -71,16 +71,16 @@ public abstract class AllowedOperationsC
     /**
      * Maps to BasicCmpHome.create
      */
-    public Integer ejbCreateObject(String name) throws CreateException {
+    public Integer ejbCreateObject(final String name) throws CreateException {
         setId(nextId++);
         testAllowedOperations("ejbCreate");
-        StringTokenizer st = new StringTokenizer(name, " ");
+        final StringTokenizer st = new StringTokenizer(name, " ");
         setFirstName(st.nextToken());
         setLastName(st.nextToken());
         return null;
     }
 
-    public void ejbPostCreateObject(String name) {
+    public void ejbPostCreateObject(final String name) {
         testAllowedOperations("ejbPostCreate");
     }
 
@@ -96,10 +96,10 @@ public abstract class AllowedOperationsC
     /**
      * Maps to BasicCmpObject.businessMethod
      */
-    public String businessMethod(String text) {
+    public String businessMethod(final String text) {
         testAllowedOperations("businessMethod");
         setNumber(getNumber() + 1);
-        StringBuffer b = new StringBuffer(text);
+        final StringBuffer b = new StringBuffer(text);
         return b.reverse().toString();
     }
 
@@ -115,7 +115,6 @@ public abstract class AllowedOperationsC
      * This is a system exception and should result in the
      * destruction of the instance and invalidation of the
      * remote reference.
-     *
      */
     public void throwSystemException_NullPointer() {
         throw new NullPointerException("Panic");
@@ -124,7 +123,7 @@ public abstract class AllowedOperationsC
 
     /**
      * Maps to BasicCmpObject.getPermissionsReport
-     *
+     * <p/>
      * Returns a report of the bean's
      * runtime permissions
      */
@@ -135,13 +134,13 @@ public abstract class AllowedOperationsC
 
     /**
      * Maps to BasicCmpObject.getAllowedOperationsReport
-     *
+     * <p/>
      * Returns a report of the allowed opperations
      * for one of the bean's methods.
      *
      * @param methodName The method for which to get the allowed opperations report
      */
-    public OperationsPolicy getAllowedOperationsReport(String methodName) {
+    public OperationsPolicy getAllowedOperationsReport(final String methodName) {
         return allowedOperationsTable.get(methodName);
     }
 
@@ -167,7 +166,7 @@ public abstract class AllowedOperationsC
      * Set the associated entity context. The container invokes this method
      * on an instance after the instance has been created.
      */
-    public void setEntityContext(EntityContext ctx) {
+    public void setEntityContext(final EntityContext ctx) {
         ejbContext = ctx;
         testAllowedOperations("setEntityContext");
     }
@@ -224,35 +223,35 @@ public abstract class AllowedOperationsC
     // EntityBean interface methods
     //================================
 
-    protected void testAllowedOperations(String methodName) {
-        OperationsPolicy policy = new OperationsPolicy();
+    protected void testAllowedOperations(final String methodName) {
+        final OperationsPolicy policy = new OperationsPolicy();
 
         /*[0] Test getEJBHome /////////////////*/
         try {
             ejbContext.getEJBHome();
             policy.allow(OperationsPolicy.Context_getEJBHome);
-        } catch (IllegalStateException ise) {
+        } catch (final IllegalStateException ise) {
         }
 
         /*[1] Test getCallerPrincipal /////////*/
         try {
             ejbContext.getCallerPrincipal();
-            policy.allow(OperationsPolicy.Context_getCallerPrincipal );
-        } catch (IllegalStateException ise) {
+            policy.allow(OperationsPolicy.Context_getCallerPrincipal);
+        } catch (final IllegalStateException ise) {
         }
 
         /*[2] Test isCallerInRole /////////////*/
         try {
             ejbContext.isCallerInRole("TheMan");
-            policy.allow(OperationsPolicy.Context_isCallerInRole );
-        } catch (IllegalStateException ise) {
+            policy.allow(OperationsPolicy.Context_isCallerInRole);
+        } catch (final IllegalStateException ise) {
         }
 
         /*[3] Test getRollbackOnly ////////////*/
         try {
             ejbContext.getRollbackOnly();
-            policy.allow(OperationsPolicy.Context_getRollbackOnly );
-        } catch (IllegalStateException ise) {
+            policy.allow(OperationsPolicy.Context_getRollbackOnly);
+        } catch (final IllegalStateException ise) {
         }
 
         /*[4] Test setRollbackOnly ////////////*/
@@ -265,15 +264,15 @@ public abstract class AllowedOperationsC
         /*[5] Test getUserTransaction /////////*/
         try {
             ejbContext.getUserTransaction();
-            policy.allow(OperationsPolicy.Context_getUserTransaction );
-        } catch (IllegalStateException ise) {
+            policy.allow(OperationsPolicy.Context_getUserTransaction);
+        } catch (final IllegalStateException ise) {
         }
 
         /*[6] Test getEJBObject ///////////////*/
         try {
             ejbContext.getEJBObject();
-            policy.allow(OperationsPolicy.Context_getEJBObject );
-        } catch (IllegalStateException ise) {
+            policy.allow(OperationsPolicy.Context_getEJBObject);
+        } catch (final IllegalStateException ise) {
         }
 
         /*[7] Test Context_getPrimaryKey ///////////////
@@ -282,19 +281,19 @@ public abstract class AllowedOperationsC
          */
         try {
             ejbContext.getPrimaryKey();
-            policy.allow(OperationsPolicy.Context_getPrimaryKey );
-        } catch (IllegalStateException ise) {
+            policy.allow(OperationsPolicy.Context_getPrimaryKey);
+        } catch (final IllegalStateException ise) {
         }
 
         /*[8] Test JNDI_access_to_java_comp_env ///////////////*/
         try {
-            InitialContext jndiContext = new InitialContext();
+            final InitialContext jndiContext = new InitialContext();
 
             jndiContext.lookup("java:comp/env/stateless/references/JNDI_access_to_java_comp_env");
 
-            policy.allow(OperationsPolicy.JNDI_access_to_java_comp_env );
-        } catch (IllegalStateException ise) {
-        } catch (NamingException ne) {
+            policy.allow(OperationsPolicy.JNDI_access_to_java_comp_env);
+        } catch (final IllegalStateException ise) {
+        } catch (final NamingException ne) {
         }
 
         allowedOperationsTable.put(methodName, policy);