Fix mocking/stubbing tests
diff --git a/maven-release-plugin/src/test/java/org/apache/maven/plugins/release/PerformReleaseMojoTest.java b/maven-release-plugin/src/test/java/org/apache/maven/plugins/release/PerformReleaseMojoTest.java
index 6e4217c..09b5a52 100644
--- a/maven-release-plugin/src/test/java/org/apache/maven/plugins/release/PerformReleaseMojoTest.java
+++ b/maven-release-plugin/src/test/java/org/apache/maven/plugins/release/PerformReleaseMojoTest.java
@@ -28,6 +28,7 @@
import java.io.File;
import java.util.Arrays;
+import org.apache.maven.execution.MavenSession;
import org.apache.maven.model.DistributionManagement;
import org.apache.maven.model.Profile;
import org.apache.maven.model.Site;
@@ -115,6 +116,9 @@
PerformReleaseMojo mojo =
(PerformReleaseMojo) lookupMojo( "perform", new File( testFileDirectory, "perform-without-site.xml" ) );
mojo.setBasedir( testFileDirectory );
+
+ MavenProject project = (MavenProject) getVariableValueFromObject( mojo, "project" );
+ setVariableValueToObject( mojo, "session", newMavenSession( project ) );
ReleaseDescriptor releaseDescriptor = new ReleaseDescriptor();
releaseDescriptor.setWorkingDirectory( workingDirectory.getAbsolutePath() );
@@ -148,6 +152,8 @@
DistributionManagement distributionManagement = new DistributionManagement();
distributionManagement.setSite( new Site() );
project.setDistributionManagement( distributionManagement );
+
+ setVariableValueToObject( mojo, "session", newMavenSession( project ) );
return mojo;
}
@@ -271,12 +277,14 @@
releaseDescriptor.setPerformGoals( "deploy site-deploy" );
releaseDescriptor.setAdditionalArguments( "-P prof1,2prof" );
- MavenProject project = (MavenProject) getVariableValueFromObject( mojo, "project" );
+ MavenSession session = (MavenSession) getVariableValueFromObject( mojo, "session");
Profile profile1 = new Profile();
profile1.setId( "prof1" );
+ session.getRequest().addProfile( profile1 );
Profile profile2 = new Profile();
profile2.setId( "2prof" );
- project.setActiveProfiles( Arrays.asList( new Profile[]{profile1, profile2} ) );
+ session.getRequest().addProfile( profile2 );
+ session.getRequest().setActiveProfiles( Arrays.asList( "prof1", "2prof" ) );
ReleaseManager mock = mock( ReleaseManager.class );
mojo.setReleaseManager( mock );
@@ -307,12 +315,14 @@
releaseDescriptor.setPerformGoals( "deploy site-deploy" );
releaseDescriptor.setAdditionalArguments( "-Dmaven.test.skip=true -P prof1,2prof" );
- MavenProject project = (MavenProject) getVariableValueFromObject( mojo, "project" );
+ MavenSession session = (MavenSession) getVariableValueFromObject( mojo, "session");
Profile profile1 = new Profile();
profile1.setId( "prof1" );
+ session.getRequest().addProfile( profile1 );
Profile profile2 = new Profile();
profile2.setId( "2prof" );
- project.setActiveProfiles( Arrays.asList( new Profile[]{profile1, profile2} ) );
+ session.getRequest().addProfile( profile2 );
+ session.getRequest().setActiveProfiles( Arrays.asList( "prof1", "2prof" ) );
ReleaseManager mock = mock( ReleaseManager.class );
mojo.setReleaseManager( mock );
diff --git a/maven-release-plugin/src/test/java/org/apache/maven/plugins/release/StageReleaseMojoTest.java b/maven-release-plugin/src/test/java/org/apache/maven/plugins/release/StageReleaseMojoTest.java
index 6ea0bb2..092d77e 100644
--- a/maven-release-plugin/src/test/java/org/apache/maven/plugins/release/StageReleaseMojoTest.java
+++ b/maven-release-plugin/src/test/java/org/apache/maven/plugins/release/StageReleaseMojoTest.java
@@ -93,12 +93,14 @@
{
StageReleaseMojo mojo = (StageReleaseMojo) lookupMojo( "stage", new File( workingDirectory, fileName ) );
mojo.setBasedir( workingDirectory );
-
+
MavenProject project = (MavenProject) getVariableValueFromObject( mojo, "project" );
DistributionManagement distributionManagement = new DistributionManagement();
distributionManagement.setSite( new Site() );
project.setDistributionManagement( distributionManagement );
+ setVariableValueToObject( mojo, "session", newMavenSession( project ) );
+
return mojo;
}