yyqdbngt opened a new pull request, #3784: URL: https://github.com/apache/rocketmq-dashboard/pull/3784
### Motivation The existing `BrokerConfigDiffServiceTest` covers uniform configs, drifted pairs, partial failures, deduplication and single-broker clusters, but three behaviours remain unverified: how a null/unconfigured field participates in drift detection, whether null broker rows are skipped when collecting targets, and the per-value rendering across three brokers. ### Changes - `unconfiguredFieldOnOneBrokerIsReportedAsDifference`: a broker whose `flushDiskType` is unset is reported as `configured=false` next to the configured value of its peer, and the drift is detected. - `nullAndAddressLessBrokerEntriesAreSkipped`: null broker rows and address-less rows are filtered out before target collection, leaving one reachable broker and a complete comparison. - `driftAcrossThreeBrokersExposesEveryValue`: every broker value participates in the reported difference row, including the repeated majority value. ### Verification ``` mvn -B test -Dtest=BrokerConfigDiffServiceTest [INFO] Tests run: 10, 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]
