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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
