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 2020/06/20 08:40:29 UTC

[maven] 22/30: Review comment: Removed Guava's @VisibleForTesting

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

rfscholte pushed a commit to branch MNG-5760
in repository https://gitbox.apache.org/repos/asf/maven.git

commit 38a5abea0eaaf1121fc4736f2d250a5bf9125468
Author: Martin Kanters <Ma...@infosupport.com>
AuthorDate: Fri Jun 5 20:59:56 2020 +0200

    Review comment: Removed Guava's @VisibleForTesting
---
 .../main/java/org/apache/maven/execution/DefaultBuildResumer.java    | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/maven-core/src/main/java/org/apache/maven/execution/DefaultBuildResumer.java b/maven-core/src/main/java/org/apache/maven/execution/DefaultBuildResumer.java
index 7232365..0e0e165 100644
--- a/maven-core/src/main/java/org/apache/maven/execution/DefaultBuildResumer.java
+++ b/maven-core/src/main/java/org/apache/maven/execution/DefaultBuildResumer.java
@@ -19,7 +19,6 @@ package org.apache.maven.execution;
  * under the License.
  */
 
-import com.google.common.annotations.VisibleForTesting;
 import org.apache.commons.lang3.StringUtils;
 import org.apache.maven.lifecycle.LifecycleExecutionException;
 import org.apache.maven.model.Dependency;
@@ -108,7 +107,7 @@ public class DefaultBuildResumer implements BuildResumer
         return ":" + failedProject.getArtifactId();
     }
 
-    @VisibleForTesting
+    // This method is made package-private for testing purposes
     Properties determineResumptionProperties( MavenExecutionResult result )
     {
         Properties properties = new Properties();
@@ -246,7 +245,7 @@ public class DefaultBuildResumer implements BuildResumer
         return properties;
     }
 
-    @VisibleForTesting
+    // This method is made package-private for testing purposes
     void applyResumptionProperties( MavenExecutionRequest request, Properties properties )
     {
         if ( properties.containsKey( RESUME_FROM_PROPERTY ) && StringUtils.isEmpty( request.getResumeFrom() ) )