Skip to content
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

Murisi/test vectors 0.30.2 rev 1 #2588

Merged
merged 11 commits into from
Feb 27, 2024
Merged

Conversation

murisi
Copy link
Collaborator

@murisi murisi commented Feb 9, 2024

Describe your changes

Various changes in order to increase the detail presented in test vectors, increase the variety of test vectors, and ease the implementation of hardware wallet apps. Specifically the changes are as follows:

  • Test vector generator now sometimes generates signed transactions
  • Greatly increased the level of detail presented in Init Proposal transactions
  • Increased the coverage PGF steward transactions to include the IBC variants
  • Started including memos in generated transactions sometimes, both in hashed and full form
  • Now print out the section hashes in debug vectors to facilitate constructing signing test vectors
  • Improved the formatting of Update Steward Commission transactions for the hardware wallet

Indicate on which release or other PRs this topic is based on

Namada v0.30.2

Checklist before merging to draft

  • I have added a changelog
  • Git history is in acceptable state

@murisi murisi requested a review from karbyshev February 9, 2024 16:55
@murisi murisi marked this pull request as ready for review February 9, 2024 16:55
@murisi murisi requested a review from sug0 February 9, 2024 16:56
Copy link
Collaborator

@sug0 sug0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a couple of comments

crates/governance/src/storage/proposal.rs Show resolved Hide resolved
crates/sdk/src/lib.rs Outdated Show resolved Hide resolved
crates/sdk/src/lib.rs Outdated Show resolved Hide resolved
crates/sdk/src/lib.rs Outdated Show resolved Hide resolved
crates/sdk/src/signing.rs Show resolved Hide resolved
tzemanovic added a commit that referenced this pull request Feb 27, 2024
* origin/murisi/test-vectors-0.31.6:
  Limit commitment sizes.
  Added a changelog entry.
  Deduplicated the arbitrary memo generation code.
  Test vector generator now produces arbitrary signed transactions.
  Now generate arbitrary PGF IBC targets in the test vectors.
  Order the test vvectors for init proposal and use the NAM prefix.
  Give more detailed test vectors for Init Proposal transactions.
  Start generating arbitrary memos and print them out in the test vectors.
  Print out the section hashes in the debug vectors.
  Separated the validators from the commission rates in order to ease hardware wallet implementations.
  Make sure that generated metadata is always non-empty. Stop printing (none) for empty values. Simplifies hardware wallets.
@tzemanovic tzemanovic merged commit 9e931a6 into main Feb 27, 2024
12 of 15 checks passed
@tzemanovic tzemanovic deleted the murisi/test-vectors-0.30.2-rev-1 branch February 27, 2024 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants