yyqdbngt opened a new pull request, #3767: URL: https://github.com/apache/rocketmq-dashboard/pull/3767
### Motivation The existing `ClusterRepositoryImplTest` proves ordering, emptiness and deep-copy semantics, but leaves the config-update mutation path and the read behaviour for unknown ids unverified. ### Changes - `updateConfigShouldRefreshOnlyTargetedCluster`: a full replacement config lands on every `ClusterConfigVO` field of the targeted cluster while its identity/endpoint/brokers/proxies/counters survive, and sibling clusters keep their own config untouched. - `updateConfigShouldBeNoOpForUnknownClusterId`: updating a missing id leaves the store size and every cached cluster config unchanged. - `findByIdShouldReturnEmptyForUnknownClusterId`: unknown ids resolve to an empty `Optional`. - `findAllShouldReturnFreshImmutableSnapshots`: repeated calls yield distinct snapshots with distinct element copies, and mutating the returned list is rejected. ### Verification ``` mvn -B test -Dtest=ClusterRepositoryImplTest [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]
