You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Dan Tran (JIRA)" <ji...@codehaus.org> on 2008/08/16 08:00:28 UTC

[jira] Closed: (SUREFIRE-511) Proposal to use java Map for surefire's systemProperties

     [ http://jira.codehaus.org/browse/SUREFIRE-511?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dan Tran closed SUREFIRE-511.
-----------------------------

    Resolution: Fixed

Fixed at

Revision: 686454
Author: dantran
Date: 10:58:02 PM, Friday, August 15, 2008
Message:
- SUREFIRE-511: Replacement of 'systemProperties' to use Map interface.
- Add partial fix for SUREFIRE-513 to support SUREFIRE-511 ITs
- ignore eclipse workspace folder
----
Modified : /maven/surefire/trunk
Modified : /maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
Modified : /maven/surefire/trunk/maven-surefire-plugin/src/site/apt/examples/system-properties.apt
Modified : /maven/surefire/trunk/surefire-integration-tests/pom.xml
Added : /maven/surefire/trunk/surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/AbstractSurefireIT.java
Modified : /maven/surefire/trunk/surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/AbstractTestCaseTest.java
Modified : /maven/surefire/trunk/surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/SystemPropertiesTest.java
Modified : /maven/surefire/trunk/surefire-integration-tests/src/test/java/org/apache/maven/surefire/its/TestNgSuiteXmlTest.java
Modified : /maven/surefire/trunk/surefire-integration-tests/src/test/resources/system-properties/pom.xml
Modified : /maven/surefire/trunk/surefire-integration-tests/src/test/resources/system-properties/src/test/java/systemProperties/BasicTest.java
Modified : /maven/surefire/trunk/surefire-integration-tests/src/test/resources/testng-suite-xml/pom.xml



> Proposal to use java Map for surefire's systemProperties 
> ---------------------------------------------------------
>
>                 Key: SUREFIRE-511
>                 URL: http://jira.codehaus.org/browse/SUREFIRE-511
>             Project: Maven Surefire
>          Issue Type: Improvement
>          Components: plugin
>    Affects Versions: 2.4.3
>            Reporter: Dan Tran
>            Assignee: Dan Tran
>             Fix For: 2.5
>
>         Attachments: SUREFIRE-511.patch
>
>
> There is a known problem documented at http://maven.apache.org/plugins/maven-surefire-plugin/examples/system-properties.html  where maven properties other than java String are not able to pass into surefire
> This is mainly because systemProperties is mapped to java Properties class.  However if we map it to java Map 
> the problem solves.  
> see http://svn.codehaus.org/mojo/trunk/mojo/build-helper-maven-plugin/src/it/surefire for details.
> So I would like to propose that we deprecate "systemProperties" and introduce a new one "systemPropertyVariables" to use java Map ( similar to surefire's environmentVariables"
> Thoughts?

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira