test: Add test for descriptor types.

[ItoroD]

Description

This test was taken from our old Offline test that @thunderbiscuit created here. I only updated with a few additions like Check for sortedmulti. I was taking a look at issue #687 and figured these are good tests to bring back. These tests have shown some more descriptor types we can do in the bindings. I am particular curious about tr and the range of complex descriptors we are able to perform currently and those we cannot. Adding these test is a great way to start.

Notes to the reviewers

Documentation

• bdk_wallet
• bitcoin
• uniffi
• Other:

Checklists

All Submissions:

• I've signed all my commits
• I followed the contribution guidelines
• I ran cargo fmt and cargo clippy before committing
• I've added a changelog in the next release tracking issue (see example)
• I've linked the relevant upstream docs or specs above

[reez]

nice addition. looks like its a lot of coverage, if we want full coverage we may want to add wsh & sh_wsh?

[ItoroD]

nice addition. looks like its a lot of coverage, if we want full coverage we may want to add wsh & sh_wsh?

Thanks for the review. I have added those now.

I am hoping we would be able to update these tests sometime later to use the actual helper methods from miniscript as in #949

[thunderbiscuit]

This one needs a quick rebase and is good to go. At some point I'd be great to maybe add to the test all the descriptors that are not supported, just to make it more explicit and easier to answer questions like this one.

[thunderbiscuit]

@ItoroD I think you accidentally committed your Cargo.lock file on your commit.

[ItoroD]

@ItoroD I think you accidentally committed your Cargo.lock file on your commit.

My bad

[thunderbiscuit]

ACK 0250bc8.