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 2.0.0.Alpha2] quarkus.test.profile not considered in TestResource #2566

Closed
zbendhiba opened this issue May 10, 2021 · 4 comments
Closed

[Quarkus 2.0.0.Alpha2] quarkus.test.profile not considered in TestResource #2566

zbendhiba opened this issue May 10, 2021 · 4 comments
Milestone
2.0.0-M1

Comments

@zbendhiba
Copy link
Contributor

Test failure
Error: Tests run: 8, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 13.839 s <<< FAILURE! - in org.apache.camel.quarkus.main.CoreMainTest Error: testProperties Time elapsed: 0.058 s <<< FAILURE! java.lang.AssertionError: 1 expectation failed. Response body doesn't match expectation. Expected: is "test" Actual: default
@zbendhiba
Copy link
Contributor Author

Related to this issue quarkus-17103

@zbendhiba
Copy link
Contributor Author

I'm going to temporary deactivate this test so that we can have a full build of camel-main branch

zbendhiba added a commit that referenced this issue May 10, 2021
@zbendhiba
temporary deactivate properties test #2566
ccf972a
@zbendhiba
Copy link
Contributor Author

As discussed in zulip chat , we need to pass the property quarkus.test.profile as systemPropertyVariables Maven Surefire plugin configuration, instead of putting it in the TestResource

zbendhiba added a commit that referenced this issue May 10, 2021
@zbendhiba
quarkus.test.profile should be passed in the properties instead of Te…
08659a1 
…stResource fixes #2566
@zbendhiba zbendhiba changed the title Camel-main failure tests with Quarkus 2.0.0.Aplha2 [Quarkus 2.0.0.Alpha2] Camel-main failure tests May 11, 2021
@ppalaga
Copy link
Contributor

ppalaga commented May 11, 2021

Thanks for fixing this, @zbendhiba!

Note to myself: we'll need to add as similar snippet to this test's pom.xml in Quarkus Platform.

@zbendhiba zbendhiba changed the title [Quarkus 2.0.0.Alpha2] Camel-main failure tests [Quarkus 2.0.0.Alpha2] quarkus.test.profile not considered in TestResource May 11, 2021
zbendhiba added a commit that referenced this issue May 11, 2021
@zbendhiba
quarkus.test.profile should be passed in the properties instead of Te…
fc1759e 
…stResource fixes #2566
zbendhiba added a commit that referenced this issue May 12, 2021
@zbendhiba
quarkus.test.profile should be passed in the properties instead of Te…
1672707 
…stResource fixes #2566
jamesnetherton pushed a commit that referenced this issue May 13, 2021
@zbendhiba @jamesnetherton
quarkus.test.profile should be passed in the properties instead of Te…
bed140f 
…stResource fixes #2566
jamesnetherton pushed a commit that referenced this issue May 14, 2021
@zbendhiba @jamesnetherton
quarkus.test.profile should be passed in the properties instead of Te…
7854a4d 
…stResource fixes #2566
jamesnetherton pushed a commit that referenced this issue May 17, 2021
@zbendhiba @jamesnetherton
quarkus.test.profile should be passed in the properties instead of Te…
b6a66ec 
…stResource fixes #2566
ppalaga pushed a commit to ppalaga/camel-quarkus that referenced this issue May 21, 2021
@zbendhiba @ppalaga
quarkus.test.profile should be passed in the properties instead of Te…
4990fca 
…stResource fixes apache#2566
ppalaga pushed a commit to ppalaga/camel-quarkus that referenced this issue May 21, 2021
@zbendhiba @ppalaga
quarkus.test.profile should be passed in the properties instead of Te…
43aa4bd 
…stResource fixes apache#2566
ppalaga pushed a commit to ppalaga/camel-quarkus that referenced this issue May 24, 2021
@zbendhiba @ppalaga
quarkus.test.profile should be passed in the properties instead of Te…
cfb910c 
…stResource fixes apache#2566
ppalaga pushed a commit to ppalaga/camel-quarkus that referenced this issue May 24, 2021
@zbendhiba @ppalaga
quarkus.test.profile should be passed in the properties instead of Te…
a6251fa 
…stResource fixes apache#2566
jamesnetherton pushed a commit that referenced this issue May 25, 2021
@zbendhiba @jamesnetherton
quarkus.test.profile should be passed in the properties instead of Te…
1aa075a 
…stResource fixes #2566
@ppalaga ppalaga added this to the 2.0.0-M1 milestone May 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.0.0-M1
Development

No branches or pull requests
2 participants
@ppalaga @zbendhiba