[ https://issues.jboss.org/browse/AS7-3987?page=com.atlassian.jira.plugin.s... ] Ondrej Zizka commented on AS7-3987: ----------------------------------- The other reason to have it split is that jboss.home is used in two different ways: 1) To pass AS location from outside to the testsuite (e.g. downloaded and unzipped EAP), 2) Internally by some tests to point to the AS instance they actually use, which is not the same as 1). Therefore, I'd prefer to keep the status quo to prevent confusion. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/AS7-3987?page=com.atlassian.jira.plugin.s... ] Paul Gier reopened AS7-3987: ---------------------------- Assignee: Ondrej Zizka (was: Paul Gier) The reason it's needed is because currently it requires setting both properties to run the arquillian modules and the testsuite against an AS zip. I'm not sure I understand why jboss.home can't be used for both. Do you have more information about this? -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/AS7-3987?page=com.atlassian.jira.plugin.s... ] Jason Greene updated AS7-3987: ------------------------------ Fix Version/s: 7.1.3.Final (EAP) (was: 7.1.2.Final (EAP)) -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/AS7-3987?page=com.atlassian.jira.plugin.s... ] Paul Gier commented on AS7-3987: -------------------------------- Ok, that makes sense. Maybe we can put a note in the README or somewhere in the POMs to describe the different usage. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/AS7-3987?page=com.atlassian.jira.plugin.s... ] Ondrej Zizka commented on AS7-3987: ----------------------------------- Alex, exactly. The other reason, besides this separation of meanings, is that jboss.home differs for each instance in test environment, whereas jboss.dist is the same (some tests need to reach to unchanged AS dist for various reasons). -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/AS7-3987?page=com.atlassian.jira.plugin.s... ] Paul Gier commented on AS7-3987: -------------------------------- {quote}Paul, they are never the same. jboss.dist is copied to target/jbossas[-*], and the tests run there.{quote} By default they are set to the same value in testsuite/pom.xml {code:xml} <jboss.dist>${project.basedir}/../build/target/jboss-as-${jboss.as.release.version}</jboss.dist> <jboss.home>${jboss.dist}</jboss.home> {code} The same default location is hardcoded in arquillian poms {code:xml} <properties> <jboss.home>${project.basedir}/../../build/target/jboss-as-${jboss.as.release.version}</jboss.home> </properties> {code} So they appear to be the same unless you override the default values. My use case is that we need to change this default path for EAP, so I'm trying to avoid having multiple places where this path needs to be changed. This broke testsuite runs in the past when not all of the values were updated. My suggestion would be to move the jboss.dist property to the parent pom, and then make jboss.home default to the value of jboss.dist in the arquillian poms. That way for the use case where the configuration and binaries are in the same location, there is only a single place to change the path. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/AS7-3987?page=com.atlassian.jira.plugin.s... ] Ondrej Zizka commented on AS7-3987: ----------------------------------- I've checked https://docs.jboss.org/author/display/AS71/AS+7+Testsuite+Test+Developer+... And remembered that I wanted to get rid of {{jboss.home}}. But there was some test using it, so I left it in the inconsistent state you see now. I would still rather get rid of {{jboss.home}} as it's not "officially supported" (read, documented on that page) and use {{jboss.dis}}t or {{jboss.inst}}, which both have clear meaning (unlike {{jboss.home}}), instead. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/AS7-3987?page=com.atlassian.jira.plugin.s... ] Brian Stansberry updated AS7-3987: ---------------------------------- Fix Version/s: 7.2.0.CR1 7.1.4.Final (EAP) (was: 7.1.3.Final (EAP)) -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira