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

Testnet NU5 transition period has wrong blocks in test vectors #9090

Closed
upbqdn opened this issue Dec 17, 2024 · 1 comment · Fixed by #9098
Closed

Testnet NU5 transition period has wrong blocks in test vectors #9090

upbqdn opened this issue Dec 17, 2024 · 1 comment · Fixed by #9098
Assignees
Labels
C-bug Category: This is a bug C-testing Category: These are tests NU-5 Network Upgrade: NU5 specific tasks

Comments

@upbqdn
Copy link
Member

upbqdn commented Dec 17, 2024

// Nu5 transition
// for i in 1599199 1599200 1599201; do
// zcash-cli -testnet getblock $i 0 > block-test-$[i/1000000]-$[i/1000%1000]-$[i%1000].txt
// done
pub static ref BLOCK_TESTNET_1599199_BYTES: Vec<u8> =
<Vec<u8>>::from_hex(include_str!("block-test-1-599-199.txt").trim())
.expect("Block bytes are in valid hex representation");
pub static ref SAPLING_FINAL_ROOT_TESTNET_1599199_BYTES: [u8; 32] =
<[u8; 32]>::from_hex("4de75d10def701ad22ecc17517a3adc8789ea8c214ac5bfc917b8924377e6c89")
.expect("final root bytes are in valid hex representation").rev();
should contain blocks for heights from 1_842_419 to 1_842_421 since NU5 activated at height 1_842_420 on Testnet.

@github-project-automation github-project-automation bot moved this to New in Zebra Dec 17, 2024
@upbqdn upbqdn added C-bug Category: This is a bug NU-5 Network Upgrade: NU5 specific tasks C-testing Category: These are tests labels Dec 17, 2024
@upbqdn upbqdn self-assigned this Dec 17, 2024
@mpguerra mpguerra linked a pull request Jan 13, 2025 that will close this issue
7 tasks
@mpguerra mpguerra moved this from New to Sprint Backlog in Zebra Jan 13, 2025
@mpguerra mpguerra moved this from Sprint Backlog to Review/QA in Zebra Jan 13, 2025
@mergify mergify bot closed this as completed in #9098 Jan 15, 2025
@github-project-automation github-project-automation bot moved this from Review/QA to Done in Zebra Jan 15, 2025
@upbqdn
Copy link
Member Author

upbqdn commented Jan 15, 2025

More context: #9098 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug C-testing Category: These are tests NU-5 Network Upgrade: NU5 specific tasks
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant