yyqdbngt opened a new pull request, #3256: URL: https://github.com/apache/rocketmq-dashboard/pull/3256
## Summary Adds `MqClientPoolTest`, first unit tests for the lifecycle guards of the long-lived RocketMQ client pool. Client creation is deliberately excluded (starting a pull consumer/producer would open remoting against a live NameServer), so the deterministic guard paths are pinned down instead. Coverage: - blank/whitespace NameServer addresses are rejected with `BusinessException` 400 before any client work, for both the pull-consumer and producer entry points; - after `shutdown()` the pool rejects further use with `BusinessException` 503; - the address check takes precedence over the closed flag (a blank address still yields 400 on a closed pool); - `release` is a no-op for blank endpoints, unknown identities, and endpoints that were never pooled. ## Why The pool is the shared owner of producer/consumer clients across requests; the failure modes that do not require network had no direct test. ## Testing `mvn -B test -Dtest=MqClientPoolTest` — 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
