You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by kh...@apache.org on 2016/05/25 21:10:11 UTC

svn commit: r1745541 - in /maven/plugins/trunk/maven-deploy-plugin/src/test/java/org/apache/maven/plugins/deploy: DeployFileMojoTest.java DeployMojoTest.java

Author: khmarbaise
Date: Wed May 25 21:10:11 2016
New Revision: 1745541

URL: http://svn.apache.org/viewvc?rev=1745541&view=rev
Log:
Cleaned up code.
 o Removed unused code.

Modified:
    maven/plugins/trunk/maven-deploy-plugin/src/test/java/org/apache/maven/plugins/deploy/DeployFileMojoTest.java
    maven/plugins/trunk/maven-deploy-plugin/src/test/java/org/apache/maven/plugins/deploy/DeployMojoTest.java

Modified: maven/plugins/trunk/maven-deploy-plugin/src/test/java/org/apache/maven/plugins/deploy/DeployFileMojoTest.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-deploy-plugin/src/test/java/org/apache/maven/plugins/deploy/DeployFileMojoTest.java?rev=1745541&r1=1745540&r2=1745541&view=diff
==============================================================================
--- maven/plugins/trunk/maven-deploy-plugin/src/test/java/org/apache/maven/plugins/deploy/DeployFileMojoTest.java (original)
+++ maven/plugins/trunk/maven-deploy-plugin/src/test/java/org/apache/maven/plugins/deploy/DeployFileMojoTest.java Wed May 25 21:10:11 2016
@@ -292,12 +292,6 @@ public class DeployFileMojoTest
 
         String repositoryLayout = (String) getVariableValueFromObject(  mojo, "repositoryLayout" );
 
-        String groupId = (String) getVariableValueFromObject( mojo, "groupId" );
-
-        String artifactId = (String) getVariableValueFromObject( mojo, "artifactId" );
-
-        String version = (String) getVariableValueFromObject( mojo, "version" );
-
         assertEquals( "legacy", repositoryLayout );
 
         try

Modified: maven/plugins/trunk/maven-deploy-plugin/src/test/java/org/apache/maven/plugins/deploy/DeployMojoTest.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-deploy-plugin/src/test/java/org/apache/maven/plugins/deploy/DeployMojoTest.java?rev=1745541&r1=1745540&r2=1745541&view=diff
==============================================================================
--- maven/plugins/trunk/maven-deploy-plugin/src/test/java/org/apache/maven/plugins/deploy/DeployMojoTest.java (original)
+++ maven/plugins/trunk/maven-deploy-plugin/src/test/java/org/apache/maven/plugins/deploy/DeployMojoTest.java Wed May 25 21:10:11 2016
@@ -29,31 +29,24 @@ import java.util.Collections;
 import java.util.List;
 import java.util.Properties;
 
-import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.repository.ArtifactRepository;
-import org.apache.maven.artifact.repository.ArtifactRepositoryFactory;
 import org.apache.maven.artifact.repository.layout.ArtifactRepositoryLayout;
 import org.apache.maven.execution.MavenSession;
 import org.apache.maven.plugin.MojoExecutionException;
+import org.apache.maven.plugin.testing.AbstractMojoTestCase;
+import org.apache.maven.plugin.testing.stubs.MavenProjectStub;
 import org.apache.maven.plugins.deploy.stubs.ArtifactDeployerStub;
 import org.apache.maven.plugins.deploy.stubs.ArtifactRepositoryStub;
 import org.apache.maven.plugins.deploy.stubs.DeployArtifactStub;
-import org.apache.maven.plugin.testing.AbstractMojoTestCase;
-import org.apache.maven.plugin.testing.stubs.MavenProjectStub;
 import org.apache.maven.project.MavenProject;
 import org.apache.maven.project.ProjectBuildingRequest;
 import org.apache.maven.repository.internal.MavenRepositorySystemSession;
-import org.apache.maven.shared.artifact.deploy.ArtifactDeployer;
 import org.codehaus.plexus.util.FileUtils;
-import org.junit.Assume;
 import org.junit.Ignore;
-import org.junit.internal.builders.IgnoredBuilder;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import org.mockito.MockitoAnnotations;
-import org.sonatype.aether.RepositorySystemSession;
 import org.sonatype.aether.impl.internal.SimpleLocalRepositoryManager;
-import org.sonatype.aether.repository.LocalRepositoryManager;
 
 /**
  * @author <a href="mailto:aramirez@apache.org">Allan Ramirez</a>
@@ -468,10 +461,6 @@ public class DeployMojoTest
         
         artifact.setFile( file );
         
-        for( Artifact attachment :project.getAttachedArtifacts() )
-        {}
-        
-        
         ArtifactRepositoryStub repo = getRepoStub( mojo );
         
         repo.setAppendToUrl( "basic-deploy-with-attached-artifacts" );