fix(cfn-include): allowedValues aren't included when specified by a p…

…arameter (#9532) ---- Closes #9531 This fix allows allowedValues in parameters to be used in cfn-include. *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
aws · Aug 7, 2020 · e7dc82f · e7dc82f
1 parent e146c23
commit e7dc82f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/packages/@aws-cdk/cloudformation-include/lib/cfn-include.ts b/packages/@aws-cdk/cloudformation-include/lib/cfn-include.ts
@@ -225,6 +225,7 @@ export class CfnInclude extends core.CfnElement {
       type: expression.Type,
       default: expression.Default,
       allowedPattern: expression.AllowedPattern,
+      allowedValues: expression.AllowedValues,
       constraintDescription: expression.ConstraintDescription,
       description: expression.Description,
       maxLength: expression.MaxLength,

diff --git a/packages/@aws-cdk/cloudformation-include/test/test-templates/bucket-with-parameters.json b/packages/@aws-cdk/cloudformation-include/test/test-templates/bucket-with-parameters.json
@@ -15,6 +15,7 @@
       "Description": "the time in seconds that a browser will cache the preflight response",
       "MaxValue": "300",
       "MinValue": "0",
+      "AllowedValues": [1, 2, 3, 10, 100, 300],
       "Type": "Number",
       "NoEcho": "true"
     }