KomodoPlatform / zebra

An ongoing Rust implementation of a Komodo node. 🦓
Apache License 2.0
3 stars 5 forks source link

Fix broken original zcash tests #29

Closed dimxy closed 1 year ago

dimxy commented 1 year ago

There are several zebra original tests broken due to some changes we made for komodo network:

dimxy commented 1 year ago

For discussion: I am thinking if it is possible to ensure that the original zcash tests work without our fixes, so we could always pull their latest changes with tests working. As stated above zcash tests are broken due to network parameters changed for the kmd network. So I think maybe we could leave their network parameters intact and create new networks for kmd main and testnet with our specific parameters so we always will add komodo rules only for our networks. This would make the original zcash rules working for their network parameters and zcash tests should work as well. (Ofc there some difficulties with that, f.e. having different MAX_MONEY for both networks - this would require changes in core level like add the network param to the Amount object or provide access from Amount to the current network in another way)

dimxy commented 1 year ago

some tests were fixed for komodo the rest were disabled (PR #2)