test: expand offline descriptor test coverage to include all BIP descriptor constructors
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 84/100
Piste de recherche
Commencez par tests/test_offline_descriptor.py et le test test_descriptor_bip86 existant afin de suivre son modèle fixed-mnemonic et full-descriptor-string. Ajoutez une couverture pour les constructeurs BIP44, BIP49, BIP84, BIP86, new_wsh_sortedmulti et new_sh_sortedmulti indiqués, puis exécutez les tests des descripteurs offline ; c’est terminé lorsque toutes les chaînes de descripteurs et sommes de contrôle attendues passent sans accès au réseau.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
What is missing
tests/test_offline_descriptor.py currently contains a single test:
test_descriptor_bip86 — tests Descriptor.new_bip86() for the external keychain only
bdkpython exposes 8 BIP descriptor template constructors (from bdk_wallet::template)
across 4 standards, each with a private key variant (DescriptorSecretKey) and a
watch-only public key variant (DescriptorPublicKey), plus 3 miniscript multisig
constructors. None of the following have test coverage today:
| Constructor | Standard | Script type | Tested? |
|---|---|---|---|
Descriptor.new_bip44() |
BIP44 | pkh(...) |
No |
Descriptor.new_bip44_public() |
BIP44 | pkh(...) |
No |
Descriptor.new_bip49() |
BIP49 | sh(wpkh(...)) |
No |
Descriptor.new_bip49_public() |
BIP49 | sh(wpkh(...)) |
No |
Descriptor.new_bip84() |
BIP84 | wpkh(...) |
No |
Descriptor.new_bip84_public() |
BIP84 | wpkh(...) |
No |
Descriptor.new_bip86() internal |
BIP86 | tr(...) |
No |
Descriptor.new_bip86_public() |
BIP86 | tr(...) |
No |
Descriptor.new_wsh_sortedmulti() |
miniscript | wsh(sortedmulti(...)) |
No |
Descriptor.new_sh_sortedmulti() |
miniscript | sh(sortedmulti(...)) |
No |
Descriptor.new_sh_wsh_sortedmulti() |
miniscript | sh(wsh(sortedmulti(...))) |
No |
What I propose to add
13 new tests in tests/test_offline_descriptor.py covering:
- BIP44 — external keychain, internal keychain, and watch-only (
new_bip44_public) variant - BIP49 — external keychain, internal keychain, and watch-only (
new_bip49_public) variant - BIP84 — external keychain, internal keychain, and watch-only (
new_bip84_public) variant - BIP86 — internal keychain and watch-only (
new_bip86_public) variant (external already exists) - Multisig —
new_wsh_sortedmultiandnew_sh_sortedmulti2-of-3
I am intentionally leaving new_sh_wsh_sortedmulti out of this PR to keep the scope
focused, and can follow up with a separate issue if the maintainers consider it worthwhile.
All tests will be:
- Fully offline — no network access required
- Deterministic — fixed test mnemonic so expected outputs are independently verifiable
with any compatible BIP32/BIP44/BIP49/BIP84/BIP86 implementation - Following the same pattern as the existing test: fixed mnemonic → derive key → assert full descriptor string including checksum
Why this matters
Descriptor construction is the first thing every bdkpython user does — it is required
to create a Wallet. Without test coverage for these constructors, a regression in the
FFI layer (e.g. wrong derivation path, wrong script type, wrong network prefix, wrong
checksum) would go undetected until a user reports it in production.
Each test pins the full descriptor output string to a known-correct value, which means
any future change to the FFI type conversion or the bdk_wallet::template behavior
that silently produces wrong output will immediately fail CI.
This is a pure Python addition — no changes to the Rust layer.
Checklist before I open a PR
- Maintainer confirms this contribution is welcome
- I will follow the existing file structure in
tests/test_offline_descriptor.py - I will verify all expected values against the fixed mnemonic before submitting
- Langage dominant
- Python
- Étoiles
- 11
- Forks
- 12
- Merge moyen
- 2 j 13 h
- PR mergées (30 j)
- 2
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de bitcoindevkit/bdk-python
-
Error parsing combo descriptor Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
bitcoindevkit/bdk-python#31 · 3 commentaires ·
Toutes les issues de bitcoindevkit/bdk-python
Issues similaires
-
documentation help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 90/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 90/100
simonw/sqlite-utils#872 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100