Skip to content

Conversation

ShreyaGangishetty
Copy link

Issue #, if available:
#1141, #248, #547

Description of changes:
added support for usage plans

Description of how you validated changes:
added test input templates and also deployed the generated templates in CloudFormation and validated
Checklist:

  • Write/update tests
  • make pr passes
  • Update documentation
  • Verify transformed template deploys and application functions as expected
  • Add/update example to examples/2016-10-31

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

@ShreyaGangishetty ShreyaGangishetty changed the title Usage plans branch feat: usage plans support for Api Auth Oct 7, 2019
@codecov-io
Copy link

codecov-io commented Oct 7, 2019

Codecov Report

Merging #1179 into develop will decrease coverage by 0.02%.
The diff coverage is 93.39%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1179      +/-   ##
===========================================
- Coverage    94.44%   94.42%   -0.03%     
===========================================
  Files           78       78              
  Lines         4557     4660     +103     
  Branches       911      929      +18     
===========================================
+ Hits          4304     4400      +96     
- Misses         119      121       +2     
- Partials       134      139       +5
Impacted Files Coverage Δ
samtranslator/model/sam_resources.py 94.05% <100%> (+0.03%) ⬆️
samtranslator/model/apigateway.py 97.1% <81.81%> (-0.86%) ⬇️
samtranslator/model/api/api_generator.py 94.98% <94.5%> (-0.15%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update faac69b...d3ed4e0. Read the comment docs.

StageName: Prod
Auth:
UsagePlan:
CreateUsagePlan: SHARED
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this not create a UsagePlan now?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it creates a usage plan with logical id ServerlessUsagePlan

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.

4 participants