yyqdbngt opened a new pull request, #3789: URL: https://github.com/apache/rocketmq-dashboard/pull/3789
### Motivation The existing `CliProcessEnvironmentTest` covers whitelisting, overrides, invalid names and the builder application, but the null-safety of `build`, null-valued parent entries and duplicate configured names were untested. ### Changes - `nullParentEnvironmentYieldsOnlyProviderValues`: a null parent map contributes nothing, so the child environment carries exactly the provider values (and stays empty when the provider map is null too). - `parentNullValuesAreDroppedWhileProviderValueWins`: a parent entry whose value is null is skipped, letting the provider-supplied value for the same name win. - `duplicateConfiguredNamesAreDeduplicated`: repeated and whitespace-padded occurrences of an additional allowed name collapse into a single entry. ### Verification ``` mvn -B test -Dtest=CliProcessEnvironmentTest [INFO] Tests run: 9, 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]
