Uploaded image for project: 'CDI TCK'
  1. CDI TCK
  2. CDITCK-199

ProcessObserverMethod Type parameters and CDITCK-174

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Major Major
    • 1.0.4.CR1
    • None
    • Tests
    • None

      Hi;

      Recently TCK 1.0.2 SP1 contains some tests that must be excluded. Those tests are located in org.jboss.jsr299.tck.tests.extensions.observer.ProcessObserverMethodEventTest

      testGetAnnotatedMethod(org.jboss.jsr299.tck.tests.extensions.observer.ProcessObserverMethodEventTest)
      testProcessObserverMethodEventsSent(org.jboss.jsr299.tck.tests.extensions.observer.ProcessObserverMethodEventTest)
      testGetObserverMethod(org.jboss.jsr299.tck.tests.extensions.observer.ProcessObserverMethodEventTest)

      Those tests uses uses wrong type arguments according to the JavaDoc (public interface ProcessObserverMethod<T,X>). For example, http://fisheye.jboss.org/browse/weld/cdi-tck/trunk/impl/src/main/java/org/jboss/jsr299/tck/tests/extensions/observer/broken/definitionError/ProcessObserverMethodObserver.java?hb=true
      contains

      public void observeObserverMethodForEventB(@Observes ProcessObserverMethod<?, EventB> event) {}

      It must be

      public void observeObserverMethodForEventB(@Observes ProcessObserverMethod<EventB, ?> event) {}

      Because JavaDoc is higher importancy than spec, those test must be excluded or changed.

      Also this issue was disccussed on mailing list, http://markmail.org/message/g6c4zsuuzckeai26

      Please exclude those tests from TCK 1.0.2 SP1. Otherwise OWB failes to pass TCK.

      Thanks;

              pmuiratbleepbleep Pete Muir (Inactive)
              gerdogdu@managecat.com Gurkan Erdogdu (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: