Skip to content

feat: Add end-to-end bitcoin-rpc-midas integration test #228

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

nervana21
Copy link
Contributor

  • Introduces a new test (bitcoin_rpc_midas.rs) demonstrating a full PSBT lifecycle:
    • start bitcoind
    • create wallet
    • mine blocks
    • generate address
    • create and sign PSBT
    • finalize and broadcast
    • confirm transaction
  • All functionality is exercised through the generated midas client with no manual RPC calls.

The test can be run with the following command:

cargo test --test bitcoin_rpc_midas --features 28_0 -- --nocapture

Closes #4

- Adds `bitcoin-rpc-midas` as a dependency in `integration_test/Cargo.toml`.

- Introduces a new test (`bitcoin_rpc_midas.rs`) demonstrating a full PSBT lifecycle:
  - start bitcoind
  - create wallet
  - mine blocks
  - generate address
  - create and sign PSBT
  - finalize and broadcast
  - confirm transaction
- All functionality is exercised through the generated `midas` client with no manual RPC calls.

Closes rust-bitcoin#4
@tcharding
Copy link
Member

I'm not exactly sure what this PR is attempting to show? IIUC bitcoin-rpc-midas is a competitor and complete replacement for this repo? Or am I missing something?

@nervana21
Copy link
Contributor Author

Hi @tcharding,
I believe you're right, but that's not a claim I felt confident making on my own.

This PR was an attempt at making the value-add as easy as possible to evaluate for those who understand the ecosystem better than I do. Happy to adapt if you have any bandwidth for guidance 🙂

@tcharding
Copy link
Member

I'm still unsure what you want me to do with this PR. I posted on #4, maybe that clears up the confusion.

@nervana21 nervana21 closed this Jun 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider using code generation to produce Rust code from structured RPC information exported from Bitcoin Core
2 participants