Improve error message

smithy-lang · Jan 12, 2021 · 9c1f284 · 9c1f284
1 parent 1084971
commit 9c1f284
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/...-openapi/src/main/java/software/amazon/smithy/aws/apigateway/openapi/AddIntegrations.java b/...-openapi/src/main/java/software/amazon/smithy/aws/apigateway/openapi/AddIntegrations.java
@@ -132,8 +132,8 @@ private static void validateTraitConfiguration(IntegrationTrait trait,
         ApiGatewayConfig.ApiType apiType = context.getConfig().getExtensions(ApiGatewayConfig.class)
                 .getApiGatewayType();
         if (!trait.getPayloadFormatVersion().isPresent() && apiType.equals(ApiGatewayConfig.ApiType.HTTP)) {
-            throw new OpenApiException("When using the HTTP apiGatewayType, a payloadFormatVersion must be set on the"
-                    + " integration applied to the operation: " + operation.getId());
+            throw new OpenApiException("When the 'apiGatewayType' OpenAPI conversion setting is 'HTTP', a "
+                    + "'payloadFormatVersion' must be set on the aws.apigateway#integration trait.");
         }
     }
 

diff --git a/...napi/src/test/java/software/amazon/smithy/aws/apigateway/openapi/AddIntegrationsTest.java b/...napi/src/test/java/software/amazon/smithy/aws/apigateway/openapi/AddIntegrationsTest.java
@@ -78,7 +78,7 @@ public void throwsOnInvalidIntegrationTraitForHttpApi() {
             OpenApiConverter.create().config(config).convertToNode(model);
         });
 
-        assertThat(thrown.getMessage(), containsString("a payloadFormatVersion must be set on the integration"
-                    + " applied to the operation:"));
+        assertThat(thrown.getMessage(), containsString("When the 'apiGatewayType' OpenAPI conversion setting is"
+                + " 'HTTP', a 'payloadFormatVersion' must be set on the aws.apigateway#integration trait."));
     }
 }