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() ) )