diff --git a/src/main/java/org/codehaus/mojo/versions/SetPropertyMojo.java b/src/main/java/org/codehaus/mojo/versions/SetPropertyMojo.java index ca3d6fbb48..a84b1b62ad 100644 --- a/src/main/java/org/codehaus/mojo/versions/SetPropertyMojo.java +++ b/src/main/java/org/codehaus/mojo/versions/SetPropertyMojo.java @@ -22,12 +22,9 @@ import javax.xml.stream.XMLStreamException; import java.io.IOException; -import java.util.ArrayList; import java.util.Arrays; -import java.util.List; import java.util.Map; -import org.apache.commons.lang3.StringUtils; import org.apache.maven.plugin.MojoExecutionException; import org.apache.maven.plugin.MojoFailureException; import org.apache.maven.plugins.annotations.Mojo; @@ -37,7 +34,9 @@ import org.codehaus.mojo.versions.rewriting.ModifiedPomXMLEventReader; import org.codehaus.mojo.versions.utils.PropertiesVersionsFileReader; -import static org.apache.commons.lang3.StringUtils.isBlank; +import static org.apache.commons.lang3.StringUtils.defaultString; +import static org.apache.commons.lang3.StringUtils.isEmpty; +import static org.apache.commons.lang3.StringUtils.split; /** * Set a property to a given version without any sanity checks. Please be careful this can lead to changes which might @@ -83,31 +82,6 @@ public class SetPropertyMojo @Parameter( property = "propertiesVersionsFile" ) private String propertiesVersionsFile; - /** - * {@inheritDoc} - */ - @Override - public void execute() throws MojoExecutionException, MojoFailureException - { - List problems = new ArrayList<>(); - if ( isBlank( propertiesVersionsFile ) ) - { - if ( isBlank( newVersion ) ) - { - problems.add( "newVersion must not be empty" ); - } - if ( isBlank( property ) ) - { - problems.add( "property must not be empty" ); - } - } - if ( !problems.isEmpty() ) - { - throw new MojoExecutionException( "Invalid execution arguments: " + String.join( ", ", problems ) ); - } - super.execute(); - } - /** * @param pom the pom to update. * @throws MojoExecutionException when things go wrong @@ -118,9 +92,9 @@ public void execute() throws MojoExecutionException, MojoFailureException protected void update( ModifiedPomXMLEventReader pom ) throws MojoExecutionException, MojoFailureException, XMLStreamException { - Property[] propertiesConfig = null; - String properties = ""; - if ( !StringUtils.isEmpty( propertiesVersionsFile ) ) + Property[] propertiesConfig; + String properties; + if ( !isEmpty( propertiesVersionsFile ) ) { logWrongConfigWarning(); getLog().debug( "Reading properties and versions to update from file: " + propertiesVersionsFile ); @@ -139,10 +113,10 @@ protected void update( ModifiedPomXMLEventReader pom ) propertiesConfig = reader.getPropertiesConfig(); properties = reader.getProperties(); } - else if ( !StringUtils.isEmpty( property ) ) + else if ( !isEmpty( property ) ) { getLog().debug( "Reading properties and versions to update from property and newVersion " ); - propertiesConfig = Arrays.stream( StringUtils.split( property, "," ) ).map( + propertiesConfig = Arrays.stream( split( property, "," ) ).map( prp -> { Property propertyConfig = new Property( prp ); @@ -177,17 +151,18 @@ private void update( ModifiedPomXMLEventReader pom, Property[] propertiesConfig, { continue; } - PomHelper.setPropertyVersion( pom, version.getProfileId(), currentProperty.getName(), newVersionGiven ); + PomHelper.setPropertyVersion( pom, version.getProfileId(), currentProperty.getName(), + defaultString( newVersionGiven ) ); } } private void logWrongConfigWarning() { - if ( !StringUtils.isEmpty( property ) ) + if ( !isEmpty( property ) ) { getLog().warn( "-Dproperty provided but will be ignored as -DpropertiesVersionsFile is used" ); } - if ( !StringUtils.isEmpty( newVersion ) ) + if ( !isEmpty( newVersion ) ) { getLog().warn( "-DnewVersion provided but will be ignored as -DpropertiesVersionsFile is used" ); } diff --git a/src/test/java/org/codehaus/mojo/versions/SetPropertyMojoTest.java b/src/test/java/org/codehaus/mojo/versions/SetPropertyMojoTest.java index b3aecf2e2b..7af3a8d568 100644 --- a/src/test/java/org/codehaus/mojo/versions/SetPropertyMojoTest.java +++ b/src/test/java/org/codehaus/mojo/versions/SetPropertyMojoTest.java @@ -19,6 +19,8 @@ * under the License. */ +import java.nio.file.Files; + import org.apache.maven.plugin.MojoExecutionException; import org.codehaus.mojo.versions.utils.BaseMojoTestCase; import org.junit.Test; @@ -26,6 +28,7 @@ import static org.hamcrest.MatcherAssert.assertThat; import static org.hamcrest.Matchers.containsString; import static org.hamcrest.Matchers.is; +import static org.hamcrest.Matchers.matchesPattern; /** * Basic tests for {@linkplain SetPropertyMojoTest}. @@ -39,18 +42,18 @@ public void testNullNewVersion() throws Exception { SetPropertyMojo mojo = createMojo( "set-property", - "src/test/resources/org/codehaus/mojo/set-property/null-new-version-pom.xml" ); + "target/test-classes/org/codehaus/mojo/set-property/pom.xml" ); assertThat( mojo.getProject().getProperties(), is( mojo.getProject().getModel().getProperties() ) ); - try - { - mojo.execute(); - fail(); - } - catch ( MojoExecutionException e ) - { - assertThat( e.getMessage(), - containsString( "Invalid execution arguments: newVersion must not be empty" ) ); - } + + setVariableValueToObject( mojo, "property", "dummy-api-version" ); + setVariableValueToObject( mojo, "newVersion", null ); + + mojo.execute(); + + String output = String.join( "", Files.readAllLines( mojo.getProject().getFile().toPath() ) ) + .replaceAll( "\\s*", "" ); + assertThat( output, + matchesPattern( ".*.*.*.*" ) ); } @Test @@ -58,7 +61,11 @@ public void testNullProperty() throws Exception { SetPropertyMojo mojo = createMojo( "set-property", - "src/test/resources/org/codehaus/mojo/set-property/null-property-pom.xml" ); + "src/test/resources/org/codehaus/mojo/set-property/pom.xml" ); + + setVariableValueToObject( mojo, "property", null ); + setVariableValueToObject( mojo, "propertiesVersionsFile", null ); + setVariableValueToObject( mojo, "newVersion", "2.0.0" ); try { mojo.execute(); @@ -67,7 +74,7 @@ public void testNullProperty() catch ( MojoExecutionException e ) { assertThat( e.getMessage(), - containsString( "Invalid execution arguments: property must not be empty" ) ); + containsString( "Please provide either 'property' or 'propertiesVersionsFile' parameter." ) ); } } } diff --git a/src/test/java/org/codehaus/mojo/versions/utils/BaseMojoTestCase.java b/src/test/java/org/codehaus/mojo/versions/utils/BaseMojoTestCase.java index 4d499afd14..1bdb484d57 100644 --- a/src/test/java/org/codehaus/mojo/versions/utils/BaseMojoTestCase.java +++ b/src/test/java/org/codehaus/mojo/versions/utils/BaseMojoTestCase.java @@ -57,6 +57,7 @@ public abstract class BaseMojoTestCase extends AbstractMojoTestCase * Lookup the mojo leveraging the actual subprojects pom * and injects the project using the given pom file path. * + * @param target Mojo subclass * @param goal to execute on the plugin * @param pomFilePath path to the pom project to inject * @return a Mojo instance diff --git a/src/test/resources/org/codehaus/mojo/set-property/null-new-version-pom.xml b/src/test/resources/org/codehaus/mojo/set-property/null-new-version-pom.xml deleted file mode 100644 index 94e1f8d097..0000000000 --- a/src/test/resources/org/codehaus/mojo/set-property/null-new-version-pom.xml +++ /dev/null @@ -1,31 +0,0 @@ - - 4.0.0 - default-group - default-artifact - 1.0 - pom - - - 1.0.0 - - - - - localhost - dummy-api - ${dummy-api-version} - - - - - - - versions-maven-plugin - - dummy-api-version - - - - - \ No newline at end of file diff --git a/src/test/resources/org/codehaus/mojo/set-property/null-property-pom.xml b/src/test/resources/org/codehaus/mojo/set-property/pom.xml similarity index 87% rename from src/test/resources/org/codehaus/mojo/set-property/null-property-pom.xml rename to src/test/resources/org/codehaus/mojo/set-property/pom.xml index b429aa3b7b..303db45ef6 100644 --- a/src/test/resources/org/codehaus/mojo/set-property/null-property-pom.xml +++ b/src/test/resources/org/codehaus/mojo/set-property/pom.xml @@ -22,9 +22,7 @@ versions-maven-plugin - - 2.0.0 - +