[GitHub] [cxf] reta commented on a change in pull request #736: Update to MicroProfile 2.0.0-RC3

[GitBox <gi...@apache.org>]

reta commented on a change in pull request #736:
URL: https://github.com/apache/cxf/pull/736#discussion_r540966945



##########
File path: systests/microprofile/client/weld/testng.xml
##########
@@ -4,11 +4,11 @@
     <test name="All TCK Tests">
         <packages>
             <package name="org.eclipse.microprofile.rest.client.tck" />
-            <package name="org.eclipse.microprofile.rest.client.tck.asynctests" />
             <package name="org.eclipse.microprofile.rest.client.tck.cditests" />
-            <package name="org.eclipse.microprofile.rest.client.tck.sse" />
+            <package name="org.eclipse.microprofile.rest.client.tck.asynctests" />

Review comment:
       @andymc12 something for us to look at, previous PR builds have test case failures:
   
   ```
   org.eclipse.microprofile.rest.client.tck.cditests.HasConversationScopeTest.testHasConversationScoped
   org.eclipse.microprofile.rest.client.tck.cditests.HasConversationScopeTest.testHasConversationScopedFromConfigKey
   ```
   
   When run locally, I was able to see it once or twice out of 10 runs. Changing the order of tests seems to cure the problem but for sure the order should not matter, something fishy here.   




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org