Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

QUARKUS_TEST_PROFILE env variable is not honored #29944

Closed
siudeks opened this issue Dec 18, 2022 · 2 comments · Fixed by #28091
Closed

QUARKUS_TEST_PROFILE env variable is not honored #29944

siudeks opened this issue Dec 18, 2022 · 2 comments · Fixed by #28091
Assignees
@geoand
Labels
area/config kind/bug Something isn't working
Milestone
2.16.0.CR1

Comments

@siudeks
Copy link

siudeks commented Dec 18, 2022

Describe the bug

Issue:
Can't change test profile using environmente variable QUARKUS_TEST_PROFILE, as described at https://quarkus.io/guides/continuous-testing#quarkus-test-dev-testing-test-config_quarkus.test.profile

Reproduction:

Actual:

Green test: *mvn clean test -Dquarkus.test.profile=my_profile*
Red test: *QUARKUS_TEST_PROFILE=my_profile mvn clean test*

Expected:

Green test: *mvn clean test -Dquarkus.test.profile=my_profile*
Green test: *QUARKUS_TEST_PROFILE=my_profile mvn clean test*

Expected behavior

See 'Describe the bug'

Actual behavior

See 'Describe the bug'

How to Reproduce?

https://github.com/siudeks/repro/tree/quarkus-test-profile

Output of uname -a or ver

Linux PC1 5.15.79.1-microsoft-standard-WSL2 #1 SMP Wed Nov 23 01:01:46 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Output of java -version

java 17

GraalVM version (if different from Java)

No response

Quarkus version or git rev

maven

Build tool (ie. output of mvnw --version or gradlew --version)

Apache Maven 3.8.6

Additional information

No response
@siudeks siudeks added the kind/bug Something isn't working label Dec 18, 2022
@geoand geoand self-assigned this Dec 19, 2022
geoand added a commit to geoand/quarkus that referenced this issue Dec 19, 2022
@geoand
Support QUARKUS_TEST_PROFILE environment variable
2f24c69 
The config [entry](https://quarkus.io/guides/continuous-testing#quarkus-test-dev-testing-test-config_quarkus.test.profile) mentions the environment variable, so let's support it

Fixes: quarkusio#29944
@geoand geoand mentioned this issue Dec 19, 2022
Closed
@geoand
Copy link
Contributor

geoand commented Dec 19, 2022

Thanks for spotting this.

#29951 should take care of it

@geoand
Copy link
Contributor

geoand commented Dec 20, 2022

It turns out that main has already fixed the issue with #28091

@geoand geoand closed this as completed Dec 20, 2022
@geoand geoand added this to the 2.16 - main milestone Dec 20, 2022
@radcortez radcortez mentioned this issue Dec 20, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@geoand geoand

Labels
area/config kind/bug Something isn't working
Projects
None yet
Milestone
2.16.0.CR1
2 participants
@siudeks @geoand