Closed ugoamanoh closed 4 years ago
Merging #35 into develop will increase coverage by
0.26%
. The diff coverage is78.57%
.
@@ Coverage Diff @@
## develop #35 +/- ##
=============================================
+ Coverage 73.86% 74.13% +0.26%
- Complexity 186 199 +13
=============================================
Files 27 31 +4
Lines 922 978 +56
Branches 145 156 +11
=============================================
+ Hits 681 725 +44
- Misses 155 158 +3
- Partials 86 95 +9
Impacted Files | Coverage Δ | Complexity Δ | |
---|---|---|---|
.../java/me/uport/credential_status/StatusResolver.kt | 100% <100%> (ø) |
0 <0> (?) |
|
...uport/credential_status/UniversalStatusResolver.kt | 58.33% <58.33%> (ø) |
4 <4> (?) |
|
.../src/main/java/me/uport/credential_status/Utils.kt | 66.66% <66.66%> (ø) |
0 <0> (?) |
|
...ava/me/uport/sdk/ethr_status/EthrStatusResolver.kt | 86.48% <86.48%> (ø) |
9 <9> (?) |
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 8837485...5da0f64. Read the comment docs.
@mirceanis Kindly review
What's Here
To support revocations, we have added an interface
StatusResolver
in thecredential-status
module which can be implemented to enable checking the revocation status of a credential.This PR also contains an Ethr implementation of the
StatusResolver
calledEthrStatusResolver
in theethr-status
module which returns anEthrStatus
, an extension of theCredentialStatus
A UniversalStatusResolver which enables registration of several implementations of the StatusResolver and at runtime, delegates of which Resolver will be used to check the status of a credential.
Tests to cover operations in
EthrStatusResolver
andUniversalStatusResolver
. Use the command./gradlew test cC --no-parallel
to run the entire test suite