Skip to content

Migrate to corepc for bitcoind integration tests#41

Merged
tcharding merged 2 commits intorust-bitcoin:masterfrom
nyonson:corepc-migration
Dec 4, 2025
Merged

Migrate to corepc for bitcoind integration tests#41
tcharding merged 2 commits intorust-bitcoin:masterfrom
nyonson:corepc-migration

Conversation

@nyonson
Copy link
Collaborator

@nyonson nyonson commented Nov 26, 2025

Added a new CI job to run the integration tests, based it on rust-miniscript.

Closes #29

@nyonson nyonson force-pushed the corepc-migration branch 4 times, most recently from 0da2cb8 to 34763ab Compare November 26, 2025 20:01
@nyonson nyonson marked this pull request as ready for review November 26, 2025 20:03
Copy link
Member

@tcharding tcharding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK f8da0a4

@tcharding
Copy link
Member

FWIW we can just use this and get bitcoin via psbt_v2::bitcoin

[dependencies]
anyhow = "1"
bitcoind = { package = "corepc-node", version = "0.10.1", features = ["download"] }
psbt-v2 = { path = "..", features = ["std", "miniscript"] }

@tcharding tcharding merged commit eb775ca into rust-bitcoin:master Dec 4, 2025
24 checks passed
tcharding added a commit to rust-bitcoin/rust-bitcoin-maintainer-tools that referenced this pull request Dec 4, 2025
2d63432 cargo-rbmt: add integration command for corepc conventions (Nick Johnson)

Pull request description:

  This codifies corepc integration testing which I am working on over in [rust-psbt](rust-bitcoin/rust-psbt#41), which is based on the structure in rust-miniscript.

ACKs for top commit:
  tcharding:
    ACK 2d63432

Tree-SHA512: 0ee5d3a4036d8938cfceb81e85763cc14e05aae691f75a6dda113a88484e32ebf12d336cce78178584fc73e5450d6adebe2e12a78d5a660fc0cf67c41d92271a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Re-write integration tests to use the new corepc stack

2 participants