Maven Surefire

Might be impossible to have empty strings in systemPropertyVariables element

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: 2.6
  • Fix Version/s: Backlog
  • Component/s: Maven Surefire Plugin
  • Labels:
    None
  • Complexity:
    Intermediate
  • Testcase included:
    yes
  • Number of attachments :
    1

Description

This stanza:

<systemProperties>
<property>
<name>emptyProperty</name>
<value></value>
</property>
</systemProperties>

...yields "" from System.getProperty("emptyProperty").

This (supposedly better) stanza:

<systemPropertyVariables>
<emptyProperty></emptyProperty>
</systemPropertyVariables>

...yields null from System.getProperty("emptyProperty").

A test case is attached that demonstrates the issue.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Laird Nelson added a comment -

I neglected to set the priority down from major, as the workaround is simply to continue to use the deprecated systemProperties stanza. My apologies.

Show
Laird Nelson added a comment - I neglected to set the priority down from major, as the workaround is simply to continue to use the deprecated systemProperties stanza. My apologies.
Hide
Permalink
Stefan Birkner added a comment -

The Bug happens inside of org.codehaus.plexus.component.configurator.converters.AbstractConfigurationConverter.fromExpression( PlexusConfiguration configuration, ExpressionEvaluator expressionEvaluator ). Within this method the empty string is converted to null.

Show
Stefan Birkner added a comment - The Bug happens inside of org.codehaus.plexus.component.configurator.converters.AbstractConfigurationConverter.fromExpression( PlexusConfiguration configuration, ExpressionEvaluator expressionEvaluator ). Within this method the empty string is converted to null.
Kristian Rosenvold made changes -
Field Original Value New Value
Priority Major [ 3 ] Minor [ 4 ]
Fix Version/s Backlog [ 13647 ]

People

  • Assignee:
    Unassigned
    Reporter:
    Laird Nelson
Vote (0)
Watch (2)

Dates

  • Created:
    Updated: