[PR] test: cover default runtime names and null handling in CliProcessEnvironment [rocketmq-dashboard]

["yyqdbngt (via GitHub)" <gi...@apache.org>]

yyqdbngt opened a new pull request, #3412:
URL: https://github.com/apache/rocketmq-dashboard/pull/3412

   ### Summary
   
   `CliProcessEnvironment` forwards a fixed allow-list of runtime variables plus configured extras. Existing tests covered the allow-list filtering and override precedence, but not the full default-name set or null edge cases.
   
   ### Changes
   
   - Every default runtime name (`LC_ALL`, `TERM`, proxy vars, Windows vars, etc.) is forwarded when present in the parent
   - An allowed parent variable whose value is `null` is skipped
   - Provider variables still apply when the parent environment is `null`
   
   ### Testing
   
   `mvn test -Dtest=CliProcessEnvironmentTest` passes: 9 tests, 0 failures (up from 6).


-- 
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: commits-unsubscribe@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] test: cover default runtime names and null handling in CliProcessEnvironment [rocketmq-dashboard]

["lizhimins (via GitHub)" <gi...@apache.org>]

lizhimins closed pull request #3412: test: cover default runtime names and null handling in CliProcessEnvironment
URL: https://github.com/apache/rocketmq-dashboard/pull/3412


-- 
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: commits-unsubscribe@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org