From b1ef73194a1d8b483ddcd5e69d52f07e6da764c5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jo=C3=A3o=20Andr=C3=A9=20Martins?= Date: Thu, 14 Jul 2016 09:58:26 -0400 Subject: [PATCH] Removes VisibleForTesting and moves comment. --- .../google/cloud/tools/appengine/cloudsdk/CloudSdk.java | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/main/java/com/google/cloud/tools/appengine/cloudsdk/CloudSdk.java b/src/main/java/com/google/cloud/tools/appengine/cloudsdk/CloudSdk.java index 55a00e5a1..6418f1f70 100644 --- a/src/main/java/com/google/cloud/tools/appengine/cloudsdk/CloudSdk.java +++ b/src/main/java/com/google/cloud/tools/appengine/cloudsdk/CloudSdk.java @@ -25,7 +25,6 @@ import com.google.cloud.tools.appengine.cloudsdk.process.ProcessExitListener; import com.google.cloud.tools.appengine.cloudsdk.process.ProcessOutputLineListener; import com.google.cloud.tools.appengine.cloudsdk.process.ProcessStartListener; -import com.google.common.annotations.VisibleForTesting; import com.google.common.base.Joiner; import com.google.common.collect.Maps; @@ -175,7 +174,7 @@ private void logCommand(List command) { logger.info("submitting command: " + WHITESPACE_JOINER.join(command)); } - @VisibleForTesting public Path getSdkPath() { + public Path getSdkPath() { return sdkPath; } @@ -250,8 +249,7 @@ public static class Builder { private int runDevAppServerWaitSeconds; /** - * The home directory of Google Cloud SDK. If not set, will attempt to look for the SDK in known - * install locations. + * The home directory of Google Cloud SDK. */ public Builder sdkPath(Path sdkPathFile) { if (sdkPathFile != null) { @@ -350,6 +348,9 @@ public Builder runDevAppServerWait(int runDevAppServerWaitSeconds) { /** * Create a new instance of {@link CloudSdk}. + * + * If {@code sdKPath} is not set, this method will attempt to look for the SDK in known install + * locations. */ public CloudSdk build() {