dimitribekale commented on PR #2705:
URL: https://github.com/apache/activemq/pull/2705#issuecomment-6288476064
Thanks for the review. Pushed a follow-up commit addressing both points:
- added the trailing newline;
- switched both tests to assertThrows. I kept one assertion on the message,
but only that it names the operation (contains("reject") / contains("release"))
rather than the full sentence, so a rewording won't break the tests while a
copy-paste between the two methods still will.
I left the modify()/accept() messages out to keep this PR to the reported
defect; happy to add tests for them in a follow-up if you'd like.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information, visit: https://activemq.apache.org/contact
