Skip to content

Commit

Permalink
HV-2001 Enable CDI JPMS test
Browse files Browse the repository at this point in the history
  • Loading branch information
marko-bekhta committed Jul 23, 2024
1 parent feb05a5 commit 04835f8
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
11 changes: 6 additions & 5 deletions integrationtest/java/modules/cdi/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -24,13 +24,14 @@
<properties>
<surefire.jvm.args.additional>-Duser.language=en -Duser.country=US</surefire.jvm.args.additional>
<!--
Caused by: java.lang.IllegalAccessError: superinterface check failed:
class org.hibernate.validator.internal.engine.ValidatorFactoryImpl$Proxy$_$$_WeldClientProxy
(in module org.hibernate.validator) cannot access class org.jboss.weld.interceptor.util.proxy.TargetInstanceProxy
(in unnamed module @0x6d763516) because module org.hibernate.validator does not read unnamed module @0x6d763516
If we do not read all unnamed for jboss logging it'll lead to:
Caused by: java.lang.IllegalArgumentException: This library does not have private access to interface org.jboss.weld.logging.VersionLogger
at org.jboss.logging@3.6.0.Final/org.jboss.logging.Logger.getMessageLogger(Logger.java:2569)
at org.jboss.logging@3.6.0.Final/org.jboss.logging.Logger.getMessageLogger(Logger.java:2549)
at org.jboss.weld.logging.VersionLogger.<clinit>(VersionLogger.java:35)
-->
<surefire.jvm.args.add-opens>
--add-reads=org.hibernate.validator=ALL-UNNAMED
--add-reads=org.jboss.logging=ALL-UNNAMED
</surefire.jvm.args.add-opens>

<hibernate-validator-parent.path>../../../..</hibernate-validator-parent.path>
Expand Down
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@
<module>integrationtest/java/modules/no-el</module>
<module>integrationtest/java/modules/test-utils</module>
<!-- CDI module test has problems with Weld and JBoss Logging version mismatch. Disabled for now. -->
<!-- <module>integrationtest/java/modules/cdi</module> -->
<module>integrationtest/java/modules/cdi</module>
</modules>

<properties>
Expand Down

0 comments on commit 04835f8

Please sign in to comment.