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

[bdevreugd-vialis]

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:

quarkus/test-framework/common/src/main/java/io/quarkus/test/common/DockerContainerLauncher.java

Lines 151 to 153 in f762885

	private String convertPropertyToEnVar(String property) {
	return property.toUpperCase().replace('-', '_').replace('.', '_');
	}

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

[quarkus-bot]

/cc @geoand

[geoand]

Interesting. I'll have a look soon

…

On Wed, Jun 9, 2021, 19:13 quarkus-bot[bot] ***@***.***> wrote: /cc @geoand <https://github.com/geoand> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#17810 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABBMDP3KRTMSSBQLFHRTIC3TR6HKZANCNFSM46MLQN7A> .