You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2022/09/04 11:38:22 UTC

[GitHub] [bookkeeper] yaalsn opened a new pull request, #3467: Add powermock global configuration

yaalsn opened a new pull request, #3467:
URL: https://github.com/apache/bookkeeper/pull/3467

   ### Motivation
   
   Fix https://github.com/apache/bookkeeper/issues/3115 
   
   When I try to run some test classes, there will occur an error message like the above issue cause `javax/management/MBeanServer` is a system class and javassist's classloader cannot load it. we can add powermock global configuration to ingore this error message.
   
   This powermock's instruction is here: https://github.com/powermock/powermock/wiki/PowerMock-Configuration
   
   ### Changes
   
   Add `org/powermock/extensions/configuration.properties`.


-- 
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: commits-unsubscribe@bookkeeper.apache.org

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

[GitHub] [bookkeeper] hangc0276 commented on pull request #3467: Add powermock global configuration

Posted by GitBox <gi...@apache.org>.

hangc0276 commented on PR #3467:
URL: https://github.com/apache/bookkeeper/pull/3467#issuecomment-1236475190

   Good job!


-- 
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: commits-unsubscribe@bookkeeper.apache.org

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

[GitHub] [bookkeeper] yaalsn commented on pull request #3467: Add powermock global configuration

Posted by GitBox <gi...@apache.org>.

yaalsn commented on PR #3467:
URL: https://github.com/apache/bookkeeper/pull/3467#issuecomment-1236318371

   @hangc0276 @eolivelli PTAL


-- 
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: commits-unsubscribe@bookkeeper.apache.org

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

[GitHub] [bookkeeper] yaalsn commented on pull request #3467: Add powermock global configuration

Posted by GitBox <gi...@apache.org>.

yaalsn commented on PR #3467:
URL: https://github.com/apache/bookkeeper/pull/3467#issuecomment-1236335149

   @nicoloboschi I forgot to add ASF license, please help to approve the workflow running again, thx! 


-- 
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: commits-unsubscribe@bookkeeper.apache.org

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

[GitHub] [bookkeeper] Shoothzj merged pull request #3467: Add powermock global configuration

Posted by GitBox <gi...@apache.org>.

Shoothzj merged PR #3467:
URL: https://github.com/apache/bookkeeper/pull/3467


-- 
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: commits-unsubscribe@bookkeeper.apache.org

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