fix(ecs-patterns): minHealthyPercent and maxHealthyPercent props validation

packages/aws-cdk-lib/aws-ecs-patterns/lib/fargate/application-load-balanced-fargate-service.ts

@@ -96,6 +96,9 @@ export class ApplicationLoadBalancedFargateService extends ApplicationLoadBalanc
       throw new Error('You must specify one of: taskDefinition or image');
     }
 
+    this.validateHealthyPercentage('minHealthyPercent', props.minHealthyPercent);
+    this.validateHealthyPercentage('maxHealthyPercent', props.maxHealthyPercent);
+
     const desiredCount = FeatureFlags.of(this).isEnabled(cxapi.ECS_REMOVE_DEFAULT_DESIRED_COUNT) ? this.internalDesiredCount : this.desiredCount;
 
     this.service = new FargateService(this, 'Service', {
@@ -120,4 +123,14 @@ export class ApplicationLoadBalancedFargateService extends ApplicationLoadBalanc
     });
     this.addServiceAsTarget(this.service);
   }
+
+  /**
+   * Throws an error if the specified percent is not an integer or negative.
+   */
+  private validateHealthyPercentage(name: string, value?: number) {
+    if (value === undefined) { return; }
+    if (!Number.isInteger(value) || value < 0) {
+      throw new Error(`${name}: Must be a non-negative integer; received ${value}`);
+    }
+  }
 }

packages/aws-cdk-lib/aws-ecs-patterns/test/fargate/load-balanced-fargate-service.test.ts

@@ -1208,3 +1208,37 @@ test('NetworkLoadBalancedFargateService multiple capacity provider strategies ar
     ]),
   });
 });
+
+test('should validate minHealthyPercent', () => {
+  // GIVEN
+  const stack = new cdk.Stack();
+  const vpc = new ec2.Vpc(stack, 'VPC');
+  const cluster = new ecs.Cluster(stack, 'Cluster', { vpc });
+
+  // WHEN
+  expect(() => new ecsPatterns.ApplicationLoadBalancedFargateService(stack, 'Service', {
+    cluster,
+    taskImageOptions: {
+      image: ecs.ContainerImage.fromRegistry('/aws/aws-example-app'),
+      dockerLabels: { label1: 'labelValue1', label2: 'labelValue2' },
+    },
+    minHealthyPercent: 0.5,
+  })).toThrow(/Must be a non-negative integer/);
+});
+
+test('should validate maxHealthyPercent', () => {
+  // GIVEN
+  const stack = new cdk.Stack();
+  const vpc = new ec2.Vpc(stack, 'VPC');
+  const cluster = new ecs.Cluster(stack, 'Cluster', { vpc });
+
+  // WHEN
+  expect(() => new ecsPatterns.ApplicationLoadBalancedFargateService(stack, 'Service', {
+    cluster,
+    taskImageOptions: {
+      image: ecs.ContainerImage.fromRegistry('/aws/aws-example-app'),
+      dockerLabels: { label1: 'labelValue1', label2: 'labelValue2' },
+    },
+    maxHealthyPercent: 0.5,
+  })).toThrow(/Must be a non-negative integer/);
+});