mcarlett opened a new pull request, #24033:
URL: https://github.com/apache/camel/pull/24033

   ## Summary
   
   The `version()` method in `CliLocalContainerService` was changed to look for 
`"Camel CLI version:"` (commit 47b1e09) but productized launcher JARs still 
output `"Camel JBang version:"` (pre-rebrand). When the new string isn't found, 
`StringHelper.between()` returns null and `.trim()` throws an NPE, crashing all 
34 launcher-it test classes during `initialize()`.
   
   **Fix:** Accept both `"Camel CLI version:"` and `"Camel JBang version:"` 
strings, and guard against null before calling `trim()`.
   
   ## Impact
   
   - Fixes 100% failure rate (34/34 tests) of launcher-it on all platforms when 
run against pre-rebrand launcher JARs
   - No impact on current `main` launcher builds (which already output `"Camel 
CLI version:"`)
   - Backwards compatible with older launcher versions


-- 
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]

Reply via email to