Hi I have opened a JIRA issue, https://issues.jboss.org/browse/CDITCK-199
>>>Is it to fail the most recent tck and ignore the problem? No. Passing TCK is always higher priority... Thanks; --Gurkan ----- Original Message ---- From: David Jencks <[email protected]> To: [email protected] Sent: Wed, December 15, 2010 12:00:31 AM Subject: Re: Yan:: svn commit: r1049215 - in /openwebbeans/trunk/webbeans-impl/src: main/java/org/apache/webbeans/event/NotificationManager.java test/java/org/apache/webbeans/newtests/portable/events/extensions/AddObserverMethodExtension.java IMO the changes committed to the tck for CDITCK-174 require the changes I committed to OWB in order to pass. The changes in the tck for CDITCk-174 are still present in the 1.0.2.SP1 tck. If you think these contradict the javadoc can you file a new CDITCK jira or reopen CDITCK-174 or take some other appropriate action with Pete or other CDI EG representative? I don't understand what course of action you want to follow. Is it to fail the most recent tck and ignore the problem? thanks david jencks On Dec 14, 2010, at 1:14 PM, Mark Struberg wrote: > David, as clarified by the JSP EC, API Javadocs are _always_ correct if there >is any question or discrepance with the written spec. > > I have not looked at the Javadoc, but please also check the corresponding >CDI-TCK-174 > > https://issues.jboss.org/browse/CDITCK-174 > > > LieGrue, > strub > > --- On Tue, 12/14/10, David Jencks <[email protected]> wrote: > >> From: David Jencks <[email protected]> >> Subject: Re: Yan:: svn commit: r1049215 - in >>/openwebbeans/trunk/webbeans-impl/src: >>main/java/org/apache/webbeans/event/NotificationManager.java >>test/java/org/apache/webbeans/newtests/portable/events/extensions/AddObserverMethodExtension.java >> >> To: [email protected] >> Date: Tuesday, December 14, 2010, 9:02 PM >> Since there's a new released tck that >> doesn't change or exclude the tests in question, I tend to >> think that current opinion in the cdi EG is that the javadoc >> is wrong. Can you take this up with pete to find out >> what's going on? Meanwhile I would think there would >> be some interest in passing the tck. >> >> thanks >> david jencks >> >> On Dec 14, 2010, at 12:50 PM, Gurkan Erdogdu wrote: >> >>> As pete mail, javadoc is correct therefore i reverted >> change before. If javadoc is correct those tck tests must be >> excluded. our code is correct because it is aligned with >> published javadoc >>> >>> Sal, 14 Ara 2010 20:51 EET tarihinde [email protected] >> şöyle yazdı: >>> >>>> Author: djencks >>>> Date: Tue Dec 14 18:51:54 2010 >>>> New Revision: 1049215 >>>> >>>> URL: http://svn.apache.org/viewvc?rev=1049215&view=rev >>>> Log: >>>> OWB-490, CDITCK-174 reapply rev 1034955 to comply >> with cdi tck 1.0.2.SP1 >>>> >>>> Modified: >>>> >>>>openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java >>>> >>>> >>>>openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/newtests/portable/events/extensions/AddObserverMethodExtension.java >>>> >>>> >>>> Modified: >>openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java >>a >>>> URL: >>>>http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java?rev=1049215&r1=1049214&r2=1049215&view=diff >>>> >>>> >> ============================================================================== >>>> --- >>openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java >>a >> (original) >>>> +++ >>openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java >>a >> Tue Dec 14 18:51:54 2010 >>>> @@ -354,13 +354,7 @@ public final class >> NotificationManager >>>> { >>>> Type >> beanClassArg = actualArgs[0]; >>>> Type >> returnClassArg = actualArgs[1]; >>>> - >>>> - >> if(isObserverMethod) >>>> - { >>>> - >> beanClassArg = actualArgs[1]; >>>> - >> returnClassArg = actualArgs[0]; >>>> - } >>>> - >>>> + >> >>>> >> if(checkEventTypeParameterForExtensions(beanClass, >> beanClassArg) && >>>> >> >> checkEventTypeParameterForExtensions(producerOrObserverReturnClass, >> returnClassArg)) >>>> { >>>> >>>> Modified: >>openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/newtests/portable/events/extensions/AddObserverMethodExtension.java >>a >>>> URL: >>>>http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/newtests/portable/events/extensions/AddObserverMethodExtension.java?rev=1049215&r1=1049214&r2=1049215&view=diff >>>> >>>> >> ============================================================================== >>>> --- >>openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/newtests/portable/events/extensions/AddObserverMethodExtension.java >>a >> (original) >>>> +++ >>openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/newtests/portable/events/extensions/AddObserverMethodExtension.java >>a >> Tue Dec 14 18:51:54 2010 >>>> @@ -40,9 +40,9 @@ public class >> AddObserverMethodExtension >>>> >>>> public static class MyBeanExtension >> implements Extension >>>> { >>>> - public static >> ObserverMethod<MyEvent> observerMethod; >>>> + public static >> ObserverMethod<AddObserverMethodExtension> >> observerMethod; >>>> >>>> - public void >> observer(@Observes ProcessObserverMethod<MyEvent, >> AddObserverMethodExtension> event) >>>> + public void >> observer(@Observes >> ProcessObserverMethod<AddObserverMethodExtension, >> MyEvent> event) >>>> { >>>> >> observerMethod = event.getObserverMethod(); >>>> >>>> >>>> >>> >>> >>> >> >> > > >
