WELD-299

impl/src/main/java/org/jboss/weld/metadata/cache/BindingTypeModel.java

@@ -16,6 +16,8 @@
  */
 package org.jboss.weld.metadata.cache;
 
+import static org.jboss.weld.logging.Category.REFLECTION;
+import static org.jboss.weld.logging.LoggerFactory.loggerFactory;
 import static org.jboss.weld.logging.messages.MetadataMessage.NON_BINDING_MEMBER_TYPE;
 
 import java.lang.annotation.Annotation;
@@ -25,12 +27,12 @@
 import javax.enterprise.util.Nonbinding;
 import javax.inject.Qualifier;
 
-import org.jboss.weld.DefinitionException;
 import org.jboss.weld.WeldException;
 import org.jboss.weld.introspector.WeldMethod;
 import org.jboss.weld.resources.ClassTransformer;
 import org.jboss.weld.util.Reflections;
 import org.jboss.weld.util.collections.Arrays2;
+import org.slf4j.cal10n.LocLogger;
 
 /**
  * 
@@ -41,7 +43,7 @@
  */
 public class BindingTypeModel<T extends Annotation> extends AnnotationModel<T>
 {
-
+   private static final LocLogger log = loggerFactory().getLogger(REFLECTION);
 
    private static final Set<Class<? extends Annotation>> META_ANNOTATIONS = Arrays2.<Class<? extends Annotation>>asSet(Qualifier.class);
 
@@ -79,7 +81,8 @@ private void checkArrayAndAnnotationValuedMembers()
       {
          if ((Reflections.isArrayType(annotatedMethod.getJavaClass()) || Annotation.class.isAssignableFrom(annotatedMethod.getJavaClass())) && !nonBindingTypes.contains(annotatedMethod))
          {
-            throw new DefinitionException(NON_BINDING_MEMBER_TYPE, annotatedMethod);
+            super.valid = false;
+            log.debug(NON_BINDING_MEMBER_TYPE, annotatedMethod);
          }
       }
 

tests/src/test/java/org/jboss/weld/tests/beanManager/annotation/ManagerAnnotationTest.java

@@ -0,0 +1,50 @@
+package org.jboss.weld.tests.beanManager.annotation;
+
+import javax.persistence.PersistenceContext;
+
+import org.jboss.testharness.impl.packaging.Artifact;
+import org.jboss.weld.test.AbstractWeldTest;
+import org.testng.annotations.Test;
+
+@Artifact
+public class ManagerAnnotationTest extends AbstractWeldTest
+{
+
+   @Test
+   public void testIsQualifier() throws Exception
+   {
+      assert !getCurrentManager().isQualifier(PersistenceContext.class);
+   }
+
+   @Test
+   public void testIsInterceptorBinding() throws Exception
+   {
+      assert !getCurrentManager().isInterceptorBinding(PersistenceContext.class);
+   }
+
+   @Test
+   public void testIsNormalScope() throws Exception
+   {
+      assert !getCurrentManager().isNormalScope(PersistenceContext.class);
+   }
+
+   @Test
+   public void testIsPassivatingScope() throws Exception
+   {
+      assert !getCurrentManager().isPassivatingScope(PersistenceContext.class);
+   }
+
+   @Test
+   public void testIsScope() throws Exception
+   {
+      assert !getCurrentManager().isScope(PersistenceContext.class);
+   }
+
+   @Test
+   public void testIsStereotype() throws Exception
+   {
+      assert !getCurrentManager().isStereotype(PersistenceContext.class);
+   }
+
+
+}