Skip to content

fix(idempotent): Correctly raise IdempotencyKeyError #378

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Apr 1, 2021

Conversation

michaelbrewer
Copy link
Contributor

@michaelbrewer michaelbrewer commented Apr 1, 2021

Issue #, if available:

Description of changes:

Idempotent handler is currently returning IdempotencyPersistenceLayerError instead of the IdempotencyKeyError

Checklist

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@codecov-io
Copy link

codecov-io commented Apr 1, 2021

Codecov Report

Merging #378 (0ee184f) into develop (a2c82b5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #378   +/-   ##
========================================
  Coverage    99.94%   99.94%           
========================================
  Files           98       98           
  Lines         3686     3688    +2     
  Branches       173      174    +1     
========================================
+ Hits          3684     3686    +2     
  Partials         2        2           
Impacted Files Coverage Δ
...da_powertools/utilities/idempotency/idempotency.py 98.57% <100.00%> (+0.04%) ⬆️

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 a2c82b5...0ee184f. Read the comment docs.

@michaelbrewer
Copy link
Contributor Author

@heitorlessa we would need to fix this one

@heitorlessa heitorlessa added this to the 1.14.0 milestone Apr 1, 2021
@heitorlessa heitorlessa added area/utilities bug Something isn't working labels Apr 1, 2021
Copy link
Contributor

@heitorlessa heitorlessa left a comment

Choose a reason for hiding this comment

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

Thank you <3

@heitorlessa heitorlessa merged commit b558b18 into aws-powertools:develop Apr 1, 2021
@michaelbrewer michaelbrewer deleted the fix-idempotent-key-error branch April 1, 2021 17:54
heitorlessa referenced this pull request in heitorlessa/aws-lambda-powertools-python Apr 4, 2021
* develop:
  fix(idempotent): Correctly raise IdempotencyKeyError (#378)
  feat(event-handler): Add AppSync handler decorator (#363)
  feat(parameter): add dynamodb_endpoint_url for local_testing (#376)
  fix(parser): S3Model support empty keys (#375)
  fix(data-classes): Add missing operationName (#373)
  fix: perf tests for Logger and fail str msgs
  feat(parser): Add S3 Object Lambda Event (#362)
  build(pre-commit): Add pre-commit to make pr (#368)
  fix(tracer): Correct type hint for MyPy (#365)
  fix(metrics): AttributeError raised by MediaManager and Typing and docs (#357)

Signed-off-by: heitorlessa <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants