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 13:51:17 UTC
[PR] test(alert): cover unknown asset 404 for view and export [rocketmq-dashboard]
yyqdbngt opened a new pull request, #3368:
URL: https://github.com/apache/rocketmq-dashboard/pull/3368
## Summary
Extends the existing `AlertRuleAssetControllerTest` (3 to 4 tests) with the unknown-asset branch.
Added case:
- requesting an unknown asset through either the view or the export endpoint surfaces the service's `BusinessException` (404) as an HTTP 404 response.
## Why
The bundled-asset endpoints document 404 for unknown names; the error path was not covered before (only list/get/export happy paths).
## Testing
`mvn -B test -Dtest=AlertRuleAssetControllerTest` — 4/4 pass; checkstyle (validate) clean.
--
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(alert): cover unknown asset 404 for view and export [rocketmq-dashboard]
Posted by "lizhimins (via GitHub)" <gi...@apache.org>.
lizhimins closed pull request #3368: test(alert): cover unknown asset 404 for view and export
URL: https://github.com/apache/rocketmq-dashboard/pull/3368
--
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