-
Notifications
You must be signed in to change notification settings - Fork 83
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
Improve mappings from vdrtools, refactor tests in libvcx #721
Conversation
Codecov Report
@@ Coverage Diff @@
## main #721 +/- ##
==========================================
+ Coverage 64.30% 64.39% +0.09%
==========================================
Files 245 244 -1
Lines 22911 22828 -83
Branches 5072 5070 -2
==========================================
- Hits 14732 14701 -31
+ Misses 4035 4003 -32
+ Partials 4144 4124 -20
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
lgtm, just the one comment
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
Signed-off-by: Patrik Stas <patrik.stas@absa.africa>
2694193
to
a7d7031
Compare
Changes
IndyErrorKind -> AriesVcxErrorKind
to more sensible mappingLedgerItemNotFound
err kind to ariesvcx, libvcxObjectAccessError
err kind to libvcxkey
is used to open wallet, throwLibvcxErrorKind::WalletAccessFailed
err kind instead of "unmapped" libindy errorLibvcxErrorKind::LibndyError(207)
. Added test to libvcx to cover this caseRefactoring
api_c
layer toapi_vcx
Signed-off-by: Patrik Stas patrik.stas@absa.africa