[electrum] add BIP44 mnemonic-to-address test with the eCash derivation paths
ClosedPublic
Actions

Authored by PiRK on Jul 29 2025, 14:36.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC4280e4c1d9fa: [electrum] add BIP44 mnemonic-to-address test with the eCash derivation paths

Summary

While looking for test vectors for eCash address generation from seed, I noticed that Electrum ABC does not have a test using the m/44'/899'/0' derivation path.
It is useful to have such a test so we can share it with other wallets willing to support eCash and ensure everyone generates the same addresses from a given mnemonic phrase.

Add also tests for the 1899' path to check CashTab generates the same address. One of the tests matches an existing CashTab unittest, and I checked the other one by importing the seed into CashTab and comparing the first receive address.

Test Plan

python test_runner.py

Diff Detail

Repository

rABC Bitcoin ABC

Branch

bip44_899_test

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 33991
Build 67451: Build Diff	electrum-functional-tests · electrum-tests
Build 67450: arc lint + arc unit

Event Timeline

PiRK created this revision.Jul 29 2025, 14:36

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 29 2025, 14:36

PiRK requested review of this revision.Jul 29 2025, 14:36

PiRK retitled this revision from [electrum] add a BIP44 mnemonic to address test with the eCash derivation paths to [electrum] add BIP44 mnemonic-to-address test with the eCash derivation paths.Jul 29 2025, 14:37

Harbormaster completed remote builds in B33990: Diff 55025.Jul 29 2025, 14:44

let's add also tests for the 1 account (m/44'/899'/1' derivation)

test also a m/44'/899'/1' derivation path, and test a few public keys
This gives us identical coverage to a brave-wallet unit test on which i'm working, which is based on an ZCash test.

Harbormaster completed remote builds in B33991: Diff 55026.Jul 29 2025, 15:51

Fabien accepted this revision.Jul 29 2025, 16:36

This revision is now accepted and ready to land.Jul 29 2025, 16:36

Closed by commit rABC4280e4c1d9fa: [electrum] add BIP44 mnemonic-to-address test with the eCash derivation paths (authored by PiRK). · Explain WhyJul 30 2025, 08:57

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABC4280e4c1d9fa: [electrum] add BIP44 mnemonic-to-address test with the eCash derivation paths.