You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openwebbeans.apache.org by Mark Struberg <st...@yahoo.de> on 2010/07/26 14:39:44 UTC
Re: svn commit: r979198 - /openwebbeans/trunk/webbeans-openejb/src/main/java/org/apache/webbeans/ejb/EjbPlugin.java
this doesn't compile due to 5 checkstyle violations.
fix is otw.
LieGrue,
strub
----- Original Message ----
> From: "gerdogdu@apache.org" <ge...@apache.org>
> To: commits@openwebbeans.apache.org
> Sent: Mon, July 26, 2010 10:37:24 AM
> Subject: svn commit: r979198 -
>/openwebbeans/trunk/webbeans-openejb/src/main/java/org/apache/webbeans/ejb/EjbPlugin.java
>
>
> Author: gerdogdu
> Date: Mon Jul 26 08:37:23 2010
> New Revision: 979198
>
> URL: http://svn.apache.org/viewvc?rev=979198&view=rev
> Log:
> [OWB-426] Tweak EJBPlugin to work with Standalone Tests
>
> Modified:
>
>openwebbeans/trunk/webbeans-openejb/src/main/java/org/apache/webbeans/ejb/EjbPlugin.java
>
>
> Modified:
>openwebbeans/trunk/webbeans-openejb/src/main/java/org/apache/webbeans/ejb/EjbPlugin.java
>
> URL: :
>http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-openejb/src/main/java/org/apache/webbeans/ejb/EjbPlugin.java?rev=979198&r1=979197&r2=979198&view=diff
>
> ==============================================================================
> ---
>openwebbeans/trunk/webbeans-openejb/src/main/java/org/apache/webbeans/ejb/EjbPlugin.java
> a (original)
> +++
>openwebbeans/trunk/webbeans-openejb/src/main/java/org/apache/webbeans/ejb/EjbPlugin.java
> Mon Jul 26 08:37:23 2010
> @@ -101,6 +101,10 @@ public class EjbPlugin extends AbstractO
> private static final SecurityService SECURITY_SERVICE = new
>OpenEJBSecurityService();
>
> private final Map<String, JndiNameStrategy> nameStrategies = new
>TreeMap<String, JndiNameStrategy>();
> +
> + //This is here for standalone tests are correctly run
> + //Not used in anywhere
> + public static boolean CONFIGURED_FOR_USED_IN_TEST = false;
>
> public EjbPlugin()
> {
> @@ -296,8 +300,12 @@ public class EjbPlugin extends AbstractO
>
> public boolean isSessionBean(Class<?> clazz)
> {
> - if (this.containerSystem == null)
> + //This is used in tests, because in reality containerSystem is not
>null
>
> + if (this.containerSystem == null || CONFIGURED_FOR_USED_IN_TEST)
> {
> + //Used for tests
> + CONFIGURED_FOR_USED_IN_TEST = false;
> +
> this.containerSystem =
>SystemInstance.get().getComponent(ContainerSystem.class);
> Container[] containers = this.containerSystem.containers();
> for (Container container : containers)
>
>
>