You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@rocketmq.apache.org by "yyqdbngt (via GitHub)" <gi...@apache.org> on 2026/09/05 18:16:08 UTC
[PR] test: add first unit test for AcknowledgeSystemAlertDTO [rocketmq-dashboard]
yyqdbngt opened a new pull request, #3555:
URL: https://github.com/apache/rocketmq-dashboard/pull/3555
### Summary
Adds the first dedicated unit test suite for `AcknowledgeSystemAlertDTO`, the payload of the system alert acknowledge endpoint.
The DTO requires a numeric alert `id`. The tests cover the accepted id and the missing-id rejection.
### Changes
- `server/src/test/java/org/apache/rocketmq/studio/ops/alert/AcknowledgeSystemAlertDTOTest.java`
- `acceptsNumericId`: a populated id passes validation.
- `rejectsMissingId`: a null id yields the `id is required` violation.
### Testing
- `mvn -B test -Dtest=AcknowledgeSystemAlertDTOTest` passes (2 tests, all green).
--
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@rocketmq.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] test: add first unit test for AcknowledgeSystemAlertDTO [rocketmq-dashboard]
Posted by "lizhimins (via GitHub)" <gi...@apache.org>.
lizhimins closed pull request #3555: test: add first unit test for AcknowledgeSystemAlertDTO
URL: https://github.com/apache/rocketmq-dashboard/pull/3555
--
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@rocketmq.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org