X-Git-Url: http://wamblee.org/gitweb/?a=blobdiff_plain;f=support%2Fcdi%2Fsrc%2Fmain%2Fjava%2Forg%2Fwamblee%2Fcdi%2FBeanManagerLookup.java;h=8d9ab46d4c252ab11675e706db2e2f0c52c445c3;hb=bd12e649d9e50c3db64fc05d01fef613b128448d;hp=50bed00a907aae1be4913e5a3108da8b75f6944b;hpb=6842c70f307b15a239f8298749507b1ab8b6fd90;p=utils

diff --git a/support/cdi/src/main/java/org/wamblee/cdi/BeanManagerLookup.java b/support/cdi/src/main/java/org/wamblee/cdi/BeanManagerLookup.java
index 50bed00a..8d9ab46d 100644
--- a/support/cdi/src/main/java/org/wamblee/cdi/BeanManagerLookup.java
+++ b/support/cdi/src/main/java/org/wamblee/cdi/BeanManagerLookup.java
@@ -23,11 +23,15 @@ import javax.naming.NamingException;
 
 /**
  * Class that encapsulates beanmanager lookup in a way so that the lookup can be
- * explicitly overriden (e.g. for unit test).
+ * explicitly overriden (for instance for unit test).
  * 
  * In case no bean manager is found the beanmanager is set to null and the
  * problem is logged.
  * 
+ * This class does caching of the obtained reference to the bean manager. 
+ * As a result, it is necessary to call {@link #setBeanManager(BeanManager)}
+ * in the setup phase of unit tests to make sure that the lookup is done again. 
+ * 
  * @author Erik Brakkee
  */
 public class BeanManagerLookup {
@@ -36,7 +40,7 @@ public class BeanManagerLookup {
         .getLogger(BeanManagerLookup.class.getName());
 
     public static final String BEAN_MANAGER_JNDI = "java:comp/BeanManager";
-    private static BeanManager mgr = null;
+    private static BeanManager BEAN_MANAGER = null;
 
     /**
      * Sets the bean manager (mainly for testability).
@@ -45,7 +49,7 @@ public class BeanManagerLookup {
      *            Bean manager.
      */
     public static void setBeanManager(BeanManager aMgr) {
-        mgr = aMgr;
+        BEAN_MANAGER = aMgr;
     }
 
     /**
@@ -54,16 +58,16 @@ public class BeanManagerLookup {
      * @return Bean manager.
      */
     public static BeanManager lookup() {
-        if (mgr == null) {
+        if (BEAN_MANAGER == null) {
             try {
                 InitialContext ctx = new InitialContext();
-                mgr = (BeanManager) ctx.lookup(BEAN_MANAGER_JNDI);
+                BEAN_MANAGER = (BeanManager) ctx.lookup(BEAN_MANAGER_JNDI);
                 LOGGER.info("Beanmanager successfully located");
             } catch (NamingException e) {
                 LOGGER.warning("No beanmanager was found, using null");
-                mgr = null;
+                BEAN_MANAGER = null;
             }
         }
-        return mgr;
+        return BEAN_MANAGER;
     }
 }