-
Notifications
You must be signed in to change notification settings - Fork 421
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
fix(idempotent): Correctly raise IdempotencyKeyError #378
Conversation
Codecov Report
@@ 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
Continue to review full report at Codecov.
|
@heitorlessa we would need to fix this one |
Co-authored-by: Heitor Lessa <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you <3
* 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]>
Issue #, if available:
Description of changes:
Idempotent handler is currently returning
IdempotencyPersistenceLayerError
instead of theIdempotencyKeyError
Checklist
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.