[maven-install-plugin] branch MSHARED-695 updated (8583219 -> a696cc4)

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[maven-install-plugin] branch MSHARED-695 updated (8583219 -> a696cc4)

khmarbaise
This is an automated email from the ASF dual-hosted git repository.

khmarbaise pushed a change to branch MSHARED-695
in repository https://gitbox.apache.org/repos/asf/maven-install-plugin.git.


 discard 8583219  Based on moving the checksum generation to deploy plugin this test can not check for the checksums.
 discard 5dea819  Downgrade maven-invoker-plugin version 3.0.1
 discard 09e47d1  fixup! [MSHARED-695] - WIP - Move checksum generation from install to deploy  o Starting to use the VOTE version.
 discard 41367c0  [MSHARED-695] - WIP - Move checksum generation from install to deploy  o Starting to use the VOTE version.
 discard 9d56fdc  Using special version of artifact-transfer
 discard 3c02f95  [MSHARED-695] - WIP - Move checksum generation from install to deploy  o First step get tests running.
 discard 48b4579  [MSHARED-695] - Move checksum generation from install to deploy
     new a696cc4  [MSHARED-695] - Move checksum generation from install to deploy

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (8583219)
            \
             N -- N -- N   refs/heads/MSHARED-695 (a696cc4)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:

Reply | Threaded
Open this post in threaded view
|

[maven-install-plugin] 01/01: [MSHARED-695] - Move checksum generation from install to deploy

khmarbaise
This is an automated email from the ASF dual-hosted git repository.

khmarbaise pushed a commit to branch MSHARED-695
in repository https://gitbox.apache.org/repos/asf/maven-install-plugin.git

commit a696cc4ef93dd12fdd2d44323856779ae3fdcdd4
Author: Karl Heinz Marbaise <[hidden email]>
AuthorDate: Thu Aug 9 18:41:05 2018 +0200

    [MSHARED-695] - Move checksum generation from install to deploy
---
 pom.xml                                            |  3 +-
 src/it/attach-jar-checksum/verify.bsh              | 36 +++++++++-------
 src/it/generate-pom-auto-1/verify.bsh              |  9 ----
 src/it/jar-sources-javadoc/verify.bsh              | 32 +++++++-------
 .../verify.bsh                                     | 26 +++++++-----
 .../local-repo-override-with-checksum/verify.bsh   | 23 ++++++----
 src/it/pom-checksum/verify.bsh                     | 23 ++++++----
 .../maven/plugins/install/AbstractInstallMojo.java | 10 +----
 .../maven/plugins/install/InstallFileMojo.java     |  6 +--
 .../apache/maven/plugins/install/InstallMojo.java  | 10 ++---
 .../maven/plugins/install/InstallFileMojoTest.java | 25 +++++------
 .../maven/plugins/install/InstallMojoTest.java     | 49 ++++++++++------------
 .../unit/basic-install-checksum/plugin-config.xml  |  1 -
 .../unit/configured-install-test/plugin-config.xml |  1 -
 .../install-file-with-checksum/plugin-config.xml   |  1 -
 15 files changed, 128 insertions(+), 127 deletions(-)

diff --git a/pom.xml b/pom.xml
index fdbe17f..3c3ab3b 100644
--- a/pom.xml
+++ b/pom.xml
@@ -82,7 +82,7 @@
     <dependency>
       <groupId>org.apache.maven.shared</groupId>
       <artifactId>maven-artifact-transfer</artifactId>
-      <version>0.9.1</version>
+      <version>0.10.0</version>
     </dependency>
 
     <!-- dependencies to annotations -->
@@ -149,6 +149,7 @@
             <plugin>
               <groupId>org.apache.maven.plugins</groupId>
               <artifactId>maven-invoker-plugin</artifactId>
+              <version>3.0.1</version>
               <configuration>
                 <cloneProjectsTo>${project.build.directory}/it</cloneProjectsTo>
                 <cloneClean>true</cloneClean>
diff --git a/src/it/attach-jar-checksum/verify.bsh b/src/it/attach-jar-checksum/verify.bsh
index 73ce5bc..e791219 100644
--- a/src/it/attach-jar-checksum/verify.bsh
+++ b/src/it/attach-jar-checksum/verify.bsh
@@ -25,24 +25,25 @@ import org.apache.maven.plugins.install.Utils;
 String[] paths =
 {
     "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.pom",
-    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.pom.md5",
-    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.pom.sha1",
     "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.jar",
-    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.jar.md5",
-    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.jar.sha1",
     "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT-sources.jar",
-    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT-sources.jar.md5",
-    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT-sources.jar.sha1",
     "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/maven-metadata-local.xml",
     "org/apache/maven/its/install/ajc/test/maven-metadata-local.xml",
 };
 
-Set cksumToCheckPaths = new HashSet( Arrays.asList( new String[]
+String[] pathsWhichShouldNotExist =
 {
-    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.pom",
-    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.jar",
-    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT-sources.jar"
-} ) );
+    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.pom.md5",
+    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.pom.sha1",
+    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.jar.md5",
+    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.jar.sha1",
+    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT-sources.jar.md5",
+    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT-sources.jar.sha1",
+    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/maven-metadata-local.xml.md5",
+    "org/apache/maven/its/install/ajc/test/1.0-SNAPSHOT/maven-metadata-local.xml.sha1",
+    "org/apache/maven/its/install/ajc/test/maven-metadata-local.xml.md5",
+    "org/apache/maven/its/install/ajc/test/maven-metadata-local.xml.sha1",
+};
 
 for ( String path : paths )
 {
@@ -52,10 +53,15 @@ for ( String path : paths )
     {
         throw new FileNotFoundException( "Missing: " + file.getAbsolutePath() );
     }
-    if ( cksumToCheckPaths.contains( path ) )
-    {    
-        System.out.println( "Verifying " + file );
-        Utils.verifyChecksum( file );
+}
+
+for ( String path : pathsWhichShouldNotExist )
+{
+    File file = new File( localRepositoryPath, path );
+    System.out.println( "Checking for existence of " + file );
+    if ( file.isFile() )
+    {
+        throw new IOException( "Existing: " + file.getAbsolutePath() );
     }
 }
 
diff --git a/src/it/generate-pom-auto-1/verify.bsh b/src/it/generate-pom-auto-1/verify.bsh
index e4152c8..d5ea928 100644
--- a/src/it/generate-pom-auto-1/verify.bsh
+++ b/src/it/generate-pom-auto-1/verify.bsh
@@ -26,11 +26,7 @@ String[] paths =
 {
     "org/apache/maven/its/install/gpa1/test/maven-metadata-local.xml",
     "org/apache/maven/its/install/gpa1/test/0.1/test-0.1.jar",
-    "org/apache/maven/its/install/gpa1/test/0.1/test-0.1.jar.md5",
-    "org/apache/maven/its/install/gpa1/test/0.1/test-0.1.jar.sha1",
     "org/apache/maven/its/install/gpa1/test/0.1/test-0.1.pom",
-    "org/apache/maven/its/install/gpa1/test/0.1/test-0.1.pom.md5",
-    "org/apache/maven/its/install/gpa1/test/0.1/test-0.1.pom.sha1",
 };
 
 Set cksumToCheckPaths = new HashSet( Arrays.asList( new String[]
@@ -47,11 +43,6 @@ for ( String path : paths )
     {
         throw new FileNotFoundException( "Missing: " + file.getAbsolutePath() );
     }
-    if ( cksumToCheckPaths.contains( path ) )
-    {    
-        System.out.println( "Verifying " + file );
-        Utils.verifyChecksum( file );
-    }
 }
 
 return true;
diff --git a/src/it/jar-sources-javadoc/verify.bsh b/src/it/jar-sources-javadoc/verify.bsh
index 6e4abd8..9f20faf 100644
--- a/src/it/jar-sources-javadoc/verify.bsh
+++ b/src/it/jar-sources-javadoc/verify.bsh
@@ -19,6 +19,7 @@
 
 import java.io.*;
 import java.util.*;
+import java.lang.*;
 
 import org.apache.maven.plugins.install.Utils;
 
@@ -26,27 +27,23 @@ String[] paths =
 {
     "org/apache/maven/its/install/jsj/test/maven-metadata-local.xml",
     "org/apache/maven/its/install/jsj/test/0.1/test-0.1.jar",
+    "org/apache/maven/its/install/jsj/test/0.1/test-0.1.pom",
+    "org/apache/maven/its/install/jsj/test/0.1/test-0.1-sources.jar",
+    "org/apache/maven/its/install/jsj/test/0.1/test-0.1-javadoc.jar",
+};
+
+String[] pathsWhichShouldNotExist =
+{
     "org/apache/maven/its/install/jsj/test/0.1/test-0.1.jar.md5",
     "org/apache/maven/its/install/jsj/test/0.1/test-0.1.jar.sha1",
-    "org/apache/maven/its/install/jsj/test/0.1/test-0.1.pom",
     "org/apache/maven/its/install/jsj/test/0.1/test-0.1.pom.md5",
     "org/apache/maven/its/install/jsj/test/0.1/test-0.1.pom.sha1",
-    "org/apache/maven/its/install/jsj/test/0.1/test-0.1-sources.jar",
     "org/apache/maven/its/install/jsj/test/0.1/test-0.1-sources.jar.md5",
     "org/apache/maven/its/install/jsj/test/0.1/test-0.1-sources.jar.sha1",
-    "org/apache/maven/its/install/jsj/test/0.1/test-0.1-javadoc.jar",
     "org/apache/maven/its/install/jsj/test/0.1/test-0.1-javadoc.jar.md5",
     "org/apache/maven/its/install/jsj/test/0.1/test-0.1-javadoc.jar.sha1",
 };
 
-Set cksumToCheckPaths = new HashSet( Arrays.asList( new String[]
-{
-    "org/apache/maven/its/install/jsj/test/0.1/test-0.1.jar",
-    "org/apache/maven/its/install/jsj/test/0.1/test-0.1.pom",
-    "org/apache/maven/its/install/jsj/test/0.1/test-0.1-sources.jar",
-    "org/apache/maven/its/install/jsj/test/0.1/test-0.1-javadoc.jar"
-} ) );
-
 for ( String path : paths )
 {
     File file = new File( localRepositoryPath, path );
@@ -55,10 +52,15 @@ for ( String path : paths )
     {
         throw new FileNotFoundException( "Missing: " + file.getAbsolutePath() );
     }
-    if ( cksumToCheckPaths.contains( path ) )
-    {    
-        System.out.println( "Verifying " + file );
-        Utils.verifyChecksum( file );
+}
+
+for ( String path : pathsWhichShouldNotExist )
+{
+    File file = new File( localRepositoryPath, path );
+    System.out.println( "Checking for non existence of " + file );
+    if ( file.isFile() )
+    {
+        throw new IOException( "Missing: " + file.getAbsolutePath() );
     }
 }
 
diff --git a/src/it/local-repo-override-with-checksum-generatePom/verify.bsh b/src/it/local-repo-override-with-checksum-generatePom/verify.bsh
index 4148919..eadace5 100644
--- a/src/it/local-repo-override-with-checksum-generatePom/verify.bsh
+++ b/src/it/local-repo-override-with-checksum-generatePom/verify.bsh
@@ -19,25 +19,24 @@
 
 import java.io.*;
 import java.util.*;
+import java.lang.*;
 
 import org.apache.maven.plugins.install.Utils;
 
 String[] paths =
 {
     "gid/test/0.1/test-0.1.jar",
+    "gid/test/0.1/test-0.1.pom",
+};
+
+String[] pathsWhichShouldNotExist =
+{
     "gid/test/0.1/test-0.1.jar.md5",
     "gid/test/0.1/test-0.1.jar.sha1",
-    "gid/test/0.1/test-0.1.pom",
     "gid/test/0.1/test-0.1.pom.md5",
     "gid/test/0.1/test-0.1.pom.sha1",
 };
 
-Set cksumToCheckPaths = new HashSet( Arrays.asList( new String[]
-{
-    "gid/test/0.1/test-0.1.pom",
-    "gid/test/0.1/test-0.1.jar"
-} ) );
-
 for ( String path : paths )
 {
     File file = new File( new File( basedir, "target/local-repo" ), path );
@@ -46,10 +45,15 @@ for ( String path : paths )
     {
         throw new FileNotFoundException( "Existing: " + file.getAbsolutePath() );
     }
-    if ( cksumToCheckPaths.contains( path ) )
-    {    
-        System.out.println( "Verifying " + file );
-        Utils.verifyChecksum( file );
+}
+
+for ( String path : pathsWhichShouldNotExist )
+{
+    File file = new File( new File( basedir, "target/local-repo" ), path );
+    System.out.println( "Checking for non existence of " + file );
+    if ( file.isFile() )
+    {
+        throw new IOException( "Existing: " + file.getAbsolutePath() );
     }
 }
 
diff --git a/src/it/local-repo-override-with-checksum/verify.bsh b/src/it/local-repo-override-with-checksum/verify.bsh
index 33178b7..11208cc 100644
--- a/src/it/local-repo-override-with-checksum/verify.bsh
+++ b/src/it/local-repo-override-with-checksum/verify.bsh
@@ -19,34 +19,39 @@
 
 import java.io.*;
 import java.util.*;
+import java.lang.*;
 
 import org.apache.maven.plugins.install.Utils;
 
 String[] paths =
 {
     "gid/test/0.1/test-0.1.jar",
+};
+
+String[] pathsWhichAreNotAllowedToBeThere =
+{
     "gid/test/0.1/test-0.1.jar.md5",
     "gid/test/0.1/test-0.1.jar.sha1",
 };
 
-Set cksumToCheckPaths = new HashSet( Arrays.asList( new String[]
+for ( String path : pathsWhichAreNotAllowedToBeThere )
 {
-    "gid/test/0.1/test-0.1.jar"
-} ) );
+    File file = new File( new File( basedir, "target/local-repo" ), path );
+    System.out.println( "Checking for existence of " + file );
+    if ( file.isFile() )
+    {
+        throw new IOException( "Existing file which should not be there: " + file.getAbsolutePath() );
+    }
+}
 
 for ( String path : paths )
 {
     File file = new File( new File( basedir, "target/local-repo" ), path );
-    System.out.println( "Checking for existence of " + file );
+    System.out.println( "Checking for non existence of " + file );
     if ( !file.isFile() )
     {
         throw new FileNotFoundException( "Existing: " + file.getAbsolutePath() );
     }
-    if ( cksumToCheckPaths.contains( path ) )
-    {    
-        System.out.println( "Verifying " + file );
-        Utils.verifyChecksum( file );
-    }
 }
 
 return true;
diff --git a/src/it/pom-checksum/verify.bsh b/src/it/pom-checksum/verify.bsh
index c852919..a8cff55 100644
--- a/src/it/pom-checksum/verify.bsh
+++ b/src/it/pom-checksum/verify.bsh
@@ -19,22 +19,22 @@
 
 import java.io.*;
 import java.util.*;
+import java.lang.*;
 
 import org.apache.maven.plugins.install.Utils;
 
 String[] paths =
 {
     "org/apache/maven/its/install/pc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.pom",
-    "org/apache/maven/its/install/pc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.pom.md5",
-    "org/apache/maven/its/install/pc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.pom.sha1",
     "org/apache/maven/its/install/pc/test/1.0-SNAPSHOT/maven-metadata-local.xml",
     "org/apache/maven/its/install/pc/test/maven-metadata-local.xml",
 };
 
-Set cksumToCheckPaths = new HashSet( Arrays.asList( new String[]
+String[] pathsWhichShouldNotExist =
 {
-    "org/apache/maven/its/install/pc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.pom"
-} ) );
+    "org/apache/maven/its/install/pc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.pom.md5",
+    "org/apache/maven/its/install/pc/test/1.0-SNAPSHOT/test-1.0-SNAPSHOT.pom.sha1",
+};
 
 for ( String path : paths )
 {
@@ -44,10 +44,15 @@ for ( String path : paths )
     {
         throw new FileNotFoundException( "Missing: " + file.getAbsolutePath() );
     }
-    if ( cksumToCheckPaths.contains( path ) )
-    {    
-        System.out.println( "Verifying " + file );
-        Utils.verifyChecksum( file );
+}
+
+for ( String path : pathsWhichShouldNotExist )
+{
+    File file = new File( localRepositoryPath, path );
+    System.out.println( "Checking for none existence of " + file );
+    if ( file.isFile() )
+    {
+        throw new IOException( "Found: " + file.getAbsolutePath() );
     }
 }
 
diff --git a/src/main/java/org/apache/maven/plugins/install/AbstractInstallMojo.java b/src/main/java/org/apache/maven/plugins/install/AbstractInstallMojo.java
index d7e6561..3584d78 100644
--- a/src/main/java/org/apache/maven/plugins/install/AbstractInstallMojo.java
+++ b/src/main/java/org/apache/maven/plugins/install/AbstractInstallMojo.java
@@ -28,7 +28,7 @@ import org.apache.maven.plugins.annotations.Component;
 import org.apache.maven.plugins.annotations.Parameter;
 import org.apache.maven.project.ProjectBuildingRequest;
 import org.apache.maven.project.artifact.ProjectArtifactMetadata;
-import org.apache.maven.shared.repository.RepositoryManager;
+import org.apache.maven.shared.transfer.repository.RepositoryManager;
 
 /**
  * Common fields for installation mojos.
@@ -44,14 +44,6 @@ public abstract class AbstractInstallMojo
     protected RepositoryManager repositoryManager;
 
     /**
-     * Flag whether to create checksums (MD5, SHA-1) or not.
-     *
-     * @since 2.2
-     */
-    @Parameter( property = "createChecksum", defaultValue = "false" )
-    protected boolean createChecksum;
-
-    /**
      * Whether to update the metadata to make the artifact a release version.
      */
     @Parameter( property = "updateReleaseInfo", defaultValue = "false" )
diff --git a/src/main/java/org/apache/maven/plugins/install/InstallFileMojo.java b/src/main/java/org/apache/maven/plugins/install/InstallFileMojo.java
index 4115694..19dfb44 100644
--- a/src/main/java/org/apache/maven/plugins/install/InstallFileMojo.java
+++ b/src/main/java/org/apache/maven/plugins/install/InstallFileMojo.java
@@ -53,8 +53,8 @@ import org.apache.maven.project.ProjectBuilder;
 import org.apache.maven.project.ProjectBuildingException;
 import org.apache.maven.project.ProjectBuildingRequest;
 import org.apache.maven.project.artifact.ProjectArtifactMetadata;
-import org.apache.maven.shared.project.install.ProjectInstaller;
-import org.apache.maven.shared.project.install.ProjectInstallerRequest;
+import org.apache.maven.shared.transfer.project.install.ProjectInstaller;
+import org.apache.maven.shared.transfer.project.install.ProjectInstallerRequest;
 import org.apache.maven.shared.utils.Os;
 import org.apache.maven.shared.utils.ReaderFactory;
 import org.apache.maven.shared.utils.WriterFactory;
@@ -296,7 +296,7 @@ public class InstallFileMojo
         {
             // CHECKSTYLE_OFF: LineLength
             ProjectInstallerRequest projectInstallerRequest =
-                new ProjectInstallerRequest().setProject( project ).setCreateChecksum( createChecksum ).setUpdateReleaseInfo( updateReleaseInfo );
+                new ProjectInstallerRequest().setProject( project );
             // CHECKSTYLE_ON: LineLength
 
             installer.install( buildingRequest, projectInstallerRequest );
diff --git a/src/main/java/org/apache/maven/plugins/install/InstallMojo.java b/src/main/java/org/apache/maven/plugins/install/InstallMojo.java
index 2eefc43..722c895 100644
--- a/src/main/java/org/apache/maven/plugins/install/InstallMojo.java
+++ b/src/main/java/org/apache/maven/plugins/install/InstallMojo.java
@@ -33,10 +33,10 @@ import org.apache.maven.plugins.annotations.Mojo;
 import org.apache.maven.plugins.annotations.Parameter;
 import org.apache.maven.project.MavenProject;
 import org.apache.maven.project.ProjectBuildingRequest;
-import org.apache.maven.shared.artifact.install.ArtifactInstallerException;
-import org.apache.maven.shared.project.NoFileAssignedException;
-import org.apache.maven.shared.project.install.ProjectInstaller;
-import org.apache.maven.shared.project.install.ProjectInstallerRequest;
+import org.apache.maven.shared.transfer.artifact.install.ArtifactInstallerException;
+import org.apache.maven.shared.transfer.project.NoFileAssignedException;
+import org.apache.maven.shared.transfer.project.install.ProjectInstaller;
+import org.apache.maven.shared.transfer.project.install.ProjectInstallerRequest;
 
 /**
  * Installs the project's main artifact, and any other artifacts attached by other plugins in the lifecycle, to the
@@ -101,7 +101,7 @@ public class InstallMojo
         {
             // CHECKSTYLE_OFF: LineLength
             ProjectInstallerRequest projectInstallerRequest =
-                new ProjectInstallerRequest().setProject( project ).setCreateChecksum( createChecksum ).setUpdateReleaseInfo( updateReleaseInfo );
+                new ProjectInstallerRequest().setProject( project );
             // CHECKSTYLE_ON: LineLength
 
             if ( !installAtEnd )
diff --git a/src/test/java/org/apache/maven/plugins/install/InstallFileMojoTest.java b/src/test/java/org/apache/maven/plugins/install/InstallFileMojoTest.java
index 5dc4fed..f915047 100644
--- a/src/test/java/org/apache/maven/plugins/install/InstallFileMojoTest.java
+++ b/src/test/java/org/apache/maven/plugins/install/InstallFileMojoTest.java
@@ -267,9 +267,9 @@ public class InstallFileMojoTest
 
         assignValuesForParameter( mojo );
 
-        boolean createChecksum = (Boolean) getVariableValueFromObject( mojo, "createChecksum" );
-
-        assertTrue( createChecksum );
+//        boolean createChecksum = (Boolean) getVariableValueFromObject( mojo, "createChecksum" );
+//
+//        assertTrue( createChecksum );
 
         mojo.execute();
 
@@ -279,18 +279,19 @@ public class InstallFileMojoTest
         File installedArtifact = new File( localPath + "." + "jar" );
         
         //get the actual checksum of the artifact
-        Map<String, Object> csums = ChecksumUtils.calc( file, Utils.CHECKSUM_ALGORITHMS );
-        for (Map.Entry<String, Object> csum : csums.entrySet()) {
-            Object actualSum = csum.getValue();
-            File sum = new File( localPath + ".jar." + csum.getKey().toLowerCase().replace( "-", "" ) );
-            assertTrue( sum.exists() );
-            String generatedSum = FileUtils.fileRead( sum, "UTF-8" );
-            assertEquals( actualSum, generatedSum );
-        }
+//        Map<String, Object> csums = ChecksumUtils.calc( file, Utils.CHECKSUM_ALGORITHMS );
+//        for (Map.Entry<String, Object> csum : csums.entrySet()) {
+//            Object actualSum = csum.getValue();
+//            File sum = new File( localPath + ".jar." + csum.getKey().toLowerCase().replace( "-", "" ) );
+//            assertTrue( sum.exists() );
+//            String generatedSum = FileUtils.fileRead( sum, "UTF-8" );
+//            assertEquals( actualSum, generatedSum );
+//        }
 
         assertTrue( installedArtifact.exists() );
         
-        assertEquals( 9, FileUtils.getFiles( new File( LOCAL_REPO ), null, null ).size() );
+//        assertEquals( 9, FileUtils.getFiles( new File( LOCAL_REPO ), null, null ).size() );
+        assertEquals( 5, FileUtils.getFiles( new File( LOCAL_REPO ), null, null ).size() );
     }
 
     private void assignValuesForParameter( Object obj )
diff --git a/src/test/java/org/apache/maven/plugins/install/InstallMojoTest.java b/src/test/java/org/apache/maven/plugins/install/InstallMojoTest.java
index 4d10d77..e211a18 100644
--- a/src/test/java/org/apache/maven/plugins/install/InstallMojoTest.java
+++ b/src/test/java/org/apache/maven/plugins/install/InstallMojoTest.java
@@ -25,24 +25,20 @@ import static org.mockito.Mockito.when;
 import java.io.File;
 import java.util.Collections;
 import java.util.List;
-import java.util.Map;
 
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.metadata.ArtifactMetadata;
 import org.apache.maven.execution.MavenSession;
 import org.apache.maven.plugin.MojoExecutionException;
 import org.apache.maven.plugin.testing.AbstractMojoTestCase;
-import org.apache.maven.plugins.install.AbstractInstallMojo;
-import org.apache.maven.plugins.install.InstallMojo;
 import org.apache.maven.plugins.install.stubs.AttachedArtifactStub0;
 import org.apache.maven.plugins.install.stubs.InstallArtifactStub;
 import org.apache.maven.project.DefaultProjectBuildingRequest;
 import org.apache.maven.project.MavenProject;
 import org.apache.maven.project.ProjectBuildingRequest;
-import org.apache.maven.shared.repository.RepositoryManager;
+import org.apache.maven.shared.transfer.repository.RepositoryManager;
 import org.apache.maven.shared.utils.io.FileUtils;
 import org.sonatype.aether.impl.internal.EnhancedLocalRepositoryManager;
-import org.sonatype.aether.util.ChecksumUtils;
 import org.sonatype.aether.util.DefaultRepositorySystemSession;
 
 /**
@@ -180,7 +176,7 @@ public class InstallMojoTest
 
         mojo.execute();
 
-        assertTrue( artifact.isRelease() );
+//        assertTrue( artifact.isRelease() );
         
         assertEquals( 5, FileUtils.getFiles( new File( LOCAL_REPO ), null, null ).size() );
     }
@@ -274,9 +270,9 @@ public class InstallMojoTest
 
         artifact = (InstallArtifactStub) project.getArtifact();
 
-        boolean createChecksum = (Boolean) getVariableValueFromObject( mojo, "createChecksum" );
-
-        assertTrue( createChecksum );
+//        boolean createChecksum = (Boolean) getVariableValueFromObject( mojo, "createChecksum" );
+//
+//        assertTrue( createChecksum );
 
         artifact.setFile( file );
 
@@ -307,30 +303,31 @@ public class InstallMojoTest
                         artifact.getVersion() + "/" + artifact.getArtifactId() + "-" + artifact.getVersion();
         
         //get the actual checksum of the pom
-        Map<String, Object> csums = ChecksumUtils.calc( pom, Utils.CHECKSUM_ALGORITHMS );
-        for (Map.Entry<String, Object> csum : csums.entrySet()) {
-            Object actualPomSum = csum.getValue();
-            File pomSum = new File( localPath + ".pom." + csum.getKey().toLowerCase().replace( "-", "" ) );
-            assertTrue( pomSum.exists() );
-            String generatedPomSum = FileUtils.fileRead( pomSum, "UTF-8" );
-            assertEquals( actualPomSum, generatedPomSum );
-        }
+//        Map<String, Object> csums = ChecksumUtils.calc( pom, Utils.CHECKSUM_ALGORITHMS );
+//        for (Map.Entry<String, Object> csum : csums.entrySet()) {
+//            Object actualPomSum = csum.getValue();
+//            File pomSum = new File( localPath + ".pom." + csum.getKey().toLowerCase().replace( "-", "" ) );
+//            assertTrue( pomSum.exists() );
+//            String generatedPomSum = FileUtils.fileRead( pomSum, "UTF-8" );
+//            assertEquals( actualPomSum, generatedPomSum );
+//        }
 
         //get the actual checksum of the artifact
-        csums = ChecksumUtils.calc( file, Utils.CHECKSUM_ALGORITHMS );
-        for (Map.Entry<String, Object> csum : csums.entrySet()) {
-            Object actualSum = csum.getValue();
-            File sum = new File( localPath + "." + packaging + "." + csum.getKey().toLowerCase().replace( "-", "" ) );
-            assertTrue( sum.exists() );
-            String generatedSum = FileUtils.fileRead( sum, "UTF-8" );
-            assertEquals( actualSum, generatedSum );
-        }
+//        csums = ChecksumUtils.calc( file, Utils.CHECKSUM_ALGORITHMS );
+//        for (Map.Entry<String, Object> csum : csums.entrySet()) {
+//            Object actualSum = csum.getValue();
+//            File sum = new File( localPath + "." + packaging + "." + csum.getKey().toLowerCase().replace( "-", "" ) );
+//            assertTrue( sum.exists() );
+//            String generatedSum = FileUtils.fileRead( sum, "UTF-8" );
+//            assertEquals( actualSum, generatedSum );
+//        }
 
         File installedArtifact = new File( localPath + "." + packaging );
 
         assertTrue( installedArtifact.exists() );
         
-        assertEquals( 9, FileUtils.getFiles( new File( LOCAL_REPO ), null, null ).size() );
+//        assertEquals( 9, FileUtils.getFiles( new File( LOCAL_REPO ), null, null ).size() );
+        assertEquals( 5, FileUtils.getFiles( new File( LOCAL_REPO ), null, null ).size() );
     }
 
     public void testSkip()
diff --git a/src/test/resources/unit/basic-install-checksum/plugin-config.xml b/src/test/resources/unit/basic-install-checksum/plugin-config.xml
index 56be1b6..6b4b56c 100644
--- a/src/test/resources/unit/basic-install-checksum/plugin-config.xml
+++ b/src/test/resources/unit/basic-install-checksum/plugin-config.xml
@@ -22,7 +22,6 @@ under the License.
       <plugin>
         <artifactId>maven-install-plugin</artifactId>
         <configuration>
-          <createChecksum>true</createChecksum>
           <project implementation="org.apache.maven.plugin.testing.stubs.MavenProjectStub">
             <file>${basedir}/src/test/resources/unit/basic-install-checksum/plugin-config.xml</file>
             <packaging>jar</packaging>
diff --git a/src/test/resources/unit/configured-install-test/plugin-config.xml b/src/test/resources/unit/configured-install-test/plugin-config.xml
index 88bc0cb..2cc19ac 100644
--- a/src/test/resources/unit/configured-install-test/plugin-config.xml
+++ b/src/test/resources/unit/configured-install-test/plugin-config.xml
@@ -22,7 +22,6 @@ under the License.
       <plugin>
         <artifactId>maven-install-plugin</artifactId>
         <configuration>
-          <updateReleaseInfo>true</updateReleaseInfo>
           <project implementation="org.apache.maven.plugin.testing.stubs.MavenProjectStub">
             <file>${basedir}/src/test/resources/unit/configured-install-test/plugin-config.xml</file>
             <packaging>jar</packaging>
diff --git a/src/test/resources/unit/install-file-with-checksum/plugin-config.xml b/src/test/resources/unit/install-file-with-checksum/plugin-config.xml
index 384b003..91655ef 100644
--- a/src/test/resources/unit/install-file-with-checksum/plugin-config.xml
+++ b/src/test/resources/unit/install-file-with-checksum/plugin-config.xml
@@ -27,7 +27,6 @@ under the License.
           <version>1.0-SNAPSHOT</version>
           <packaging>jar</packaging>
           <file>${basedir}/src/test/resources/unit/install-file-with-checksum/maven-test-jar.jar</file>
-          <createChecksum>true</createChecksum>
           <generatePom>true</generatePom>
         </configuration>
       </plugin>