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. Tests
don't fail when run in isolation. 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:
us...@infra.apache.org
