-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Integration tests fail due to Mixing RESTEasy Reactive and RESTEasy Classic client parts #36124
Comments
/cc @FroMage (resteasy-reactive), @geoand (resteasy-reactive), @stuartwdouglas (resteasy-reactive) |
Thanks for reporting. Can you check and see if this issue is still present in the current |
Yes, the issue is still present with the version 999-SNAPSHOT:
One possible workaround for Quarkus services that do not need to call Keycloak Admin APIs during tests is to exlude the KC admin client: <dependency>
<groupId>com.github.dasniko</groupId>
<artifactId>testcontainers-keycloak</artifactId>
<version>3.0.0</version>
<scope>test</scope>
<exclusions>
<exclusion>
<groupId>org.keycloak</groupId>
<artifactId>keycloak-admin-client</artifactId>
</exclusion>
</exclusions>
</dependency> |
#36134 is the fix |
Allow using RESTEasy Classic test deps when app is RESTEasy Reactive app
When will this be released. Is 3.5 targetted to be released soon. |
This issue pop up for MongoResource Test as well. |
Hi guys any workaround for this? Exclude keycloak-admin-client from testcontainers-keycloak is impossible for me, because I have many tests wich uses this client. |
Still seeing this issue with the newly released 3.5.0, is there an estimate for which release it will target? |
Can you attach a sample that fails? |
Sorry for the false alarm, it turned out we had a lingering import on an unused |
No problem at all. Thanks for the update. |
Hello, I'm encountering this issue with quarkus 3.18.2 and testcontainers-keycloak 3.6.0. Excluding EDIT: currently I'm unable to reproduce the problem with a new project. My project uses gradle, while my sample project uses maven with quarkus 3.18.3. I changed my original project to maven + quarkus 3.18.3 and the problem disappears. |
Describe the bug
After #35468 if a project delares usage of
quarkus-rest-client-reactive-jackson
and usestestcontainers-keycloak
for integration tests then Quarkus fails to start with[error]: Build step io.quarkus.resteasy.reactive.common.deployment.ResteasyReactiveCommonProcessor#checkMixingStacks threw an exception: jakarta.enterprise.inject.spi.DeploymentException: Mixing RESTEasy Reactive and RESTEasy Classic client parts is not supported
testcontainers-keycloak happen to depend on the non-reactive
resteasy-client
.Expected behavior
Mixing stacks is not enforced for dependencies with the
test
scope.Actual behavior
Quarkus fails to start if test dependencies lead to mixing stacks.
How to Reproduce?
cd quarkus-rest-reactive-issue/code-with-quarkus/
mvn clean verify
Output of
uname -a
orver
Linux pop-os 6.4.6-76060406-generic #202307241739
169462191722.04~ac5e1a8 SMP PREEMPT_DYNAMIC Wed S x86_64 x86_64 x86_64 GNU/LinuxOutput of
java -version
OpenJDK 64-Bit Server VM Corretto-17.0.4.8.1 (build 17.0.4+8-LTS, mixed mode, sharing)
GraalVM version (if different from Java)
No response
Quarkus version or git rev
3.4.1
Build tool (ie. output of
mvnw --version
orgradlew --version
)Apache Maven 3.9.4 (dfbb324ad4a7c8fb0bf182e6d91b0ae20e3d2dd9)
Additional information
This issue may belong to the maintainers of
testcontainers-keycloak
.The text was updated successfully, but these errors were encountered: