davsclaus opened a new pull request, #29626: URL: https://github.com/apache/camel/pull/29626
JIRA: [CAMEL-26855](https://issues.apache.org/jira/browse/CAMEL-26855) Two small camel-jbang bugs, found by the review of #29621: - **Embedded plugins in a folder with a space**: `PluginHelper.scanJarForPlugins` turned the `jar:file:...!/` URL of the CLI jar into a file name with `URL.getPath()`, which keeps the URL encoding (a space is `%20`). With the CLI installed in a folder with a space (such as `C:\Program Files\...`), `new JarFile(...)` failed, the exception was ignored, and the embedded plugins were not found. Now the jar file is taken from the file URL of the jar URL (`JarURLConnection.getJarFileURL()`) as a `Path` via its URI. New test with a jar in a folder `my camel`. - **A test that never ran**: `ExportMainApplicationProperties` did not end with `Test`, so surefire skipped it, and it was broken: the export adds the `application.properties` it finds to the files, which the test had set as a fixed-size `Arrays.asList` (`UnsupportedOperationException`). Renamed to `ExportMainApplicationPropertiesTest`, with a list the export can add to; its 3 tests pass. Tests: camel-jbang-core 2777 pass; in the serial execution only `ExportTest` cases that start a server on port 8080 failed locally, as another process held the port. _Claude Code on behalf of davsclaus_ 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
