Skip to content

Commit

Permalink
Alter CDI testenricher exclusion so that embedded tests don't get it,…
Browse files Browse the repository at this point in the history
… but WFLY does.
  • Loading branch information
manovotn committed Aug 28, 2020
1 parent fda39ee commit bcc32e5
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 4 deletions.
4 changes: 0 additions & 4 deletions pom.xml
Expand Up @@ -346,10 +346,6 @@
<groupId>org.slf4j</groupId>
<artifactId>slf4j-api</artifactId>
</exclusion>
<exclusion>
<groupId>org.jboss.arquillian.testenricher</groupId>
<artifactId>arquillian-testenricher-cdi</artifactId>
</exclusion>
</exclusions>
<scope>test</scope>
</dependency>
Expand Down
13 changes: 13 additions & 0 deletions tests-arquillian/pom.xml
Expand Up @@ -143,6 +143,14 @@
<dependency>
<groupId>org.wildfly.arquillian</groupId>
<artifactId>wildfly-arquillian-common</artifactId>
<exclusions>
<!-- This dep brings in Arq. testenricher for CDI in the non-jakarta version
which causes issues in embedded tests -->
<exclusion>
<groupId>org.jboss.arquillian.testenricher</groupId>
<artifactId>arquillian-testenricher-cdi</artifactId>
</exclusion>
</exclusions>
</dependency>

<!-- Jakarta JAX/JWS are needed in jdk8+ -->
Expand Down Expand Up @@ -272,6 +280,11 @@
<artifactId>wildfly-arquillian-protocol-jmx</artifactId>
<scope>test</scope>
</dependency>
<!-- Due to exclusion on wildfly-arquillian-common, we need to add enricher manually for WFLY tests -->
<dependency>
<groupId>org.jboss.arquillian.testenricher</groupId>
<artifactId>arquillian-testenricher-cdi-jakarta</artifactId>
</dependency>
</dependencies>

<build>
Expand Down

0 comments on commit bcc32e5

Please sign in to comment.