Skip to content

Conversation

@wchengru
Copy link

@wchengru wchengru commented Sep 11, 2021

Issue #, if available:
This pr is to add the integration tests to aws#2458
It validate the following cases:

  • For swaggers in restApi, the operationId is successfully added to requestContext
  • For swaggers in httpApi, the operationId is not in requestContext
  • For AWS::ApiGateway::Method, the operationName field is successfully added to requestContext
  • For AWS::ApiGatewayV2::Route, the operationName field is NOT added to requestContext

Also, it resolved the dedupe_function_routes.

Why is this change necessary?

How does it address the issue?

What side effects does this change have?

Did you change a dependency in requirements/base.txt?
If so, did you run make update-reproducible-reqs

Checklist:

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

@ramosbugs ramosbugs merged commit b2e7c99 into ramosbugs:develop-operation-name Sep 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants