Uploaded image for project: 'WildFly'
  1. WildFly
  2. WFLY-15014

Upgrade to MP Reactive Messaging TCK 2.0.1 and enable excluded tests

    XMLWordPrintable

Details

    Description

      There are a few problems in the Reactive Messaging TCK. I have excluded the affected tests in testsuite/integration/microprofile-tck/reactive-messaging/pom.xml:

                  <plugin>
                      <groupId>org.apache.maven.plugins</groupId>
                      <artifactId>maven-surefire-plugin</artifactId>
                      <configuration>
                          <dependenciesToScan>
                              <dependenciesToScan>org.eclipse.microprofile.reactive.messaging:microprofile-reactive-messaging-tck</dependenciesToScan>
                          </dependenciesToScan>
                          <excludes>
                              <!--
                                  TODO https://issues.redhat.com/browse/WFLY-15014
                                  For the following excluded tests, I have opened the indicated PRs for the TCK, which will
                                  be part of 2.0.1. I've confirmed locally with a snapshot of the TCK that it works.
                              -->
                              <!--
                                  These use beans which are not valid according to the CDI spec.
                                  https://github.com/eclipse/microprofile-reactive-messaging/pull/136/
                              -->
                              <exclude>**/ChannelInjectionPublisherBuilderMessageTest.java</exclude>
                              <exclude>**/ChannelInjectionPublisherBuilderPayloadTest.java</exclude>
                              <exclude>**/ChannelInjectionPublisherTest.java</exclude>
                              <exclude>**/ChannelInjectionPublisherPayloadTest.java</exclude>
                              <!--
                                  Test the deployment is missing a bean.
                                  https://github.com/eclipse/microprofile-reactive-messaging/pull/135
                              -->
                              <exclude>**/EmitterInjectionMessageBeanWithPayloadsWithAckTest.java</exclude>
                              <!--
                                  The deployment is missing the ArchiveExtender
                                  https://github.com/eclipse/microprofile-reactive-messaging/pull/135
                              -->
                              <exclude>**/.MetricsTest.java</exclude>
                          </excludes>
      

      The mentioned pull requests have been merged and will be part of a future 2.0.1 release.

      I have verified locally that these tests will pass once the TCK is fixed.

      Attachments

        Issue Links

          Activity

            People

              kkhan1@redhat.com Kabir Khan
              kkhan1@redhat.com Kabir Khan
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: