dimitribekale opened a new pull request, #2705:
URL: https://github.com/apache/activemq/pull/2705
`AmqpMessage.reject()` threw the guard message belonging to `release()`
("Can't release non-received message."), so a test that misuses `reject()`
is pointed at the wrong method. The sibling guards in `accept()` and
`modified()` each name their own operation.
Fixes #2623.
Adds `AmqpMessageTest` covering the message text of both guards, so a
future
copy-paste is caught. Verified locally with JDK 21:
- `mvn test -Dtest=AmqpMessageTest` — 2 tests, green
- with the one-line fix reverted, `testRejectWithoutReceiverMentionsReject`
fails as expected, confirming the test detects the defect
As noted in #2623, I'm not an experienced Java developer; this comes from
an
academic study of LLM-assisted code review and the patch was prepared with
Claude's assistance.
--
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
