-
Notifications
You must be signed in to change notification settings - Fork 771
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
client: snap protocol tests #2119
Conversation
Codecov Report
Flags with carried forward coverage won't be shown. Click here to find out more. |
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.
@scorbajio i would love to see some logs of the other requests, and then encoding/decoding compared against some actual sample. Right now using our conver encoder => decoder sort of is self reinforcing, so would be awesome to test them against some actual sample data received from a geth/nethermind or other snap capable peer.
I'm not sure I understand what sample data you're referring to for testing encoding/decoding. Could you explain or give an example? |
ok, i will add the same on top of these tests, for now we can merge these ones |
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! will add more tests on top regarding encoding decoding with real data
This PR adds tests for
snapprotocol
andsnapsync
.