Skip to content

Adding test vector generation example#445

Merged
kevinlewi merged 1 commit intofacebook:mainfrom
kevinlewi:test_vectors
Jul 18, 2024
Merged

Adding test vector generation example#445
kevinlewi merged 1 commit intofacebook:mainfrom
kevinlewi:test_vectors

Conversation

@kevinlewi
Copy link
Contributor

Adding a new example (under examples/) for producing the hex-encoded byte representations of lookup and key history proofs.

This will allow other implementations to compare against and exercise protobuf deserialization / parsing in the event that there is another client-side implementation which can consume lookup and history proofs for verification.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 16, 2024
@codecov-commenter
Copy link

codecov-commenter commented Jul 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.88%. Comparing base (3ce5335) to head (64bc708).
Report is 24 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #445      +/-   ##
==========================================
- Coverage   88.61%   87.88%   -0.73%     
==========================================
  Files          39       38       -1     
  Lines        9109     8165     -944     
==========================================
- Hits         8072     7176     -896     
+ Misses       1037      989      -48     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kevinlewi kevinlewi marked this pull request as ready for review July 16, 2024 07:33
@kevinlewi kevinlewi merged commit 16cb19d into facebook:main Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants