yyqdbngt opened a new pull request, #3788: URL: https://github.com/apache/rocketmq-dashboard/pull/3788
### Motivation The existing `ProxyConsumerResolverTest` covers discovery derivation, caching, transient retry and the connection-query null degradation, but three guard paths were unverified: the shared cache key for blank instance ids, the filtering of blank client addresses during discovery, and the running-info fallback when no proxy is discovered. ### Changes - `discoverProxyAddressesShouldShareCacheForBlankInstanceKeys`: null and whitespace-only instance ids map to the same default cache key, so the second lookup is served from cache. - `discoverProxyAddressesShouldSkipBlankClientAddresses`: entries with null or whitespace-only client addresses are filtered out, leaving only valid proxy remoting addresses. - `resolveConsumerRunningInfoShouldReturnNullWhenNoProxyDiscovered`: a discovery failure degrades the running-info query to null so the caller can fall back to the broker path. ### Verification ``` mvn -B test -Dtest=ProxyConsumerResolverTest [INFO] Tests run: 8, Failures: 0, Errors: 0, Skipped: 0 [INFO] BUILD SUCCESS ``` -- 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]
