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

Replace resteasy usage with resteasy reactive in hibernate-reactive integration tests #13965

Merged
merged 1 commit into from
Dec 19, 2020

Conversation

Postremus
Copy link
Member

Need to wait for jaxb support in resteasy-reactive.
hibernate-reactive-panache depends on it.

This is also why this pr is still a draft

Fixes #13944

@geoand
Copy link
Contributor

geoand commented Dec 18, 2020

I wouldn't hold this PR up waiting for JAX-B support in RESTEasy Reactive - we are only going to add that if people really start asking for it.

@FroMage
Copy link
Member

FroMage commented Dec 18, 2020

We can move the JAXB IT tests to the hibernate-reactive-panache deployment tests, since we're testing serialisation so we can manufacture the entities without persisting them.

@Postremus
Copy link
Member Author

I moved all JAXB IT Tests to the deployment tests.

@Postremus Postremus marked this pull request as ready for review December 18, 2020 23:02
@geoand geoand requested a review from Sanne December 19, 2020 06:14
@Sanne
Copy link
Member

Sanne commented Dec 19, 2020

Awesome, thanks!

All this help might allow me to finish the next step for Hibernate Reactive on time for the next beta, I really appreciate it.

@Sanne Sanne merged commit 9647038 into quarkusio:master Dec 19, 2020
@ghost ghost added this to the 1.11 - master milestone Dec 19, 2020
@Postremus Postremus deleted the 13944-use-rr branch December 19, 2020 11:27
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.

All integration tests using Hibernate Reactive need to move to RESTEasy Reactive
4 participants