You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cxf.apache.org by GitBox <gi...@apache.org> on 2022/11/02 01:02:02 UTC

[GitHub] [cxf] reta opened a new pull request, #1019: Fixing Microprofile test suites (temporary adding javax.servlet dependency and downgrading to Jetty 9)

reta opened a new pull request, #1019:
URL: https://github.com/apache/cxf/pull/1019

   Fixing Microprofile test suites (temporary adding javax.servlet dependency and downgrading to Jetty 9)


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@cxf.apache.org

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

[GitHub] [cxf] jimma commented on pull request #1019: Fixing Microprofile test suites (temporary adding javax.servlet dependency and downgrading to Jetty 9)

Posted by GitBox <gi...@apache.org>.
jimma commented on PR #1019:
URL: https://github.com/apache/cxf/pull/1019#issuecomment-1299714381

   @reta From the CI result, there are still some test failures under  `cxf-systests-microprofile-async` and `cxf-systests-microprofile-nocdi` caused by ` java.lang.ClassNotFoundException: javax.json.JsonException`
   The `wiremock` or `wiremock-standalone` dependency should both work for test scope ,  but it seems it needs to add more javax dependencies for `wiremock` dependency. 
   
   For `cxf-systests-microprofile-weld`, I did the similar change in weld module to get more tests passed yesterday , but there are still some failures I still need to have a look: https://github.com/apache/cxf/pull/1018/commits/94ce0543f4167d75dc2c9352b124e8358aec72be. 
   And the latest CI shows there are 23 failures(in my branch) to fix : https://ci-builds.apache.org/job/CXF/job/CXF-JDK17-PR/110/. Do you have an idea what I need to have a look ?
   
   


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@cxf.apache.org

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

[GitHub] [cxf] reta commented on pull request #1019: Fixing Microprofile test suites (temporary adding javax.servlet dependency and downgrading to Jetty 9)

Posted by GitBox <gi...@apache.org>.
reta commented on PR #1019:
URL: https://github.com/apache/cxf/pull/1019#issuecomment-1300324347

   Thanks @jimma 
   
   > The `wiremock` or `wiremock-standalone` dependency should both work for test scope , but it seems it needs to add more javax dependencies for `wiremock` dependency.
   
   I tried to fix that but went into rabbit hole of dependency mess, probably we need to wait for wiremock to catch up ...
   
   > Do you have an idea what I need to have a look ?
   
   I haven't yet into the failures yet, but I will help you out on https://github.com/apache/cxf/pull/1018
   


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@cxf.apache.org

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

[GitHub] [cxf] reta commented on pull request #1019: Fixing Microprofile test suites (temporary adding javax.servlet dependency and downgrading to Jetty 9)

Posted by GitBox <gi...@apache.org>.
reta commented on PR #1019:
URL: https://github.com/apache/cxf/pull/1019#issuecomment-1300325039

   Closing to continue work in https://github.com/apache/cxf/pull/1018


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@cxf.apache.org

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

[GitHub] [cxf] reta closed pull request #1019: Fixing Microprofile test suites (temporary adding javax.servlet dependency and downgrading to Jetty 9)

Posted by GitBox <gi...@apache.org>.
reta closed pull request #1019: Fixing Microprofile test suites (temporary adding javax.servlet dependency and downgrading to Jetty 9)
URL: https://github.com/apache/cxf/pull/1019


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@cxf.apache.org

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

[GitHub] [cxf] jimma commented on pull request #1019: Fixing Microprofile test suites (temporary adding javax.servlet dependency and downgrading to Jetty 9)

Posted by GitBox <gi...@apache.org>.
jimma commented on PR #1019:
URL: https://github.com/apache/cxf/pull/1019#issuecomment-1300346407

   > Thanks @jimma
   > 
   > > The `wiremock` or `wiremock-standalone` dependency should both work for test scope , but it seems it needs to add more javax dependencies for `wiremock` dependency.
   > 
   > I tried to fix that but went into rabbit hole of dependency mess, probably we need to wait for wiremock to catch up ...
   > 
   
   +1, now adding `wiremock-standalone` seems a bit easier and it doesn't need to add much other dependencies.  This will make it easier too when we replace the `wiremock-standalone` with the new `wiremock` release.
   
   > > Do you have an idea what I need to have a look ?
   > 
   > I haven't yet into the failures yet, but I will help you out on #1018
   
   Thanks @reta.  Let's discuss there. 
   
   


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@cxf.apache.org

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