[MSHARED-637] Remove IOException from ProjectDeployer cause it's not thrown by the code

khmarbaise · khmarbaise · commit f45e7ad855ac · 2017-05-07T13:24:25.000Z
o Fixed duplicated license header ProjectDeployer. o Changed the deploy() method and removed IOException and added IllegalArgumentException git-svn-id: https://svn.apache.org/repos/asf/maven/shared/trunk@1794196 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/deploy/ProjectDeployer.java b/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/deploy/ProjectDeployer.java
@@ -1,26 +1,5 @@
 package org.apache.maven.shared.project.deploy;
 
-/*
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements.  See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership.  The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License.  You may obtain a copy of the License at
- *
- *  http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-
-import java.io.IOException;
-
 /*
  * Licensed to the Apache Software Foundation (ASF) under one
  * or more contributor license agreements.  See the NOTICE file
@@ -75,12 +54,12 @@ public interface ProjectDeployer
      * @param buildingRequest {@link ProjectBuildingRequest}
      * @param request {@link ProjectDeployerRequest}
      * @param artifactRepository {@link ArtifactRepository}
-     * @throws IOException In case of problems related to checksums.
      * @throws NoFileAssignedException In case of missing file which has not been assigned to project.
+     * @throws IllegalArgumentException in case of artifact is not correctly assigned.
      */
-    //FIXME: Need to check to remove IOException cause it looks like it's not being thrown by code.
     void deploy( ProjectBuildingRequest buildingRequest, ProjectDeployerRequest request,
                         ArtifactRepository artifactRepository )
-        throws IOException, NoFileAssignedException;
+        throws NoFileAssignedException, IllegalArgumentException;
+
 
 }
diff --git a/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/deploy/internal/DefaultProjectDeployer.java b/maven-artifact-transfer/src/main/java/org/apache/maven/shared/project/deploy/internal/DefaultProjectDeployer.java
@@ -20,7 +20,6 @@
  */
 
 import java.io.File;
-import java.io.IOException;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.List;
@@ -66,7 +65,7 @@ public class DefaultProjectDeployer
      */
     public void deploy( ProjectBuildingRequest buildingRequest, ProjectDeployerRequest request,
                         ArtifactRepository artifactRepository )
-        throws IOException, NoFileAssignedException
+        throws NoFileAssignedException, IllegalArgumentException
     {
 
         Artifact artifact = request.getProject().getArtifact();
