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.

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

Allow access to dev services props #25799

Merged
merged 1 commit into from
May 26, 2022
Merged

Conversation

stuartwdouglas
Copy link
Member

This allows integration tests to access dev services config in the same
way you would with a TestResourceManager.

Fixes #25730

This allows integration tests to access dev services config in the same
way you would with a TestResourceManager.

Fixes quarkusio#25730
@quarkus-bot
Copy link

quarkus-bot bot commented May 26, 2022

Failing Jobs - Building 2b6222c

Status Name Step Failures Logs Raw logs
MicroProfile TCKs Tests Verify Failures Logs Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ MicroProfile TCKs Tests #

- Failing: tcks/microprofile-fault-tolerance 

📦 tcks/microprofile-fault-tolerance

org.eclipse.microprofile.fault.tolerance.tck.TimeoutUninterruptableTest.testTimeoutAsyncBulkhead line 190 - More details - Source on GitHub

java.lang.AssertionError: Unexpected exception thrown from Future
	at org.testng.Assert.fail(Assert.java:85)
	at org.eclipse.microprofile.fault.tolerance.tck.util.Exceptions.expect(Exceptions.java:98)

@geoand geoand merged commit 2e01333 into quarkusio:main May 26, 2022
@quarkus-bot quarkus-bot bot added this to the 2.10 - main milestone May 26, 2022
@mederel
Copy link
Contributor

mederel commented Jun 14, 2022

I am using quarkus 2.7, and I am facing the exact same issue with the NativeImageTest annotation (so probably the 2 other as well). It is hindering using the DevServices while maintaining a good testing level of the native build of an application.

Would it be possible to request a retrofit of the fix in at least version 2.7?

(if I look up the source code and changes it seems totally feasible)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DevServices config not making it to quarkusIntTest config
4 participants