Skip to content

Commit

Permalink
WELD-1045 ProcessBeanAttributes not fired for interceptors and decora…
Browse files Browse the repository at this point in the history
…tors
  • Loading branch information
jharting committed Jan 11, 2012
1 parent ff0a58b commit 5098c85
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -238,13 +238,21 @@ protected <T> void createNewManagedBean(WeldClass<T> annotatedClass) {
protected <T> void createDecorator(WeldClass<T> weldClass) {
BeanAttributes<T> attributes = BeanAttributesFactory.forManagedBean(weldClass, getManager());
DecoratorImpl<T> bean = DecoratorImpl.of(attributes, weldClass, manager, services);
getEnvironment().addDecorator(bean);
// fire ProcessBeanAttributes for decorator
boolean vetoed = fireProcessBeanAttributes(bean);
if (!vetoed) {
getEnvironment().addDecorator(bean);
}
}

protected <T> void createInterceptor(WeldClass<T> weldClass) {
BeanAttributes<T> attributes = BeanAttributesFactory.forManagedBean(weldClass, getManager());
InterceptorImpl<T> bean = InterceptorImpl.of(attributes, weldClass, manager, services);
getEnvironment().addInterceptor(bean);
// fire ProcessBeanAttributes for decorator
boolean vetoed = fireProcessBeanAttributes(bean);
if (!vetoed) {
getEnvironment().addInterceptor(bean);
}
}

protected <T> SessionBean<T> createSessionBean(InternalEjbDescriptor<T> descriptor) {
Expand Down
7 changes: 0 additions & 7 deletions jboss-tck-runner/1.1/src/test/resources/tck-tests.xml
Original file line number Diff line number Diff line change
Expand Up @@ -163,13 +163,6 @@
<exclude name="testProcessInjectionTargetEventFiredForMessageDrivenBean" />
</methods>
</class>
<!-- WELD-1045 -->
<class name="org.jboss.jsr299.tck.tests.extensions.lifecycle.processBeanAttributes.VerifyValuesTest">
<methods>
<exclude name="testInterceptorBeanAttributes" />
<exclude name="testDecoratorBeanAttributes" />
</methods>
</class>
<!-- WELD-1046 -->
<class name="org.jboss.jsr299.tck.tests.extensions.lifecycle.processBeanAttributes.VerifyValuesTest">
<methods>
Expand Down

0 comments on commit 5098c85

Please sign in to comment.