You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by rf...@apache.org on 2016/05/21 09:58:23 UTC
svn commit: r1744885 -
/maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/
Author: rfscholte
Date: Sat May 21 09:58:23 2016
New Revision: 1744885
URL: http://svn.apache.org/viewvc?rev=1744885&view=rev
Log:
Make AbstractRewritingReleasePhaseTestCase an parameterized test, so we can simply add and verify new modelETLs
Modified:
maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractEditModeRewritingReleasePhaseTestCase.java
maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractRewritingReleasePhaseTestCase.java
maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/GenerateReleasePomsPhaseTest.java
maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForBranchPhaseTest.java
maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhaseTest.java
maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForReleasePhaseTest.java
Modified: maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractEditModeRewritingReleasePhaseTestCase.java
URL: http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractEditModeRewritingReleasePhaseTestCase.java?rev=1744885&r1=1744884&r2=1744885&view=diff
==============================================================================
--- maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractEditModeRewritingReleasePhaseTestCase.java (original)
+++ maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractEditModeRewritingReleasePhaseTestCase.java Sat May 21 09:58:23 2016
@@ -56,6 +56,11 @@ import org.mockito.internal.util.reflect
public abstract class AbstractEditModeRewritingReleasePhaseTestCase
extends AbstractRewritingReleasePhaseTestCase
{
+ public AbstractEditModeRewritingReleasePhaseTestCase( String modelETL )
+ {
+ super( modelETL );
+ }
+
@Test
public void testRewriteBasicPomWithEditMode()
throws Exception
Modified: maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractRewritingReleasePhaseTestCase.java
URL: http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractRewritingReleasePhaseTestCase.java?rev=1744885&r1=1744884&r2=1744885&view=diff
==============================================================================
--- maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractRewritingReleasePhaseTestCase.java (original)
+++ maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractRewritingReleasePhaseTestCase.java Sat May 21 09:58:23 2016
@@ -30,6 +30,8 @@ import static org.mockito.Mockito.when;
import java.io.IOException;
import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collection;
import java.util.Collections;
import java.util.List;
@@ -46,8 +48,12 @@ import org.apache.maven.shared.release.e
import org.apache.maven.shared.release.scm.DefaultScmRepositoryConfigurator;
import org.apache.maven.shared.release.scm.ReleaseScmRepositoryException;
import org.apache.maven.shared.release.scm.ScmRepositoryConfigurator;
+import org.apache.maven.shared.release.transform.jdom.JDomModelETLFactory;
import org.apache.maven.shared.release.util.ReleaseUtil;
import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.Parameterized;
+import org.junit.runners.Parameterized.Parameters;
import org.mockito.internal.util.reflection.Whitebox;
/**
@@ -55,9 +61,38 @@ import org.mockito.internal.util.reflect
*
* @author <a href="mailto:brett@apache.org">Brett Porter</a>
*/
+@RunWith(Parameterized.class)
public abstract class AbstractRewritingReleasePhaseTestCase
extends AbstractReleaseTestCase
{
+ private String modelETL;
+
+ @Parameters
+ public static Collection<Object[]> data()
+ {
+ return Arrays.asList( new Object[][] { { JDomModelETLFactory.ROLE_HINT } } );
+ }
+
+ public AbstractRewritingReleasePhaseTestCase( String modelETL )
+ {
+ this.modelETL = modelETL;
+ }
+
+ @Override
+ public void setUp()
+ throws Exception
+ {
+ super.setUp();
+ phase = (ReleasePhase) lookup( ReleasePhase.ROLE, getRoleHint() );
+
+ if( phase instanceof AbstractRewritePomsPhase)
+ {
+ ((AbstractRewritePomsPhase) phase).setModelETL( modelETL );
+ }
+ }
+
+ protected abstract String getRoleHint();
+
@Test
public void testRewriteBasicPom()
throws Exception
Modified: maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/GenerateReleasePomsPhaseTest.java
URL: http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/GenerateReleasePomsPhaseTest.java?rev=1744885&r1=1744884&r2=1744885&view=diff
==============================================================================
--- maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/GenerateReleasePomsPhaseTest.java (original)
+++ maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/GenerateReleasePomsPhaseTest.java Sat May 21 09:58:23 2016
@@ -64,14 +64,24 @@ public class GenerateReleasePomsPhaseTes
private ScmProvider scmProviderMock;
+ public GenerateReleasePomsPhaseTest( String modelETL )
+ {
+ super( modelETL );
+ }
+
public void setUp()
throws Exception
{
super.setUp();
- phase = (ReleasePhase) lookup( ReleasePhase.ROLE, "generate-release-poms" );
scmProviderMock = null;
}
+
+ @Override
+ protected String getRoleHint()
+ {
+ return "generate-release-poms";
+ }
// TODO: MRELEASE-262
// @Test public void testRewriteInternalRangeDependency() throws Exception
Modified: maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForBranchPhaseTest.java
URL: http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForBranchPhaseTest.java?rev=1744885&r1=1744884&r2=1744885&view=diff
==============================================================================
--- maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForBranchPhaseTest.java (original)
+++ maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForBranchPhaseTest.java Sat May 21 09:58:23 2016
@@ -46,12 +46,15 @@ public class RewritePomsForBranchPhaseTe
private static final String ALTERNATIVE_NEXT_VERSION = "2.0-SNAPSHOT";
- public void setUp()
- throws Exception
+ public RewritePomsForBranchPhaseTest( String modelETL )
{
- super.setUp();
+ super( modelETL );
+ }
- phase = (ReleasePhase) lookup( ReleasePhase.ROLE, "rewrite-poms-for-branch" );
+ @Override
+ protected String getRoleHint()
+ {
+ return "rewrite-poms-for-branch";
}
protected List<MavenProject> prepareReactorProjects( String path, boolean copyFiles )
Modified: maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhaseTest.java
URL: http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhaseTest.java?rev=1744885&r1=1744884&r2=1744885&view=diff
==============================================================================
--- maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhaseTest.java (original)
+++ maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForDevelopmentPhaseTest.java Sat May 21 09:58:23 2016
@@ -55,12 +55,15 @@ public class RewritePomsForDevelopmentPh
private static final String ALTERNATIVE_RELEASE_VERSION = "2.0";
- public void setUp()
- throws Exception
+ public RewritePomsForDevelopmentPhaseTest( String modelETL )
{
- super.setUp();
+ super( modelETL );
+ }
- phase = (ReleasePhase) lookup( ReleasePhase.ROLE, "rewrite-poms-for-development" );
+ @Override
+ protected String getRoleHint()
+ {
+ return "rewrite-poms-for-development";
}
@Test
Modified: maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForReleasePhaseTest.java
URL: http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForReleasePhaseTest.java?rev=1744885&r1=1744884&r2=1744885&view=diff
==============================================================================
--- maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForReleasePhaseTest.java (original)
+++ maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/RewritePomsForReleasePhaseTest.java Sat May 21 09:58:23 2016
@@ -46,12 +46,15 @@ public class RewritePomsForReleasePhaseT
private static final String ALTERNATIVE_NEXT_VERSION = "2.0";
- public void setUp()
- throws Exception
+ public RewritePomsForReleasePhaseTest( String modelETL )
{
- super.setUp();
+ super( modelETL );
+ }
- phase = (ReleasePhase) lookup( ReleasePhase.ROLE, "rewrite-poms-for-release" );
+ @Override
+ protected String getRoleHint()
+ {
+ return "rewrite-poms-for-release";
}
protected List<MavenProject> prepareReactorProjects( String path, boolean copyFiles )