search

snowdrop-zen / quarkus

Quarkus: Supersonic Subatomic Java.
https://quarkus.io
Apache License 2.0
1 stars 0 forks source link

QuarkusIntegrationTest docker env does not work with property names containing '/' #358

Closed snowdrop-bot closed 3 years ago

snowdrop-bot commented 3 years ago

We use the GraphQLClient in Quarkus 2.0.0, it requires a property called test-service/mp-graphql/url. When the docker image is started, it is started with param: --env TEST_SERVICE/MP_GRAPHQL/URL=http://localhost:50683 This is an invalid environment variable, so it fails.

This is caused by: https://github.com/quarkusio/quarkus/blob/f762885947f6af116eda4bbb8793be6688533f8d/test-framework/common/src/main/java/io/quarkus/test/common/DockerContainerLauncher.java#L151-L153

Please make sure it also converts '/' to '_'

https://github.com/quarkusio/quarkus/issues/17810

$upstream:17810$