ricardozanini opened a new issue, #2817:
URL: https://github.com/apache/incubator-kie-tools/issues/2817
## Summary
Remove `SCRIPT_DEBUG=true` flag from behave tests to save time on CI.
## Description
Major e2e tests on the images side have the debug flag set to
`SCRIPT_DEBUG=true`. This environment variable turns on the `set—x` flag for
every bash script running in that context. On my local tests, this considerably
improves the logging and time to spin up the image container (sometimes 20x).
All to verify one single check, for example:
```behave
Scenario: Verify that the application is built and started correctly when
QUARKUS_EXTENSIONS env is used
When container is started with command bash -c
'/home/kogito/launch/build-app.sh && java -jar
target/quarkus-app/quarkus-run.jar'
| variable | value |
| SCRIPT_DEBUG | true |
| QUARKUS_EXTENSIONS | io.quarkus:quarkus-elytron-security-jdbc |
Then check that page is served
| property | value |
| port | 8080 |
| path | /q/health/ready |
| wait | 480 |
| request_method | GET |
| expected_status_code | 200 |
And container log should contain -Duser.home=/home/kogito
```
In this case, the `And container log should contain
-Duser.home=/home/kogito`. Personally, I found this unnecessary. Bats is
already doing this check in the scripts testing folder.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
