yyqdbngt opened a new pull request, #3907: URL: https://github.com/apache/rocketmq-dashboard/pull/3907
### Motivation The broker config service tests covered update/audit paths and flush-disk parsing, but the full property-to-VO mapping, defaults for malformed properties, failure surfacing, the instance-id routing path, and the FAILED audit branch were untested. This PR grows the suite from 4 to 10 cases. ### Changes - Every broker property maps onto the `ClusterConfigVO` (auto-create flags, sizes, queue counts, retention, permission, delete window, trace topic). - Missing or malformed properties fall back to the documented defaults. - `getBrokerConfig` surfaces broker failures as `BusinessException` 500 and routes through the runtime resolver when an instance id is supplied. - A rejected update records a `FAILED` audit detail containing the broker error before rethrowing. - Updating without a configured NameServer address fails with the actionable 503 message. ### Verification - `mvn -B test -Dtest=RocketMQBrokerConfigServiceTest`: 10/10 passed, 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]
