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/07 21:07:53 UTC

svn commit: r1608563 [8/38] - in /tomee/tomee/branches/tomee-1.7.x/itests: failover-ejb/src/main/java/org/apache/openejb/itest/failover/ejb/ failover/src/main/java/org/apache/openejb/itest/failover/ failover/src/main/java/org/apache/openejb/server/cont...

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicenseBean.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicenseBean.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicenseBean.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicenseBean.java Mon Jul  7 19:07:41 2014
@@ -29,39 +29,44 @@ import org.apache.openejb.test.entity.cm
 public abstract class LicenseBean implements EntityBean {
     // CMP
     public abstract Integer getId();
+
     public abstract void setId(Integer id);
 
     public abstract String getNumber();
+
     public abstract void setNumber(String number);
 
     public abstract Integer getPoints();
+
     public abstract void setPoints(Integer points);
 
     public abstract String getNotes();
+
     public abstract void setNotes(String notes);
 
     // CMR
     public abstract PersonLocal getPerson();
+
     public abstract void setPerson(PersonLocal personLocal);
-    
-    public Integer ejbCreate(Integer id)  throws CreateException {
+
+    public Integer ejbCreate(final Integer id) throws CreateException {
         setId(id);
         return null;
     }
 
-    public void ejbPostCreate(Integer id) {
+    public void ejbPostCreate(final Integer id) {
     }
 
-    public CompoundPK ejbCreate(LicensePk primaryKey)  throws CreateException {
+    public CompoundPK ejbCreate(final LicensePk primaryKey) throws CreateException {
         setId(primaryKey.id);
         setNumber(primaryKey.number);
         return null;
     }
 
-    public void ejbPostCreate(LicensePk primaryKey) {
+    public void ejbPostCreate(final LicensePk primaryKey) {
     }
 
-    public void setEntityContext(EntityContext ctx) {
+    public void setEntityContext(final EntityContext ctx) {
     }
 
     public void unsetEntityContext() {

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicenseLocal.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicenseLocal.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicenseLocal.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicenseLocal.java Mon Jul  7 19:07:41 2014
@@ -19,26 +19,30 @@ package org.apache.openejb.test.entity.c
 import javax.ejb.EJBLocalObject;
 
 /**
- *
  * @version $Revision$ $Date$
  */
 public interface LicenseLocal extends EJBLocalObject {
 
     // CMP
     public Integer getId();
+
     public void setId(Integer id);
 
     public String getNumber();
+
     public void setNumber(String number);
 
     public Integer getPoints();
+
     public void setPoints(Integer points);
 
     public String getNotes();
+
     public void setNotes(String notes);
 
     // CMR
     public PersonLocal getPerson();
+
     public void setPerson(PersonLocal person);
-    
+
 }

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicenseLocalHome.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicenseLocalHome.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicenseLocalHome.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicenseLocalHome.java Mon Jul  7 19:07:41 2014
@@ -27,9 +27,11 @@ public interface LicenseLocalHome extend
 
     // Create
     public LicenseLocal create(Integer field1) throws CreateException;
+
     public LicenseLocal create(LicensePk primaryKey) throws CreateException;
 
     // Finder
     public LicenseLocal findByPrimaryKey(Integer primaryKey) throws FinderException;
+
     public LicenseLocal findByPrimaryKey(LicensePk primaryKey) throws FinderException;
 }

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicensePk.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicensePk.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicensePk.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/LicensePk.java Mon Jul  7 19:07:41 2014
@@ -24,16 +24,16 @@ public class LicensePk {
     public LicensePk() {
     }
 
-    public LicensePk(Integer id, String name) {
+    public LicensePk(final Integer id, final String name) {
         this.id = id;
         this.number = name;
     }
 
-    public boolean equals(Object o) {
+    public boolean equals(final Object o) {
         if (this == o) return true;
         if (o == null || getClass() != o.getClass()) return false;
 
-        LicensePk personPk = (LicensePk) o;
+        final LicensePk personPk = (LicensePk) o;
 
         return id.equals(personPk.id) && number.equals(personPk.number);
     }

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonBean.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonBean.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonBean.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonBean.java Mon Jul  7 19:07:41 2014
@@ -25,39 +25,41 @@ import javax.ejb.CreateException;
 import javax.ejb.RemoveException;
 
 /**
- *
  * @version $Revision$ $Date$
  */
 public abstract class PersonBean implements EntityBean {
     // CMP
     public abstract Integer getId();
+
     public abstract void setId(Integer id);
 
     public abstract String getName();
+
     public abstract void setName(String name);
 
     // CMR
     public abstract LicenseLocal getLicense();
+
     public abstract void setLicense(LicenseLocal license);
 
-    public Integer ejbCreate(Integer id)  throws CreateException {
+    public Integer ejbCreate(final Integer id) throws CreateException {
         setId(id);
         return null;
     }
 
-    public void ejbPostCreate(Integer id) {
+    public void ejbPostCreate(final Integer id) {
     }
 
-    public CompoundPK ejbCreate(PersonPk primaryKey)  throws CreateException {
+    public CompoundPK ejbCreate(final PersonPk primaryKey) throws CreateException {
         setId(primaryKey.id);
         setName(primaryKey.name);
         return null;
     }
 
-    public void ejbPostCreate(PersonPk primaryKey) {
+    public void ejbPostCreate(final PersonPk primaryKey) {
     }
 
-    public void setEntityContext(EntityContext ctx) {
+    public void setEntityContext(final EntityContext ctx) {
     }
 
     public void unsetEntityContext() {

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonLocal.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonLocal.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonLocal.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonLocal.java Mon Jul  7 19:07:41 2014
@@ -19,18 +19,20 @@ package org.apache.openejb.test.entity.c
 import javax.ejb.EJBLocalObject;
 
 /**
- *
  * @version $Revision$ $Date$
  */
 public interface PersonLocal extends EJBLocalObject {
     // CMP
     public Integer getId();
+
     public void setId(Integer field1);
 
     public String getName();
+
     public void setName(String field2);
-    
+
     // CMR
     public LicenseLocal getLicense();
+
     public void setLicense(LicenseLocal license);
 }

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonLocalHome.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonLocalHome.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonLocalHome.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonLocalHome.java Mon Jul  7 19:07:41 2014
@@ -26,10 +26,12 @@ import javax.ejb.FinderException;
 public interface PersonLocalHome extends EJBLocalHome {
     // Create
     public PersonLocal create(Integer field1) throws CreateException;
+
     public PersonLocal create(PersonPk primaryKey) throws CreateException;
 
     // Finder
     public PersonLocal findByPrimaryKey(Integer primaryKey) throws FinderException;
+
     public PersonLocal findByPrimaryKey(PersonPk primaryKey) throws FinderException;
 
 }

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonPk.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonPk.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonPk.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/cmr/onetoone/PersonPk.java Mon Jul  7 19:07:41 2014
@@ -24,16 +24,16 @@ public class PersonPk {
     public PersonPk() {
     }
 
-    public PersonPk(Integer id, String name) {
+    public PersonPk(final Integer id, final String name) {
         this.id = id;
         this.name = name;
     }
 
-    public boolean equals(Object o) {
+    public boolean equals(final Object o) {
         if (this == o) return true;
         if (o == null || getClass() != o.getClass()) return false;
 
-        PersonPk personPk = (PersonPk) o;
+        final PersonPk personPk = (PersonPk) o;
 
         return id.equals(personPk.id) && name.equals(personPk.name);
     }

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryBean.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryBean.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryBean.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryBean.java Mon Jul  7 19:07:41 2014
@@ -23,9 +23,10 @@ import java.util.Collection;
 
 public abstract class QueryBean implements EntityBean {
     public abstract Integer getId();
+
     public abstract void setId(Integer id);
 
-    public void setEntityContext(EntityContext ctx) {
+    public void setEntityContext(final EntityContext ctx) {
     }
 
     public void unsetEntityContext() {
@@ -49,7 +50,7 @@ public abstract class QueryBean implemen
     /**
      * Select a single string field
      */
-    public String ejbHomeSelectSingleStringField(String value) throws FinderException {
+    public String ejbHomeSelectSingleStringField(final String value) throws FinderException {
         return ejbSelectSingleStringField(value);
     }
 
@@ -58,7 +59,7 @@ public abstract class QueryBean implemen
     /**
      * Select a single boolean field
      */
-    public boolean ejbHomeSelectSingleBooleanField(boolean value) throws FinderException {
+    public boolean ejbHomeSelectSingleBooleanField(final boolean value) throws FinderException {
         return ejbSelectSingleBooleanField(value);
     }
 
@@ -67,7 +68,7 @@ public abstract class QueryBean implemen
     /**
      * Select a single char field
      */
-    public char ejbHomeSelectSingleCharField(char value) throws FinderException {
+    public char ejbHomeSelectSingleCharField(final char value) throws FinderException {
         return ejbSelectSingleCharField(value);
     }
 
@@ -76,7 +77,7 @@ public abstract class QueryBean implemen
     /**
      * Select a single byte field
      */
-    public byte ejbHomeSelectSingleByteField(byte value) throws FinderException {
+    public byte ejbHomeSelectSingleByteField(final byte value) throws FinderException {
         return ejbSelectSingleByteField(value);
     }
 
@@ -85,7 +86,7 @@ public abstract class QueryBean implemen
     /**
      * Select a single short field
      */
-    public short ejbHomeSelectSingleShortField(short value) throws FinderException {
+    public short ejbHomeSelectSingleShortField(final short value) throws FinderException {
         return ejbSelectSingleShortField(value);
     }
 
@@ -94,7 +95,7 @@ public abstract class QueryBean implemen
     /**
      * Select a single int field
      */
-    public int ejbHomeSelectSingleIntField(int value) throws FinderException {
+    public int ejbHomeSelectSingleIntField(final int value) throws FinderException {
         return ejbSelectSingleIntField(value);
     }
 
@@ -103,7 +104,7 @@ public abstract class QueryBean implemen
     /**
      * Select a single long field
      */
-    public long ejbHomeSelectSingleLongField(long value) throws FinderException {
+    public long ejbHomeSelectSingleLongField(final long value) throws FinderException {
         return ejbSelectSingleLongField(value);
     }
 
@@ -112,7 +113,7 @@ public abstract class QueryBean implemen
     /**
      * Select a single float field
      */
-    public float ejbHomeSelectSingleFloatField(float value) throws FinderException {
+    public float ejbHomeSelectSingleFloatField(final float value) throws FinderException {
         return ejbSelectSingleFloatField(value);
     }
 
@@ -121,7 +122,7 @@ public abstract class QueryBean implemen
     /**
      * Select a single double field
      */
-    public double ejbHomeSelectSingleDoubleField(double value) throws FinderException {
+    public double ejbHomeSelectSingleDoubleField(final double value) throws FinderException {
         return ejbSelectSingleDoubleField(value);
     }
 
@@ -140,7 +141,7 @@ public abstract class QueryBean implemen
      * Select a collection boolean field
      */
     public Collection ejbHomeSelectCollectionBooleanField() throws FinderException {
-         return ejbSelectCollectionBooleanField();
+        return ejbSelectCollectionBooleanField();
     }
 
     public abstract Collection ejbSelectCollectionBooleanField() throws FinderException;
@@ -211,7 +212,7 @@ public abstract class QueryBean implemen
     /**
      * Select a single local ejb
      */
-    public Object ejbHomeSelectSingleLocalEjb(int value) throws FinderException {
+    public Object ejbHomeSelectSingleLocalEjb(final int value) throws FinderException {
         return ejbSelectSingleLocalEjb(value);
     }
 
@@ -220,7 +221,7 @@ public abstract class QueryBean implemen
     /**
      * Select a single remote ejb
      */
-    public Object ejbHomeSelectSingleRemoteEjb(int value) throws FinderException {
+    public Object ejbHomeSelectSingleRemoteEjb(final int value) throws FinderException {
         return ejbSelectSingleRemoteEjb(value);
     }
 

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataBean.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataBean.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataBean.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataBean.java Mon Jul  7 19:07:41 2014
@@ -22,36 +22,46 @@ import javax.ejb.EntityContext;
 
 public abstract class QueryDataBean implements EntityBean {
     public abstract Integer getId();
+
     public abstract void setId(Integer id);
 
     public abstract boolean getBooleanField();
+
     public abstract void setBooleanField(boolean value);
 
     public abstract char getCharField();
+
     public abstract void setCharField(char value);
 
     public abstract byte getByteField();
+
     public abstract void setByteField(byte value);
 
     public abstract short getShortField();
+
     public abstract void setShortField(short value);
 
     public abstract int getIntField();
+
     public abstract void setIntField(int value);
 
     public abstract long getLongField();
+
     public abstract void setLongField(long value);
 
     public abstract float getFloatField();
+
     public abstract void setFloatField(float value);
 
     public abstract double getDoubleField();
+
     public abstract void setDoubleField(double value);
 
     public abstract String getStringField();
+
     public abstract void setStringField(String value);
 
-    public Integer ejbCreate(int value) {
+    public Integer ejbCreate(final int value) {
         setId(value);
         setBooleanField(value == 2);
         setCharField((char) ('0' + value));
@@ -65,10 +75,10 @@ public abstract class QueryDataBean impl
         return null;
     }
 
-    public void ejbPostCreate(int field) {
+    public void ejbPostCreate(final int field) {
     }
 
-    public void setEntityContext(EntityContext ctx) {
+    public void setEntityContext(final EntityContext ctx) {
     }
 
     public void unsetEntityContext() {

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataHome.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataHome.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataHome.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataHome.java Mon Jul  7 19:07:41 2014
@@ -23,5 +23,6 @@ import java.rmi.RemoteException;
 
 public interface QueryDataHome extends EJBHome {
     public QueryDataRemote create(int value) throws FinderException, RemoteException;
+
     public QueryDataRemote findByPrimaryKey(Integer primaryKey) throws FinderException, RemoteException;
 }

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataLocal.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataLocal.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataLocal.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataLocal.java Mon Jul  7 19:07:41 2014
@@ -21,33 +21,43 @@ import javax.ejb.EJBLocalObject;
 
 public interface QueryDataLocal extends EJBLocalObject {
     public Integer getId();
+
     public void setId(Integer id);
 
     public boolean getBooleanField();
+
     public void setBooleanField(boolean value);
 
     public char getCharField();
+
     public void setCharField(char value);
 
     public byte getByteField();
+
     public void setByteField(byte value);
 
     public short getShortField();
+
     public void setShortField(short value);
 
     public int getIntField();
+
     public void setIntField(int value);
 
     public long getLongField();
+
     public void setLongField(long value);
 
     public float getFloatField();
+
     public void setFloatField(float value);
 
     public double getDoubleField();
+
     public void setDoubleField(double value);
 
     public String getStringField();
+
     public void setStringField(String value);
 
 }

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataLocalHome.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataLocalHome.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataLocalHome.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataLocalHome.java Mon Jul  7 19:07:41 2014
@@ -22,5 +22,6 @@ import javax.ejb.FinderException;
 
 public interface QueryDataLocalHome extends EJBLocalHome {
     public QueryDataLocal create(int value) throws FinderException;
+
     public QueryDataLocal findByPrimaryKey(Integer primaryKey) throws FinderException;
 }

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataRemote.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataRemote.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataRemote.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryDataRemote.java Mon Jul  7 19:07:41 2014
@@ -22,32 +22,42 @@ import java.rmi.RemoteException;
 
 public interface QueryDataRemote extends EJBObject {
     public Integer getId() throws RemoteException;
+
     public void setId(Integer id) throws RemoteException;
 
     public boolean getBooleanField() throws RemoteException;
+
     public void setBooleanField(boolean value) throws RemoteException;
 
     public char getCharField() throws RemoteException;
+
     public void setCharField(char value) throws RemoteException;
 
     public byte getByteField() throws RemoteException;
+
     public void setByteField(byte value) throws RemoteException;
 
     public short getShortField() throws RemoteException;
+
     public void setShortField(short value) throws RemoteException;
 
     public int getIntField() throws RemoteException;
+
     public void setIntField(int value) throws RemoteException;
 
     public long getLongField() throws RemoteException;
+
     public void setLongField(long value) throws RemoteException;
 
     public float getFloatField() throws RemoteException;
+
     public void setFloatField(float value) throws RemoteException;
 
     public double getDoubleField() throws RemoteException;
+
     public void setDoubleField(double value) throws RemoteException;
 
     public String getStringField() throws RemoteException;
+
     public void setStringField(String value) throws RemoteException;
 }

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryLocalHome.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryLocalHome.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryLocalHome.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/entity/ejbql/QueryLocalHome.java Mon Jul  7 19:07:41 2014
@@ -131,4 +131,5 @@ public interface QueryLocalHome extends 
     /**
      * Select a collection remote ejb
      */
-    public abstract Collection ejbSelectCollectionRemoteEjb(String test) throws FinderException;}
+    public abstract Collection ejbSelectCollectionRemoteEjb(String test) throws FinderException;
+}

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/ClassInterceptor.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/ClassInterceptor.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/ClassInterceptor.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/ClassInterceptor.java Mon Jul  7 19:07:41 2014
@@ -25,88 +25,81 @@ import javax.interceptor.InvocationConte
 
 
 /**
- *
  * @version $Rev$ $Date$
  */
 public class ClassInterceptor extends SuperClassInterceptor {
-    
-    
+
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept all business methods in this bean class
      * except those annotated with <code>@ExcludeClassInterceptors</code>
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
-     * @return - the result of the next method invoked. If a method returns void, proceed returns null. 
-     * For lifecycle callback interceptor methods, if there is no callback method defined on the bean class, 
-     * the invocation of proceed in the last interceptor method in the chain is a no-op, and null is returned. 
+     * @return - the result of the next method invoked. If a method returns void, proceed returns null.
+     * For lifecycle callback interceptor methods, if there is no callback method defined on the bean class,
+     * the invocation of proceed in the last interceptor method in the chain is a no-op, and null is returned.
      * If there is more than one such interceptor method, the invocation of proceed causes the container to execute those methods in order.
-     * 
      * @throws runtime exceptions or application exceptions that are allowed in the throws clause of the business method.
-     */    
+     */
     @AroundInvoke
-    public Object classInterceptor(InvocationContext ctx) throws Exception {        
+    public Object classInterceptor(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "classInterceptor");
         return ctx.proceed();
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept postConstruct of the bean
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
+     */
     @PostConstruct
-    public void classInterceptorPostConstruct(InvocationContext ctx) throws Exception {
+    public void classInterceptorPostConstruct(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "classInterceptorPostConstruct");
         ctx.proceed();
         return;
     }
-    
-    
+
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept postActivate of the bean
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
+     */
     @PostActivate
-    public void classInterceptorPostActivate(InvocationContext ctx) throws Exception {
+    public void classInterceptorPostActivate(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "classInterceptorPostActivate");
         ctx.proceed();
         return;
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept prePassivate of the bean.
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
+     */
     @PrePassivate
-    public void classInterceptorPrePassivate(InvocationContext ctx) throws Exception {
+    public void classInterceptorPrePassivate(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "classInterceptorPrePassivate");
         ctx.proceed();
         return;
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept preDestroy of the bean.
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
+     */
     @PreDestroy
-    public void classInterceptorPreDestroy(InvocationContext ctx) throws Exception {
+    public void classInterceptorPreDestroy(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "classInterceptorPreDestroy");
         ctx.proceed();
         return;

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/DDInterceptor.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/DDInterceptor.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/DDInterceptor.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/DDInterceptor.java Mon Jul  7 19:07:41 2014
@@ -20,83 +20,76 @@ import javax.interceptor.InvocationConte
 
 
 /**
- *
  * @version $Rev$ $Date$
  */
 public class DDInterceptor {
-    
-    
+
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept all business methods in this bean class
      * except those annotated with <code>@ExcludeddInterceptors</code>
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
-     * @return - the result of the next method invoked. If a method returns void, proceed returns null. 
-     * For lifecycle callback interceptor methods, if there is no callback method defined on the bean class, 
-     * the invocation of proceed in the last interceptor method in the chain is a no-op, and null is returned. 
+     * @return - the result of the next method invoked. If a method returns void, proceed returns null.
+     * For lifecycle callback interceptor methods, if there is no callback method defined on the bean class,
+     * the invocation of proceed in the last interceptor method in the chain is a no-op, and null is returned.
      * If there is more than one such interceptor method, the invocation of proceed causes the container to execute those methods in order.
-     * 
      * @throws runtime exceptions or application exceptions that are allowed in the throws clause of the business method.
-     */    
-    public Object ddInterceptor(InvocationContext ctx) throws Exception {        
+     */
+    public Object ddInterceptor(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "ddInterceptor");
         return ctx.proceed();
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept postConstruct of the bean
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
-    public void ddInterceptorPostConstruct(InvocationContext ctx) throws Exception {
+     */
+    public void ddInterceptorPostConstruct(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "ddInterceptorPostConstruct");
         ctx.proceed();
         return;
     }
-    
-    
+
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept postActivate of the bean
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
-    public void ddInterceptorPostActivate(InvocationContext ctx) throws Exception {
+     */
+    public void ddInterceptorPostActivate(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "ddInterceptorPostActivate");
         ctx.proceed();
         return;
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept prePassivate of the bean.
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
-    public void ddInterceptorPrePassivate(InvocationContext ctx) throws Exception {
+     */
+    public void ddInterceptorPrePassivate(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "ddInterceptorPrePassivate");
         ctx.proceed();
         return;
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept preDestroy of the bean.
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
-    public void ddInterceptorPreDestroy(InvocationContext ctx) throws Exception {
+     */
+    public void ddInterceptorPreDestroy(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "ddInterceptorPreDestroy");
         ctx.proceed();
         return;

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/DefaultInterceptor.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/DefaultInterceptor.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/DefaultInterceptor.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/DefaultInterceptor.java Mon Jul  7 19:07:41 2014
@@ -25,85 +25,78 @@ import javax.interceptor.InvocationConte
 
 
 /**
- *
  * @version $Rev$ $Date$
  */
 public class DefaultInterceptor {
-    
-    
+
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept all business methods in this bean class
      * except those annotated with <code>@ExcludeClassInterceptors</code>
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
-     * @return - the result of the next method invoked. If a method returns void, proceed returns null. 
-     * For lifecycle callback interceptor methods, if there is no callback method defined on the bean class, 
-     * the invocation of proceed in the last interceptor method in the chain is a no-op, and null is returned. 
+     * @return - the result of the next method invoked. If a method returns void, proceed returns null.
+     * For lifecycle callback interceptor methods, if there is no callback method defined on the bean class,
+     * the invocation of proceed in the last interceptor method in the chain is a no-op, and null is returned.
      * If there is more than one such interceptor method, the invocation of proceed causes the container to execute those methods in order.
-     * 
      * @throws Exception runtime exceptions or application exceptions that are allowed in the throws clause of the business method.
-     */    
+     */
     @AroundInvoke
-    public Object defaultInterceptor(InvocationContext ctx) throws Exception {        
+    public Object defaultInterceptor(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "defaultInterceptor");
         return ctx.proceed();
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept postConstruct of the bean
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws Exception runtime exceptions.
-     */    
+     */
     @PostConstruct
-    public void defaultInterceptorPostConstruct(InvocationContext ctx) throws Exception {
+    public void defaultInterceptorPostConstruct(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "defaultInterceptorPostConstruct");
         ctx.proceed();
     }
-    
-    
+
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept postActivate of the bean
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws Exception runtime exceptions.
-     */    
+     */
     @PostActivate
-    public void defaultInterceptorPostActivate(InvocationContext ctx) throws Exception {
+    public void defaultInterceptorPostActivate(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "defaultInterceptorPostActivate");
         ctx.proceed();
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept prePassivate of the bean.
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws Exception runtime exceptions.
-     */    
+     */
     @PrePassivate
-    public void defaultInterceptorPrePassivate(InvocationContext ctx) throws Exception {
+    public void defaultInterceptorPrePassivate(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "defaultInterceptorPrePassivate");
         ctx.proceed();
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept preDestroy of the bean.
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws Exception runtime exceptions.
-     */    
+     */
     @PreDestroy
-    public void defaultInterceptorPreDestroy(InvocationContext ctx) throws Exception {
+    public void defaultInterceptorPreDestroy(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "defaultInterceptorPreDestroy");
         ctx.proceed();
     }

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/Interceptor.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/Interceptor.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/Interceptor.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/Interceptor.java Mon Jul  7 19:07:41 2014
@@ -25,26 +25,25 @@ import javax.interceptor.InvocationConte
 import org.apache.openejb.test.SuperInterceptedBean;
 
 /**
- *
  * @version $Rev$ $Date$
  */
 public class Interceptor {
     
     /*@Resource
     static SessionContext sessionContext;*/
-    
+
     /**
      * This interceptor creates/updates an inner map for every method that it intercepts.
      * The inner map contains the array of method parameters in the key PARAMETERS.
      * The inner map contains the list of interceptor methods in the key INTERCEPTORS.
      * The inner map is put back into the contextData against the method name as the key.
-     * 
-     * @param ctx - InvocationContext
+     *
+     * @param ctx             - InvocationContext
      * @param interceptorName name of the interceptor
      * @return contextData - the contextData which now has been filled with a hashmap of hashmap.
      */
     @SuppressWarnings("unchecked")
-    public static Map<String, Object> profile(InvocationContext ctx, String interceptorName) {
+    public static Map<String, Object> profile(final InvocationContext ctx, final String interceptorName) {
         /*if (sessionContext != null) {
             System.out.println(sessionContext.lookup("java:comp/env"));        
         }
@@ -53,44 +52,43 @@ public class Interceptor {
         }*/
 
 
-        Map<String, Object> ctxData = ctx.getContextData();
+        final Map<String, Object> ctxData = ctx.getContextData();
 
-        String KEY;
+        final String KEY;
         if (ctx.getMethod() != null) {
-            KEY = ctx.getMethod().getName();    
-        }        
-        else {
+            KEY = ctx.getMethod().getName();
+        } else {
             KEY = (ctx.getTarget()).getClass().getSimpleName();
         }
-        
+
         Map<String, Object> innerMap = (HashMap<String, Object>) ctxData.get(KEY);
         innerMap = updateInterceptorsList(innerMap, interceptorName);
 
         // don't try to get parameters for call back methods (you'll get an IllegalStateException)
         if (ctx.getMethod() != null) {
-            Object[] params = ctx.getParameters();
+            final Object[] params = ctx.getParameters();
             innerMap.put("PARAMETERS", params);
         }
 
         ctxData.put(KEY, innerMap);
-        
-        return ctxData; 
+
+        return ctxData;
     }
 
     /**
      * This is invoked by the lifecycle interceptor callback methods that are defined inside a bean.
      */
     @SuppressWarnings("unchecked")
-    public static Map<String, Object> profile(SuperInterceptedBean bean, String interceptorName) {
-        Map<String, Object> ctxData = new HashMap<String, Object>();
-        
-        String KEY = bean.getClass().getSimpleName();
-        
+    public static Map<String, Object> profile(final SuperInterceptedBean bean, final String interceptorName) {
+        final Map<String, Object> ctxData = new HashMap<String, Object>();
+
+        final String KEY = bean.getClass().getSimpleName();
+
         Map<String, Object> innerMap = (HashMap<String, Object>) ctxData.get(KEY);
         innerMap = updateInterceptorsList(innerMap, interceptorName);
-        
+
         ctxData.put(KEY, innerMap);
-        return ctxData;        
+        return ctxData;
     }
 
     /**
@@ -99,18 +97,18 @@ public class Interceptor {
      * @return innerMap
      */
     @SuppressWarnings("unchecked")
-    private static Map<String, Object> updateInterceptorsList(Map<String, Object> innerMap, String interceptorName) {
-        if(innerMap == null) {
+    private static Map<String, Object> updateInterceptorsList(Map<String, Object> innerMap, final String interceptorName) {
+        if (innerMap == null) {
             innerMap = new HashMap<String, Object>();
-        }        
-        
+        }
+
         ArrayList<String> interceptorsList = (ArrayList<String>) innerMap.get("INTERCEPTORS");
-        if(interceptorsList == null) {
-            interceptorsList = new ArrayList<String>();            
+        if (interceptorsList == null) {
+            interceptorsList = new ArrayList<String>();
         }
         interceptorsList.add(interceptorName);
         innerMap.put("INTERCEPTORS", interceptorsList);
-        
+
         return innerMap;
     }
 

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/MethodInterceptor.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/MethodInterceptor.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/MethodInterceptor.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/MethodInterceptor.java Mon Jul  7 19:07:41 2014
@@ -22,43 +22,40 @@ import javax.interceptor.InvocationConte
 
 
 /**
- *
  * @version $Rev$ $Date$
  */
 public class MethodInterceptor {
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept all business methods in this bean class
      * except those annotated with <code>@ExcludeClassInterceptors</code>
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
-     * @return - the result of the next method invoked. If a method returns void, proceed returns null. 
-     * For lifecycle callback interceptor methods, if there is no callback method defined on the bean class, 
-     * the invocation of proceed in the last interceptor method in the chain is a no-op, and null is returned. 
+     * @return - the result of the next method invoked. If a method returns void, proceed returns null.
+     * For lifecycle callback interceptor methods, if there is no callback method defined on the bean class,
+     * the invocation of proceed in the last interceptor method in the chain is a no-op, and null is returned.
      * If there is more than one such interceptor method, the invocation of proceed causes the container to execute those methods in order.
-     * 
      * @throws runtime exceptions or application exceptions that are allowed in the throws clause of the business method.
-     */    
+     */
     @AroundInvoke
-    public Object methodInterceptor(InvocationContext ctx) throws Exception {
+    public Object methodInterceptor(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "methodInterceptor");
         return ctx.proceed();
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept postConstruct of the bean when intercepted at the class level.
      * Verify that this interceptor is not invoked when intercepted at the method-level.
-     * 
+     *
      * @throws runtime exceptions.
-     */    
+     */
     @PostConstruct
-    public void methodInterceptorPostConstruct(InvocationContext ctx) throws Exception {
+    public void methodInterceptorPostConstruct(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "methodInterceptorPostConstruct");
         ctx.proceed();
         return;
     }
-       
+
 }

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/SecondClassInterceptor.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/SecondClassInterceptor.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/SecondClassInterceptor.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/SecondClassInterceptor.java Mon Jul  7 19:07:41 2014
@@ -25,88 +25,81 @@ import javax.interceptor.InvocationConte
 
 
 /**
- *
  * @version $Rev$ $Date$
  */
 public class SecondClassInterceptor {
-    
-    
+
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept all business methods in this bean class
      * except those annotated with <code>@ExcludeClassInterceptors</code>
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
-     * @return - the result of the next method invoked. If a method returns void, proceed returns null. 
-     * For lifecycle callback interceptor methods, if there is no callback method defined on the bean class, 
-     * the invocation of proceed in the last interceptor method in the chain is a no-op, and null is returned. 
+     * @return - the result of the next method invoked. If a method returns void, proceed returns null.
+     * For lifecycle callback interceptor methods, if there is no callback method defined on the bean class,
+     * the invocation of proceed in the last interceptor method in the chain is a no-op, and null is returned.
      * If there is more than one such interceptor method, the invocation of proceed causes the container to execute those methods in order.
-     * 
      * @throws runtime exceptions or application exceptions that are allowed in the throws clause of the business method.
-     */    
+     */
     @AroundInvoke
-    public Object secondClassInterceptor(InvocationContext ctx) throws Exception {        
+    public Object secondClassInterceptor(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "secondClassInterceptor");
         return ctx.proceed();
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept postConstruct of the bean
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
+     */
     @PostConstruct
-    public void secondClassInterceptorPostConstruct(InvocationContext ctx) throws Exception {
+    public void secondClassInterceptorPostConstruct(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "secondClassInterceptorPostConstruct");
         ctx.proceed();
         return;
     }
-    
-    
+
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept postActivate of the bean
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
+     */
     @PostActivate
-    public void secondClassInterceptorPostActivate(InvocationContext ctx) throws Exception {
+    public void secondClassInterceptorPostActivate(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "secondClassInterceptorPostActivate");
         ctx.proceed();
         return;
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept prePassivate of the bean.
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
+     */
     @PrePassivate
-    public void secondClassInterceptorPrePassivate(InvocationContext ctx) throws Exception {
+    public void secondClassInterceptorPrePassivate(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "secondClassInterceptorPrePassivate");
         ctx.proceed();
         return;
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept preDestroy of the bean.
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
+     */
     @PreDestroy
-    public void secondClassInterceptorPreDestroy(InvocationContext ctx) throws Exception {
+    public void secondClassInterceptorPreDestroy(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "secondClassInterceptorPreDestroy");
         ctx.proceed();
         return;

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/SuperClassInterceptor.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/SuperClassInterceptor.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/SuperClassInterceptor.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/interceptor/SuperClassInterceptor.java Mon Jul  7 19:07:41 2014
@@ -27,52 +27,48 @@ import javax.interceptor.InvocationConte
 
 
 /**
- *
  * @version $Rev$ $Date$
  */
 public class SuperClassInterceptor {
-    
+
     @Resource
     SessionContext sessionContext;
 
     /**
-     * 
+     *
      */
     public SuperClassInterceptor() {
         super();
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept all business methods in this bean class
      * except those annotated with <code>@ExcludeClassInterceptors</code>
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
-     * @return - the result of the next method invoked. If a method returns void, proceed returns null. 
-     * For lifecycle callback interceptor methods, if there is no callback method defined on the bean class, 
-     * the invocation of proceed in the last interceptor method in the chain is a no-op, and null is returned. 
+     * @return - the result of the next method invoked. If a method returns void, proceed returns null.
+     * For lifecycle callback interceptor methods, if there is no callback method defined on the bean class,
+     * the invocation of proceed in the last interceptor method in the chain is a no-op, and null is returned.
      * If there is more than one such interceptor method, the invocation of proceed causes the container to execute those methods in order.
-     * 
      * @throws runtime exceptions or application exceptions that are allowed in the throws clause of the business method.
-     */    
+     */
     @SuppressWarnings("unchecked")
     @AroundInvoke
-    public Object superClassInterceptor(InvocationContext ctx) throws Exception {
+    public Object superClassInterceptor(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "superClassInterceptor");
         return ctx.proceed();
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept postConstruct of the bean
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
+     */
     @PostConstruct
-    public void superClassInterceptorPostConstruct(InvocationContext ctx) throws Exception {
+    public void superClassInterceptorPostConstruct(final InvocationContext ctx) throws Exception {
         /*if (sessionContext != null) {
             System.out.println(sessionContext.lookup("java:comp/env"));        
         }
@@ -84,48 +80,45 @@ public class SuperClassInterceptor {
         ctx.proceed();
         return;
     }
-    
-    
+
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept postActivate of the bean
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
+     */
     @PostActivate
-    public void superClassInterceptorPostActivate(InvocationContext ctx) throws Exception {
+    public void superClassInterceptorPostActivate(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "superClassInterceptorPostActivate");
         ctx.proceed();
         return;
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept prePassivate of the bean.
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
+     */
     @PrePassivate
-    public void superClassInterceptorPrePassivate(InvocationContext ctx) throws Exception {
+    public void superClassInterceptorPrePassivate(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "superClassInterceptorPrePassivate");
         ctx.proceed();
         return;
     }
-    
+
     /**
-     * The interceptor method. 
+     * The interceptor method.
      * This should intercept preDestroy of the bean.
-     * 
+     *
      * @param ctx - InvocationContext
-     * 
      * @throws runtime exceptions.
-     */    
+     */
     @PreDestroy
-    public void superClassInterceptorPreDestroy(InvocationContext ctx) throws Exception {
+    public void superClassInterceptorPreDestroy(final InvocationContext ctx) throws Exception {
         Interceptor.profile(ctx, "superClassInterceptorPreDestroy");
         ctx.proceed();
         return;

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/AnnotatedFieldInjectionMdbBean.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/AnnotatedFieldInjectionMdbBean.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/AnnotatedFieldInjectionMdbBean.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/AnnotatedFieldInjectionMdbBean.java Mon Jul  7 19:07:41 2014
@@ -52,8 +52,8 @@ import javax.persistence.PersistenceUnit
 import javax.sql.DataSource;
 
 @MessageDriven(activationConfig = {
-        @ActivationConfigProperty(propertyName = "destinationType", propertyValue = "javax.jms.Queue"),
-        @ActivationConfigProperty(propertyName = "destination", propertyValue = "AnnotatedFieldInjectionMdb")})
+    @ActivationConfigProperty(propertyName = "destinationType", propertyValue = "javax.jms.Queue"),
+    @ActivationConfigProperty(propertyName = "destination", propertyValue = "AnnotatedFieldInjectionMdb")})
 public class AnnotatedFieldInjectionMdbBean implements MessageListener {
     @Resource
     private MessageDrivenContext mdbContext;
@@ -108,7 +108,7 @@ public class AnnotatedFieldInjectionMdbB
     public void start() {
         try {
             mdbInvoker = new MdbInvoker(coonnectionFactory, this);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             throw new EJBException(e);
         }
     }
@@ -118,7 +118,7 @@ public class AnnotatedFieldInjectionMdbB
         if (null != mdbInvoker) {
             try {
                 mdbInvoker.destroy();
-            } catch (Throwable e) {
+            } catch (final Throwable e) {
                 //Ignore
             }
         }
@@ -133,11 +133,11 @@ public class AnnotatedFieldInjectionMdbB
 //                    "***************************************\n\n");
             try {
                 message.acknowledge();
-            } catch (JMSException e) {
+            } catch (final JMSException e) {
                 e.printStackTrace();
             }
             mdbInvoker.onMessage(message);
-        } catch (Throwable e) {
+        } catch (final Throwable e) {
             e.printStackTrace();
         }
     }
@@ -145,7 +145,7 @@ public class AnnotatedFieldInjectionMdbB
     public void lookupEntityBean() throws TestFailureException {
         try {
             Assert.assertNotNull("The EJBObject is null", bmpHome);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -153,7 +153,7 @@ public class AnnotatedFieldInjectionMdbB
     public void lookupStatefulBean() throws TestFailureException {
         try {
             Assert.assertNotNull("The EJBObject is null", statefulHome);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -161,7 +161,7 @@ public class AnnotatedFieldInjectionMdbB
     public void lookupStatelessBean() throws TestFailureException {
         try {
             Assert.assertNotNull("The EJBObject is null", statelessHome);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -169,7 +169,7 @@ public class AnnotatedFieldInjectionMdbB
     public void lookupStatelessBusinessLocal() throws TestFailureException {
         try {
             Assert.assertNotNull("The EJB BusinessLocal is null", statelessBusinessLocal);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -177,7 +177,7 @@ public class AnnotatedFieldInjectionMdbB
     public void lookupStatelessBusinessRemote() throws TestFailureException {
         try {
             Assert.assertNotNull("The EJB BusinessRemote is null", statelessBusinessRemote);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -185,7 +185,7 @@ public class AnnotatedFieldInjectionMdbB
     public void lookupStatefulBusinessLocal() throws TestFailureException {
         try {
             Assert.assertNotNull("The EJB BusinessLocal is null", statefulBusinessLocal);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -193,7 +193,7 @@ public class AnnotatedFieldInjectionMdbB
     public void lookupStatefulBusinessRemote() throws TestFailureException {
         try {
             Assert.assertNotNull("The EJB BusinessRemote is null", statefulBusinessRemote);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -203,7 +203,7 @@ public class AnnotatedFieldInjectionMdbB
             final String expected = "1";
             Assert.assertNotNull("The String looked up is null", striing);
             Assert.assertEquals(expected, striing);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -215,7 +215,7 @@ public class AnnotatedFieldInjectionMdbB
             Assert.assertNotNull("The Double looked up is null", doouble);
             Assert.assertEquals(expected, doouble);
 
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -226,7 +226,7 @@ public class AnnotatedFieldInjectionMdbB
 
             Assert.assertNotNull("The Long looked up is null", loong);
             Assert.assertEquals(expected, loong);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -237,7 +237,7 @@ public class AnnotatedFieldInjectionMdbB
 
             Assert.assertNotNull("The Float looked up is null", flooat);
             Assert.assertEquals(expected, flooat);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -249,7 +249,7 @@ public class AnnotatedFieldInjectionMdbB
             Assert.assertNotNull("The Integer looked up is null", inteeger);
             Assert.assertEquals(expected, inteeger);
 
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -260,7 +260,7 @@ public class AnnotatedFieldInjectionMdbB
 
             Assert.assertNotNull("The Short looked up is null", shoort);
             Assert.assertEquals(expected, shoort);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -271,7 +271,7 @@ public class AnnotatedFieldInjectionMdbB
 
             Assert.assertNotNull("The Boolean looked up is null", booolean);
             Assert.assertEquals(expected, booolean);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -282,7 +282,7 @@ public class AnnotatedFieldInjectionMdbB
 
             Assert.assertNotNull("The Byte looked up is null", byyte);
             Assert.assertEquals(expected, byyte);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -293,7 +293,7 @@ public class AnnotatedFieldInjectionMdbB
 
             Assert.assertNotNull("The Character looked up is null", chaaracter);
             Assert.assertEquals(expected, chaaracter);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -301,7 +301,7 @@ public class AnnotatedFieldInjectionMdbB
     public void lookupResource() throws TestFailureException {
         try {
             Assert.assertNotNull("The DataSource is null", daataSource);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -312,11 +312,11 @@ public class AnnotatedFieldInjectionMdbB
                 testJmsConnection(coonnectionFactory.createConnection());
                 testJmsConnection(queueCoonnectionFactory.createConnection());
                 testJmsConnection(topicCoonnectionFactory.createConnection());
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 e.printStackTrace();
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -334,7 +334,7 @@ public class AnnotatedFieldInjectionMdbB
     public void lookupPersistenceUnit() throws TestFailureException {
         try {
             Assert.assertNotNull("The EntityManagerFactory is null", emf);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -346,10 +346,10 @@ public class AnnotatedFieldInjectionMdbB
             try {
                 // call a do nothing method to assure entity manager actually exists
                 em.getFlushMode();
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -357,7 +357,7 @@ public class AnnotatedFieldInjectionMdbB
     public void lookupMessageDrivenContext() throws TestFailureException {
         try {
             Assert.assertNotNull("The MessageDrivenContext is null", mdbContext);
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
 

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/BasicMdbBean.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/BasicMdbBean.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/BasicMdbBean.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/BasicMdbBean.java Mon Jul  7 19:07:41 2014
@@ -42,7 +42,7 @@ public class BasicMdbBean implements Bas
         try {
             final ConnectionFactory connectionFactory = (ConnectionFactory) new InitialContext().lookup("java:comp/env/jms");
             mdbInvoker = new MdbInvoker(connectionFactory, this);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             throw new EJBException(e);
         }
     }
@@ -56,11 +56,11 @@ public class BasicMdbBean implements Bas
 //                    "***************************************\n\n");
             try {
                 message.acknowledge();
-            } catch (JMSException e) {
+            } catch (final JMSException e) {
                 e.printStackTrace();
             }
             mdbInvoker.onMessage(message);
-        } catch (Throwable e) {
+        } catch (final Throwable e) {
             e.printStackTrace();
         }
     }
@@ -169,28 +169,28 @@ public class BasicMdbBean implements Bas
         try {
             mdbContext.getEJBHome();
             policy.allow(OperationsPolicy.Context_getEJBHome);
-        } catch (IllegalStateException ignored) {
+        } catch (final IllegalStateException ignored) {
         }
 
         /*[1] Test getCallerPrincipal /////////*/
         try {
             mdbContext.getCallerPrincipal();
             policy.allow(OperationsPolicy.Context_getCallerPrincipal);
-        } catch (IllegalStateException ignored) {
+        } catch (final IllegalStateException ignored) {
         }
 
         /*[2] Test isCallerInRole /////////////*/
         try {
             mdbContext.isCallerInRole("TheMan");
             policy.allow(OperationsPolicy.Context_isCallerInRole);
-        } catch (IllegalStateException ignored) {
+        } catch (final IllegalStateException ignored) {
         }
 
         /*[3] Test getRollbackOnly ////////////*/
         try {
             mdbContext.getRollbackOnly();
             policy.allow(OperationsPolicy.Context_getRollbackOnly);
-        } catch (IllegalStateException ignored) {
+        } catch (final IllegalStateException ignored) {
         }
 
         /*[4] Test setRollbackOnly ////////////*/
@@ -200,7 +200,7 @@ public class BasicMdbBean implements Bas
         try {
             mdbContext.getUserTransaction();
             policy.allow(OperationsPolicy.Context_getUserTransaction);
-        } catch (IllegalStateException ignored) {
+        } catch (final IllegalStateException ignored) {
         }
 
         /*[6] Test getEJBObject ///////////////
@@ -220,15 +220,15 @@ public class BasicMdbBean implements Bas
             final String actual = (String) jndiContext.lookup("java:comp/env/stateless/references/JNDI_access_to_java_comp_env");
 
             policy.allow(OperationsPolicy.JNDI_access_to_java_comp_env);
-        } catch (IllegalStateException ignored) {
-        } catch (javax.naming.NamingException ignored) {
+        } catch (final IllegalStateException ignored) {
+        } catch (final javax.naming.NamingException ignored) {
         }
 
         /*[11] Test lookup /////////*/
         try {
             mdbContext.lookup("stateless/references/JNDI_access_to_java_comp_env");
             policy.allow(OperationsPolicy.Context_lookup);
-        } catch (IllegalArgumentException ignored) {
+        } catch (final IllegalArgumentException ignored) {
         }
 
         allowedOperationsTable.put(methodName, policy);

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/BasicMdbObject.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/BasicMdbObject.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/BasicMdbObject.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/BasicMdbObject.java Mon Jul  7 19:07:41 2014
@@ -32,7 +32,6 @@ public interface BasicMdbObject {
 
     /**
      * Throws an ApplicationException when invoked
-     *
      */
     void throwApplicationException() throws ApplicationException;
 
@@ -41,7 +40,6 @@ public interface BasicMdbObject {
      * This is a system exception and should result in the
      * destruction of the instance and invalidation of the
      * remote reference.
-     *
      */
     void throwSystemException_NullPointer();
 

Modified: tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/ContextLookupMdbBean.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/ContextLookupMdbBean.java?rev=1608563&r1=1608562&r2=1608563&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/ContextLookupMdbBean.java (original)
+++ tomee/tomee/branches/tomee-1.7.x/itests/openejb-itests-beans/src/main/java/org/apache/openejb/test/mdb/ContextLookupMdbBean.java Mon Jul  7 19:07:41 2014
@@ -60,7 +60,7 @@ public class ContextLookupMdbBean implem
         try {
             final ConnectionFactory connectionFactory = (ConnectionFactory) new InitialContext().lookup("java:comp/env/jms");
             mdbInvoker = new MdbInvoker(connectionFactory, this);
-        } catch (Exception e) {
+        } catch (final Exception e) {
             throw new EJBException(e);
         }
     }
@@ -74,11 +74,11 @@ public class ContextLookupMdbBean implem
 //                    "***************************************\n\n");
             try {
                 message.acknowledge();
-            } catch (JMSException e) {
+            } catch (final JMSException e) {
                 e.printStackTrace();
             }
             mdbInvoker.onMessage(message);
-        } catch (Throwable e) {
+        } catch (final Throwable e) {
             e.printStackTrace();
         }
     }
@@ -92,10 +92,10 @@ public class ContextLookupMdbBean implem
 
                 final BasicBmpObject object = home.createObject("Enc Bean");
                 Assert.assertNotNull("The EJBObject is null", object);
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -109,10 +109,10 @@ public class ContextLookupMdbBean implem
 
                 final BasicStatefulObject object = home.createObject("Enc Bean");
                 Assert.assertNotNull("The EJBObject is null", object);
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -126,10 +126,10 @@ public class ContextLookupMdbBean implem
 
                 final BasicStatelessObject object = home.createObject();
                 Assert.assertNotNull("The EJBObject is null", object);
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -140,10 +140,10 @@ public class ContextLookupMdbBean implem
             try {
                 final BasicStatelessBusinessLocal object = (BasicStatelessBusinessLocal) mdbContext.lookup("stateless/beanReferences/stateless-business-local");
                 Assert.assertNotNull("The EJB BusinessLocal is null", object);
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -154,10 +154,10 @@ public class ContextLookupMdbBean implem
             try {
                 final BasicStatelessBusinessRemote object = (BasicStatelessBusinessRemote) mdbContext.lookup("stateless/beanReferences/stateless-business-remote");
                 Assert.assertNotNull("The EJB BusinessRemote is null", object);
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -168,10 +168,10 @@ public class ContextLookupMdbBean implem
             try {
                 final BasicStatefulBusinessLocal object = (BasicStatefulBusinessLocal) mdbContext.lookup("stateless/beanReferences/stateful-business-local");
                 Assert.assertNotNull("The EJB BusinessLocal is null", object);
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -182,10 +182,10 @@ public class ContextLookupMdbBean implem
             try {
                 final BasicStatefulBusinessRemote object = (BasicStatefulBusinessRemote) mdbContext.lookup("stateless/beanReferences/stateful-business-remote");
                 Assert.assertNotNull("The EJB BusinessRemote is null", object);
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -200,10 +200,10 @@ public class ContextLookupMdbBean implem
                 Assert.assertNotNull("The String looked up is null", actual);
                 Assert.assertEquals(expected, actual);
 
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -218,10 +218,10 @@ public class ContextLookupMdbBean implem
                 Assert.assertNotNull("The Double looked up is null", actual);
                 Assert.assertEquals(expected, actual);
 
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -236,10 +236,10 @@ public class ContextLookupMdbBean implem
                 Assert.assertNotNull("The Long looked up is null", actual);
                 Assert.assertEquals(expected, actual);
 
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -254,10 +254,10 @@ public class ContextLookupMdbBean implem
                 Assert.assertNotNull("The Float looked up is null", actual);
                 Assert.assertEquals(expected, actual);
 
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -272,10 +272,10 @@ public class ContextLookupMdbBean implem
                 Assert.assertNotNull("The Integer looked up is null", actual);
                 Assert.assertEquals(expected, actual);
 
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -290,10 +290,10 @@ public class ContextLookupMdbBean implem
                 Assert.assertNotNull("The Short looked up is null", actual);
                 Assert.assertEquals(expected, actual);
 
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -308,10 +308,10 @@ public class ContextLookupMdbBean implem
                 Assert.assertNotNull("The Boolean looked up is null", actual);
                 Assert.assertEquals(expected, actual);
 
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -326,10 +326,10 @@ public class ContextLookupMdbBean implem
                 Assert.assertNotNull("The Byte looked up is null", actual);
                 Assert.assertEquals(expected, actual);
 
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -344,10 +344,10 @@ public class ContextLookupMdbBean implem
                 Assert.assertNotNull("The Character looked up is null", actual);
                 Assert.assertEquals(expected, actual);
 
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -359,10 +359,10 @@ public class ContextLookupMdbBean implem
                 final Object obj = mdbContext.lookup("datasource");
                 Assert.assertNotNull("The DataSource is null", obj);
                 Assert.assertTrue("Not an instance of DataSource", obj instanceof DataSource);
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -388,11 +388,11 @@ public class ContextLookupMdbBean implem
                 Assert.assertTrue("Not an instance of QueueConnectionFactory", obj instanceof QueueConnectionFactory);
                 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());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -414,10 +414,10 @@ public class ContextLookupMdbBean implem
                 final EntityManagerFactory emf = (EntityManagerFactory) mdbContext.lookup("persistence/TestUnit");
                 Assert.assertNotNull("The EntityManagerFactory is null", emf);
 
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -431,10 +431,10 @@ public class ContextLookupMdbBean implem
 
                 // call a do nothing method to assure entity manager actually exists
                 em.getFlushMode();
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }
     }
@@ -456,10 +456,10 @@ public class ContextLookupMdbBean implem
 
                 // verify context was set via legacy set method
                 Assert.assertNotNull("The MessageDrivenContext is null from setter method", mdbContext);
-            } catch (Exception e) {
+            } catch (final Exception e) {
                 Assert.fail("Received Exception " + e.getClass() + " : " + e.getMessage());
             }
-        } catch (AssertionFailedError afe) {
+        } catch (final AssertionFailedError afe) {
             throw new TestFailureException(afe);
         }