Re: [weld-dev] CDITCK-174 - ProcessObserverMethod type parameters
Hi Scott
From the spec,
"The event object must be of type
javax.enterprise.inject.spi.ProcessObserverMethod<T, X>, where T is the bean
class of the bean that declares the observer method and X is the observed event type of
the observer method.
public interface ProcessObserverMethod<T, X> {
public AnnotatedParameter<T> getAnnotatedEventParameter();
public ObserverMethod<X> getObserverMethod();
public void addDefinitionError(Throwable t);
}
"
It looks like this has got completely wrong against the javadoc and signatures released
with the spec, I will go to the EE CTS guys and see what the best path forward is, however
I suspect it is unfixable until we do a MR. I will also go back in and fix this test so
that it works correctly against the API as defined in the javadoc.
Pete
On 18 Nov 2010, at 00:07, Scott Ferguson wrote:
Hi Pete,
I'm looking at the TCK snapshot and at CDITCK-174, but I don't
understand the change. The original looks correct to me and I don't see
how the new change is supposed to work or why it was made.
In ProcessObserverMethodErrorTest, there's a listener:
public void observeObserverMethodForEventB(
@Observes ProcessObserverMethod<?,EventB> event
);
But the ProcessObserverMethod is defined as
public interface ProcessObserverMethod<T,X> {
public ObserverMethod<T> getObserverMethod();
public AnnotatedMethod<X> getAnnotatedMethod();
public void addDefinitionError(Throwable t);
}
In other words, "T" is the type of the event and "X" is the type of
the
listener, but the observer flips those around.
I double checked with the latest weld and with cdi-api.sig, but it
doesn't look like the ProcessObserverMethod itself has changed.
Is there something I'm missing?
-- Scott
_______________________________________________
weld-dev mailing list
weld-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/weld-dev