-
Notifications
You must be signed in to change notification settings - Fork 919
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
feat(primitives): transaction encoding tests #103
Conversation
1a9a6dc
to
cbc3541
Compare
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.
one leftover, I think.
nice, needs rebase once blocking PR merged
// let recovered = tx.recover().unwrap(); | ||
// let expected: Address = "0xa12e1462d0ced572f396f58b6e2d03894cd7c8a4".parse().unwrap(); | ||
// assert_eq!(expected, recovered); |
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.
needed?
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.
we should uncomment this test once recovery is implemented. should that be in this PR or should we add an issue for it / do later?
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.
I see
should we add an issue for it
sgtm!
cbc3541
to
8437413
Compare
Adds tests on top of #102 for signed transaction encoding and decoding
Changes
Signature
to useodd_y_parity: bool
instead of au8