[arquillian-issues] [JBoss JIRA] (ARQ-2131) Fix for multiple JVM arguments not release

[Benjamin Marwell (JIRA)]

Benjamin Marwell created ARQ-2131:
-------------------------------------

             Summary: Fix for multiple JVM arguments not release
                 Key: ARQ-2131
                 URL: https://issues.jboss.org/browse/ARQ-2131
             Project: Arquillian
          Issue Type: Feature Request
          Components: WebSphere Containers
         Environment: {code}arquillian-container-was-1.0.0.Beta2{code}

{{arquillian.xml}} snippet:
{code:xml}  <container qualifier="liberty-ci-managed">
    <configuration>
      <property name="deployType">xml</property>
      <property name="javaVmArguments">-Dsetting1=x -Dsetting2=y</property>
    </configuration>
  </container>{code}
            Reporter: Benjamin Marwell
            Assignee: Gerhard Poul


Hello,

h2. expected output
Process started with (from Debug):
{{FINER: Starting server with command: [java, -Dsetting1=x, -Dsetting2=y, -javaagent:lib/bootstrap-agent.jar, -jar, lib/ws-launch.jar, defaultServer]}}

h2. actual output
note the missing comma betweed setting1 and setting2:
{{FINER: Starting server with command: [java, -Dsetting1=x -Dsetting2=y, -javaagent:lib/bootstrap-agent.jar, -jar, lib/ws-launch.jar, defaultServer]}}

h2. Solution
Already exists. This commit is currently not in a release:
https://github.com/arquillian/arquillian-container-was/commit/a1a77fe92418f49943a97a57a4dd1ff7ebc2bd25#diff-957685487719c29ce4c464fc2ec9cf39

What can't be seen from here: this commit is needed to have multiple JVM args. Otherwise all jvm args are concatenated into a single String.

So, the bug is fixed in trunk, but not released.

h2. Proposed test case
It is also missing a test  case.
Please add a test case with two or more {{-D}}-Parameters and check in your servlet if they are set.

Thank you very much!



--
This message was sent by Atlassian JIRA
(v7.2.3#72005)