Fix bug: Remove incorrect quotation Marks in Java Command Paths

[jonatanwestling]

When running Simod in a Docker container with complete_configuration.yml, the Java command for BPMN layout generation fails due to quotation marks (") around file paths. The error occurs when executing the bpmn-layout-1.0.6-jar-with-dependencies.jar command, resulting in:

Adding BPMN diagram to the model: ['java', '-jar', '"/usr/src/Simod/src/simod/control_flow/lib/bpmn-layout-1.0.6-jar-with-dependencies.jar"', '"/usr/src/Simod/outputs/20250428_211724_A07E83B1_0E85_45DD_B8C1_422790F8FEB1/best_result/LoanApp_simplified_train.bpmn"']
Error: Unable to access jarfile "/usr/src/Simod/src/simod/control_flow/lib/bpmn-layout-1.0.6-jar-with-dependencies.jar"

This happens because the quotation marks are incorrectly included in the command, causing the Java runtime to misinterpret the JAR file path. When removing the quotation mark around the paths, the command runs successfully if the modified source code is mounted.

To reproduce this bug just follow the Docker guide in the readme file in the repo and run with complete_configuration.yml (havent tried with the others). And the error message should be displayed at the end.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[davidsequera]

I am having the same error

[david-chapela]

Hello @jonis1337 and @davidsequera ,

Thanks for pointing out this bug! And sorry for the delay in the fix.

We included the double quotes to support Windows paths with spaces, and forgot to cover the UNIX alternative in the case of the BPMN layout (we did cover it when running SplitMiner jar).

I added a fix in the latest PR (#171) so that it covers both cases depending on the platform.

Please, reopen if the error persists.

[davidsequera]

Thank you!