Adding annotations to MDR via bean metadata

[kabirkhan]

I am creating a bean metadata with some annotations added

 AbstractBeanMetaData metaData = new AbstractBeanMetaData("McBean", McBeanAnnotationsFromMetaData.class.getName());
 metaData.setAccessMode(BeanAccessMode.ALL);

 AbstractConstructorMetaData ctor = new AbstractConstructorMetaData();
 AnnotationMetaData initializer = new AbstractAnnotationMetaData("@" + Initializer.class.getName());
 ctor.setAnnotations(Collections.singleton(initializer));
 metaData.setConstructor(ctor);

I create a KCC from that and deploy it. When I read the metadata from my describe action there are no annotations in the metadata:

 private AnnotatedType<T> createMdrDecoratedAnnotatedType(Class<T> clazz)
 {
 AnnotatedType<T> type = context.getManager().createAnnotatedType(clazz);

 KernelController controller = (KernelController)context.getController();
 Kernel kernel = controller.getKernel();
 KernelMetaDataRepository repository = kernel.getMetaDataRepository();
 MetaData metaData = repository.getMetaData(context);

 return MDRAnnotatedTypeFactory.getInstance().getAnnotatedType(type, metaData);
 }

Am I adding them wrong?

[alesj]

KernelScopeInfo doesn't (yet) add ctor annotations:

 public void updateMetaData(MutableMetaDataRepository repository, ControllerContext context, MutableMetaDataLoader mutable, boolean add)
 {
 if (context instanceof KernelControllerContext == false)
 return;

 KernelControllerContext kernelContext = (KernelControllerContext) context;
 updateClassAnnotations(mutable, kernelContext, add);
 if (mutable instanceof ComponentMutableMetaData)
 updatePropertyAnnotations(repository, (ComponentMutableMetaData) mutable, kernelContext, add);
 else if (add == true)
 log.warn("Unable to add properties to mutable metadata that does not support components: " + mutable + " for " + context.toShortString());
 }

Only class and properties.

[kabirkhan]

Yup I found that, and also that Web Beans ignores the mdr stuff anyway https://jira.jboss.org/jira/browse/WBRI-370

The MetaData seems fine for properties, I was looking at the wrong fields in the debugger when I made the initial post. I'll implement constructor annotations, do you remember where this is being tested from?

[kabirkhan]

While on this topic, should we support metadata annotations for methods as well? I think we should be able to do things like (using metadata) since @Initializer methods are web beans injection points just like constructors and properties:

class Bean
{
 @Initializer
 public void init(@Current Thing t, @Current Other o)
 {

 }
}

[kabirkhan]

I'll implement constructor annotations, do you remember where this is being tested from?

https://jira.jboss.org/jira/browse/JBKERNEL-52

[kabirkhan]

I added org.jboss.test.kernel.annotations.test.constructor.AnnotatedConstructorTestCase for this. If you want it somewhere else just let me know