feat: Add Multipath descriptor support

[tvpeter]

Description

This PR enables support for BIP389 multipath descriptors. It uses the existing external descriptor and verifies whether the provided descriptor is a multipath descriptor. If it is, a wallet will be created using the Wallet::create_from_two_path_descriptor() method. If not, the standard wallet creation logic will be applied.

Fixes#195

Changelog notice

• Add support for BIP389 multipath descriptors

Checklists

All Submissions:

• I've signed all my commits
• I followed the contribution guidelines
• I ran cargo fmt and cargo clippy before committing

New Features:

• I've added tests for the new feature
• I've updated CHANGELOG.md

[coveralls]

Pull Request Test Coverage Report for Build 21787788027

Details

• 89 of 108(82.41%) changed or added relevant lines in 2 files are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage increased (+4.0%) to 14.733%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/utils.rs	89	92	96.74%
src/handlers.rs	0	16	0.0%

Files with Coverage Reduction	New Missed Lines	%
src/handlers.rs	1	12.92%

Totals
Change from base Build 21153868360:	4.0%
Covered Lines:	381
Relevant Lines:	2586

---

💛 - Coveralls

[AmosOO7]

tACK 1433513

[codecov]

Codecov Report

❌ Patch coverage is 82.72727% with 19 lines in your changes missing coverage. Please review.
✅ Project coverage is 15.00%. Comparing base (a675935) to head (b03ebc7).
⚠️ Report is 27 commits behind head on master.

Files with missing lines	Patch %	Lines
src/handlers.rs	0.00%	16 Missing ⚠️
src/utils.rs	96.80%	3 Missing ⚠️

Additional details and impacted files

@@ Coverage Diff @@## master #238 +/- ##
==========================================
+ Coverage 10.86% 15.00% +4.13% 
==========================================
Files 8 8 Lines 2466 2553 +87 ==========================================
+ Hits 268 383 +115 + Misses 2198 2170 -28 

Flag	Coverage Δ
rust	15.00% <82.72%> (+4.13%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Musab1258]

Concept ACK.
But I noticed the if/else block determining the Wallet builder is repeated in both new_persisted_wallet (lines 293-299) and new_wallet (lines 321-327) functions in src/utils.rs . To keep things DRY, would it make sense to extract that logic into an helper function like build_wallet_creator?

Something like:

fnbuild_wallet_creator(ext_descriptor:String,int_descriptor:Option<String>,) -> CreateParams{ifletSome(int_descriptor) = int_descriptor {Wallet::create(ext_descriptor, int_descriptor)}elseifis_multipath_desc(&ext_descriptor){Wallet::create_from_two_path_descriptor(ext_descriptor)}else{Wallet::create_single(ext_descriptor)}}

[tvpeter]

Closing this PR in favour of #309