Fix some failures using wildfly-swarm profile#423
Fix some failures using wildfly-swarm profile#423juangon wants to merge 1 commit intojavaee-samples:masterfrom
Conversation
…nd wildfly swarm version. 2- Exclude payara container to avoid duplicated arquillian container error
|
Thx again! I wonder though, the test-utils project should probably not have the |
|
Yes, that's weird! But I couldn't fix it other way. Dependency tree looks like this for ejb and wildfly-swarm profile for example: [INFO] org.javaee7:ejb-stateless:war:1.0-SNAPSHOT Weird... |
|
Did you use any of the profile flags when looking for that dependency graph? It looks like to be just the default profile, which is deactivated when using At any length, could you try replacing the test-utils pom with the following content to see if that works? <?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
<modelVersion>4.0.0</modelVersion>
<properties>
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
<maven.compiler.source>1.7</maven.compiler.source>
<maven.compiler.target>1.7</maven.compiler.target>
</properties>
<groupId>org.javaee7</groupId>
<artifactId>test-utils</artifactId>
<version>1.0-SNAPSHOT</version>
<name>Java EE 7 Sample: javaee7-samples - test-utils</name>
<dependencyManagement>
<dependencies>
<dependency>
<groupId>org.jboss.arquillian</groupId>
<artifactId>arquillian-bom</artifactId>
<version>1.1.11.Final</version>
<scope>import</scope>
<type>pom</type>
</dependency>
</dependencies>
</dependencyManagement>
<dependencies>
<dependency>
<groupId>junit</groupId>
<artifactId>junit</artifactId>
<version>4.11</version>
</dependency>
<dependency>
<groupId>org.jboss.arquillian.container</groupId>
<artifactId>arquillian-container-test-api</artifactId>
<version>1.1.11.Final</version>
</dependency>
<dependency>
<groupId>org.jboss.shrinkwrap.resolver</groupId>
<artifactId>shrinkwrap-resolver-api-maven</artifactId>
</dependency>
</dependencies>
</project> |
|
@arjantijms wow that worked! Let me redo this PR with your changes and new versions.. |
|
@arjantijms would it be ok to use a global pom.xml property? The problem I see is that, now that test-utils doesn't have parent pom.xml, if compiled alone won't work, as that property won't exist. EDIT: I actually mean pom.xml property, not arquillian. |
|
Depending on what you exactly want to do, you may use some defaults. Still, it's probably unlikely to have to compile the test-utils on their own without going through the parent. |
|
@arjantijms just want to unify ${arquillian.version} in parent, then use it in test-utils so everyone uses it. So do you agree to use that approach as, as you said, it isn't very likely to use it alone? |
|
I think that would be fine. Additionally, the version could always be provided externally if really needed to be compiled stand-alone via a -D option. |
|
Made other PR here: #424 Closing this one. Thanks a lot! |
Hi @arjantijms. Here comes more fixes for Wildfly Swarm that improves our coverage.
1-Upgrade arquillian and wildfly swarm version. In latest version there are two issues fixed that affects wildfly swarm: https://issues.jboss.org/browse/ARQ-2107 and https://issues.jboss.org/browse/ARQ-2109.
2- Exclude payara container to avoid duplicated arquillian container error. This error is reproducible since some of the latest commits changing the payara container. Weirdly, this dependency comes in when using test-utils.
Thanks!