fix(aws-applicationautoscaling): StepScalingPolicy missing ScalingTargetId and AdjustmentType

[piradeepk]

Fix StepScalingAction and StepScalingPolicy to add ScalingTargetId and default AdjustmentType to CloudFormation template.

---

Pull Request Checklist

• Testing
  • Unit test added (prefer not to modify an existing test, otherwise, it's probably a breaking change)
  • CLI change?: coordinate update of integration tests with team
  • cdk-init template change?: coordinated update of integration tests with team
• Docs
  • jsdocs: All public APIs documented
  • README: README and/or documentation topic updated
  • Design: For significant features, design document added to design folder
• Title and Description
  • Change type: title prefixed with fix, feat and module name in parens, which will appear in changelog
  • Title: use lower-case and doesn't end with a period
  • Breaking?: last paragraph: "BREAKING CHANGE: <describe what changed + link for details>"
  • Issues: Indicate issues fixed via: "Fixes #xxx" or "Closes #xxx"
• Sensitive Modules (requires 2 PR approvers)
  • IAM Policy Document (in @aws-cdk/aws-iam)
  • EC2 Security Groups and ACLs (in @aws-cdk/aws-ec2)
  • Grant APIs (only if not based on official documentation with a reference)

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

[SoManyHs]

LGTM!

[piradeepk]

Added comment regarding why I've added the properties. Cloudformation requires either the ResourceId, ScalableDimension, and ServiceNamespace properties, or the ScalingTargetId property, but not both.

[SoManyHs]

Ah -- I meant to add the context in the commit message, not in the code itself. Given that we may want to add those other three fields in the future, I figured it made more sense to give context in the commit only.