Add a link to the documentation on how to enable Appengine APIs in th…

…e error message. PiperOrigin-RevId: 651000411 Change-Id: I07205a9ad13794a724a0d42137037f42108a92db
GoogleCloudPlatform · Jul 10, 2024 · 2145c33 · 2145c33
1 parent a4a3b3c
commit 2145c33
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/runtime/impl/src/main/java/com/google/apphosting/utils/runtime/ApiProxyUtils.java b/runtime/impl/src/main/java/com/google/apphosting/utils/runtime/ApiProxyUtils.java
@@ -78,7 +78,11 @@ public static ApiProxyException convertApiError(APIResponse apiResponse,
         return new ApiProxy.ArgumentException(packageName, methodName);
       case FEATURE_DISABLED:
         return new ApiProxy.FeatureNotEnabledException(
-            "%s.%s " + apiResponse.getErrorMessage(), packageName, methodName);
+            "%s.%s Please, enable the Appengine APIs via "
+                + "https://cloud.google.com/appengine/docs/standard/java-gen2/services/access "
+                + apiResponse.getErrorMessage(),
+            packageName,
+            methodName);
       case RPC_ERROR:
         return convertApiResponseRpcErrorToException(
             apiResponse.getRpcError(),

diff --git a/runtime/impl/src/test/java/com/google/apphosting/runtime/ApiProxyImplTest.java b/runtime/impl/src/test/java/com/google/apphosting/runtime/ApiProxyImplTest.java
@@ -957,7 +957,10 @@ public void testFeatureNotEnabledError() {
     assertStackTraceIsCorrect(e);
     assertThat(e)
         .hasMessageThat()
-        .isEqualTo("generate.feature.disabled.error. You need to turn on billing!");
+        .isEqualTo(
+            "generate.feature.disabled.error. Please, enable the Appengine APIs via"
+                + " https://cloud.google.com/appengine/docs/standard/java-gen2/services/access You"
+                + " need to turn on billing!");
   }
 
   @Test