ee6ee8139a Add simple kotlin,swift, and python integration tests (Steve Myers)
Pull request description:
### Description
Add simple kotlin, swift and python integration tests. These tests confirm fixes in #216 and #214.
### Notes to the reviewers
To skip integration tests use:
`cargo test --lib`.
Otherwise java, kotlin, swift, and python need to be installed, and you must run tests with:
`CLASSPATH=./tests/jna/jna-5.8.0.jar cargo test`
### Changelog notice
- Integration tests added, see tests/README.md for updated instructions for running tests. #215
### Checklists
#### All Submissions:
* [x] I've signed all my commits
* [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md)
* [x] I ran `cargo fmt` and `cargo clippy` before committing
#### New Features:
* [x] I've added tests for the new feature
* [x] I've added docs for the new feature
#### Bugfixes:
* [x] I've added tests to reproduce the issue which are now passing
ACKs for top commit:
thunderbiscuit:
ACK [ee6ee81](https://github.com/bitcoindevkit/bdk-ffi/pull/215/commits/ee6ee8139afd74c858216ed99ffdec92dc101787).
waterst0ne:
> ACK [ee6ee81](https://github.com/bitcoindevkit/bdk-ffi/pull/215/commits/ee6ee8139afd74c858216ed99ffdec92dc101787).
Tree-SHA512: 06ab14da1185de431c16b767f187bc8e7792106df54314242b26f225e3a8ddada28317b7cb8bec47b8b248d3088d0305ab777770525540c72d2815349a73728b
fadb316451 Change TxBuilder.finish() to return new TxBuilderResult (Steve Myers)
Pull request description:
### Description
Change TxBuilder.finish() to return new TxBuilderResult.
### Notes to the reviewers
This fixes#179 in that it return both PartiallySignedBitcoinTransaction and TransactionDetails encapsulated in a new TxBuilderResult structure. It does not calculate the fee rate which requires #208.
### Changelog notice
- Breaking Changes
- Changed `TxBuilder.finish()` to return new `TxBuilderResult`.
- APIs Added
- Added `TxBuilderResult` with PSBT and TransactionDetails.
### Checklists
#### All Submissions:
* [x] I've signed all my commits
* [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md)
* [x] I ran `cargo fmt` and `cargo clippy` before committing
#### New Features:
* [x] I've added tests for the new feature
* [x] I've added docs for the new feature
ACKs for top commit:
thunderbiscuit:
ACK [fadb316](https://github.com/bitcoindevkit/bdk-ffi/pull/209/commits/fadb316451af080bf22774a164278cef04acd9cc).
Tree-SHA512: b8aafc53ba86bf7ab12ae3d5bb251d1ec5cb54e4b7d6bfc342155d5cd3726d9a6d114591dc740a7467afb8ef402f3d77ad51e273a27865d6e47192291b570af9
This feature is needed for compatibility with LDKLite, where the
initial entropy given to LDK is the private key of the root of
the BIP32 derivation tree.
Closes#188