contentauth / c2pa-rs

Rust SDK for the core C2PA (Coalition for Content Provenance and Authenticity) specification
Other
128 stars 57 forks source link

chore: Move time stamp code into c2pa-crypto #696

Open scouten-adobe opened 1 week ago

codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 78.50000% with 86 lines in your changes missing coverage. Please review.

Project coverage is 81.50%. Comparing base (2df2f1d) to head (7469330). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
internal/crypto/src/time_stamp/verify.rs 72.68% 53 Missing :warning:
internal/crypto/src/time_stamp/http_request.rs 79.48% 16 Missing :warning:
internal/crypto/src/time_stamp/response.rs 81.81% 8 Missing :warning:
internal/crypto/src/time_stamp/error.rs 12.50% 7 Missing :warning:
internal/crypto/src/time_stamp/provider.rs 97.36% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #696 +/- ## ========================================== - Coverage 81.52% 81.50% -0.03% ========================================== Files 104 109 +5 Lines 30614 31014 +400 ========================================== + Hits 24958 25277 +319 - Misses 5656 5737 +81 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


🚨 Try these New Features:

scouten-adobe commented 1 week ago

@mauricefisher64 @mikeklein13 do we have any sample time stamp data that we could use to achieve independent code coverage for this?

scouten-adobe commented 1 week ago

Per @mauricefisher64, look at the claim_v2 branch to catch up on some recent changes to time stamp work.