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

Posted to gitbox@activemq.apache.org by GitBox <gi...@apache.org> on 2022/02/11 12:33:11 UTC

[GitHub] [activemq-artemis] gemmellr commented on pull request #3948: ARTEMIS-3677 mitigate NPE when browsing messages

gemmellr commented on pull request #3948:
URL: https://github.com/apache/activemq-artemis/pull/3948#issuecomment-1036167223


   More comprehensive unit tests of the actual changes would have been nicer than a systest, the related areas already have unit tests that could be easily adapted. The systest also looks to only use Core and so wouldnt seem to cover the original change at all, given it was in AMQPMessage only.


-- 
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: gitbox-unsubscribe@activemq.apache.org

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