TheRealHaui commented on PR #9823: URL: https://github.com/apache/rocketmq/pull/9823#issuecomment-3518663223
@lizhimins, if you want I can of course merge all the individual tests I added for the individual methods - CommandUtil.fetchBrokerNameByAddr, ...fetchMasterAndSlaveAddrByBrokerName, ...fetchMasterAddrByBrokerName and ...fetchMasterAndSlaveAddrByClusterName - into one big test method. Most organizsations however prefer individual Unit Tests for their individual methods/units of code. Further, this pattern is used in the code base for all pre-existing Unit Tests of the CommandUtil class too. Aside of other already existing Unit Tests - e.g. in the DefaultMonitorListenerTest class. The Unit Tests were not generated by any whatsoever AI! -- 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]
