Bug fix: Fix DisableExecuteApiEndpoint bug fix

[aaythapa]

Issue #, if available

2396

Description of changes

The DisableExecuteApiEndpoint property in Serverless API didn't work if the OpenApiVersion was 3 because, when transformed, it didn't add the x-amazon-apigateway-endpoint-configuration extension to the Servers object as stated in this guide. Now it adds the extension to the Servers object.

Description of how you validated changes

Checklist

• Add/update unit tests using:
  • Correct values
  • Bad/wrong values (None, empty, wrong type, length, etc.)
  • Intrinsic Functions
• Add/update integration tests
• Update documentation
• Verify transformed template deploys and application functions as expected
• Do these changes include any template validations?
  • Did the newly validated properties support intrinsics prior to adding the validations? (If unsure, please review Intrinsic Functions before proceeding).
    • Does the pull request ensure that intrinsics remain functional with the new validations?

Examples?

Please reach out in the comments, if you want to add an example. Examples will be
added to sam init through https://github.com/awslabs/aws-sam-cli-app-templates/

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

[GavinZZ]

Out of curiosity, how did you generate the transform test input and output files? Did you try using the script add_transform_test.py?

[aaythapa]

Out of curiosity, how did you generate the transform test input and output files? Did you try using the script add_transform_test.py?

Didn't use the script unfortunately

[GavinZZ]

Out of curiosity, how did you generate the transform test input and output files? Did you try using the script add_transform_test.py?

Didn't use the script unfortunately

That's alright. FYI, you can use the script for easier file generation. It will generate the 3 files each for one partition, add to the correct directories and add REGIONAL endpoint configuration for you.

[GavinZZ]

Other than the changes Chris mentioned, rest LGTM