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

ProcessSessionBeanTest testProcessSessionBeanEvent count

    XMLWordPrintable

Details

    • Bug
    • Resolution: Won't Do
    • Major
    • None
    • 1.0.4.Final
    • None
    • None

    Description

      [Side note maybe there should be versions for 1.0.4.SP1 and 1.0.4.SP2]

      Test: org.jboss.jsr299.tck.tests.extensions.processBean.ProcessSessionBeanTest.testProcessSessionBeanEvent

      Not sure if this is an issue or not, so filling this as a bug and not a challenge. The test essentially has two observer methods and is asserting that only one of them are called.

      public void observeElephantSessionBean(@Observes ProcessSessionBean<Elephant> event)

      { ProcessBeanObserver.elephantProcessSessionBean = event; }

      public void observeElephantBean(@Observes ProcessBean<Elephant> event)

      { ProcessBeanObserver.elephantProcessBeanCount++; }

      Specifically the test asserts that observeElephantSessionBean is called and that observeElephantBean is not called.

      Currently we call both because ProcessSessionBean is assignable to ProcessBean and the generics are the same.

      Is there a part of the spec that mandates only the most specific observer method is called?

      The only thing I can find is in 10.4 which says pretty clearly:

      There may be arbitrarily many observer methods with the same event parameter type and qualifiers.
      A bean (or extension) may declare multiple observer methods.

      Well I guess it's not that clear as it says essentially "this may happen" rather than "this may happen...and when it does the behavior is...[all are invoked || only the most specific is invoked]"

      Attachments

        Activity

          People

            pmuiratbleepbleep Pete Muir (Inactive)
            dblevins_jira David Blevins (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              PagerDuty