You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by rm...@apache.org on 2012/02/11 19:50:51 UTC

svn commit: r1243119 - /openejb/trunk/openejb/container/openejb-core/src/test/java/org/apache/openejb/interceptors/InterceptorBindingEjbTest.java

Author: rmannibucau
Date: Sat Feb 11 18:50:50 2012
New Revision: 1243119

URL: http://svn.apache.org/viewvc?rev=1243119&view=rev
Log:
reproduced http://openejb.979440.n4.nabble.com/Wrong-CDI-Interception-in-OpenEJB-td4378502.html

Modified:
    openejb/trunk/openejb/container/openejb-core/src/test/java/org/apache/openejb/interceptors/InterceptorBindingEjbTest.java

Modified: openejb/trunk/openejb/container/openejb-core/src/test/java/org/apache/openejb/interceptors/InterceptorBindingEjbTest.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb/container/openejb-core/src/test/java/org/apache/openejb/interceptors/InterceptorBindingEjbTest.java?rev=1243119&r1=1243118&r2=1243119&view=diff
==============================================================================
--- openejb/trunk/openejb/container/openejb-core/src/test/java/org/apache/openejb/interceptors/InterceptorBindingEjbTest.java (original)
+++ openejb/trunk/openejb/container/openejb-core/src/test/java/org/apache/openejb/interceptors/InterceptorBindingEjbTest.java Sat Feb 11 18:50:50 2012
@@ -7,6 +7,7 @@ import org.apache.openejb.jee.Intercepto
 import org.apache.openejb.jee.StatelessBean;
 import org.apache.openejb.junit.ApplicationComposer;
 import org.apache.openejb.junit.Module;
+import org.junit.Ignore;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 
@@ -30,7 +31,7 @@ public class InterceptorBindingEjbTest {
     @EJB
     private EJB2 ejb2;
 
-    @Test
+    @Test @Ignore("doesn't pass today")
     public void test() {
         ejb2.foo();
         assertEquals(1, MarkedInterceptor.CLASSES.size());
@@ -58,10 +59,10 @@ public class InterceptorBindingEjbTest {
     public @interface Interception {
     }
 
-    @javax.interceptor.Interceptor
     @Interception
+    @javax.interceptor.Interceptor
     public static class MarkedInterceptor {
-        public static Collection<String> CLASSES = new ArrayList<String>();
+        public static final Collection<String> CLASSES = new ArrayList<String>();
 
         @AroundInvoke
         public Object intercept(InvocationContext invocationContext) throws Exception {
@@ -80,6 +81,9 @@ public class InterceptorBindingEjbTest {
         @EJB
         private EJB1 ejb1;
 
+        @Interception
+        public void notCalled() {}
+
         public String foo() {
             ejb1.foo();
             return "ok";