Skip to content

Conversation

@dev-warrior777
Copy link
Contributor

This PR updates the ZEC regtest harness to support NU6.1, enabling testing of the DEX wallet integration with the latest Zcash protocol.

Key Changes

  • NU6.1 activation: Added nuparams=c8e71055:5 and nuparams=4dec4df0:7 (ZIP 253/255)
  • Transparent addresses: Switched from z_getaddressforaccountgetnewaddress (deprecated in v6+)
  • Regtest compatibility: Added -allowdeprecated=getnewaddress
  • Robustness: fixed sendtoaddress spacing
  • All nodes activated: NU6.1 params now on alpha, beta, delta, gamma

Why

  • zcashd v6.10.0 defaults to unified addresses and account 0 behavior
  • DEX harness uses transparent UTXOs → must use getnewaddress
  • Without these, go test fails with "account not found" or empty UTXOs

Tested

./harness.sh
./beta getbalance   # → 74.00000000

@dev-warrior777
Copy link
Contributor Author

dev-warrior777 commented Nov 2, 2025

Notes:

Current NU6.1 wallets are recreated on each run

  • need to export to dex/testing/zec
  • uncomment wallet import logic in the script

Test: go test -v -tags=harness -run=TestWallet ./client/asset/zec - still fails

  • likely needs a regtest specific switch (avoid using common code)

Same upgrades to daemon startup applied to loadbot issue #3378

  • untested

NU5 -> NU6.1 - More research needed:

  • Account 0 behavior has changed
  • Unified address APIs are default
  • many RPC's are removed and/or deprecated

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.

1 participant