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

ProcessObserverMethod Type parameters and CDITCK-174

    XMLWordPrintable

Details

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

    Description

      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;

      Attachments

        Activity

          People

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

            Dates

              Created:
              Updated:
              Resolved:

              PagerDuty