Gate payjoin-ffi bitcoind types behind feature

Split _test-utils into _test-utils (constants + TestServices only)
and _test-utils-bitcoind (adds BitcoindEnv, RpcClient, etc.).
This lets FFI consumers like Dart unit tests import test constants
without triggering the corepc-node/bitcoind download.

Update all binding-generation and test scripts to use
_test-utils-bitcoind where integration-test types are needed.

Author: DanGould

payjoin-ffi/Cargo.toml

@@ -9,7 +9,8 @@ exclude = ["tests"]
 default = []
 csharp = ["dep:uniffi-bindgen-cs"]
 dart = ["dep:uniffi-dart"]
-_test-utils = ["payjoin-test-utils", "payjoin-test-utils/bitcoind", "tokio"]
+_test-utils = ["payjoin-test-utils", "tokio"]
+_test-utils-bitcoind = ["_test-utils", "payjoin-test-utils/bitcoind"]
 _manual-tls = ["payjoin/_manual-tls"]
 wasm_js = ["getrandom/js"]
 

payjoin-ffi/contrib/lint.sh

@@ -2,7 +2,7 @@
 set -e
 
 # Individual features with no defaults.
-features=("_manual-tls" "_test-utils")
+features=("_manual-tls" "_test-utils" "_test-utils-bitcoind")
 
 for feature in "${features[@]}"; do
     # Don't duplicate --all-targets clippy. Clippy end-user code, not tests.

payjoin-ffi/contrib/test.sh

@@ -4,7 +4,7 @@ set -e
 RUST_VERSION=$(rustc --version | awk '{print $2}')
 
 if [[ ! $RUST_VERSION =~ ^1\.85\. ]]; then
-    cargo test --package payjoin-ffi --verbose --features=_manual-tls,_test-utils
+    cargo test --package payjoin-ffi --verbose --features=_manual-tls,_test-utils-bitcoind
 else
     echo "Skipping payjoin-ffi tests for Rust version $RUST_VERSION (MSRV)"
 fi

payjoin-ffi/csharp/README.md

@@ -56,20 +56,20 @@ dotnet test
 
 Generation uses the Cargo-managed C# generator from `payjoin-ffi/Cargo.toml`.
 
-By default, generation builds `payjoin-ffi` with `_test-utils` enabled to keep parity with other language test scripts. Override via `PAYJOIN_FFI_FEATURES`.
+By default, generation builds `payjoin-ffi` with `_test-utils-bitcoind` enabled to keep parity with other language test scripts. Override via `PAYJOIN_FFI_FEATURES`.
 
 ### Unix shells
 
 ```shell
-export PAYJOIN_FFI_FEATURES=_test-utils     # default behavior
+export PAYJOIN_FFI_FEATURES=_test-utils-bitcoind     # default behavior
 # export PAYJOIN_FFI_FEATURES=""            # build without extra features
 bash ./scripts/generate_bindings.sh
 ```
 
 ### PowerShell
 
 ```powershell
-$env:PAYJOIN_FFI_FEATURES = "_test-utils"   # default behavior
+$env:PAYJOIN_FFI_FEATURES = "_test-utils-bitcoind"   # default behavior
 # $env:PAYJOIN_FFI_FEATURES = ""            # build without extra features
 powershell -ExecutionPolicy Bypass -File .\scripts\generate_bindings.ps1
 dotnet build

payjoin-ffi/csharp/scripts/generate_bindings.ps1

@@ -40,8 +40,8 @@ Write-Host "Generating payjoin C#..."
 if ($null -ne $env:PAYJOIN_FFI_FEATURES) {
     $payjoinFfiFeatures = $env:PAYJOIN_FFI_FEATURES
 } else {
-    # Keep parity with other language test scripts: include _test-utils by default.
-    $payjoinFfiFeatures = "_test-utils"
+    # Keep parity with other language test scripts: include _test-utils-bitcoind by default.
+    $payjoinFfiFeatures = "_test-utils-bitcoind"
 }
 
 if ($payjoinFfiFeatures) {

payjoin-ffi/csharp/scripts/generate_bindings.sh

@@ -22,8 +22,8 @@ SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 cd "$SCRIPT_DIR/../.."
 
 echo "Generating payjoin C#..."
-# Keep parity with other language test scripts: include _test-utils by default.
-PAYJOIN_FFI_FEATURES=${PAYJOIN_FFI_FEATURES:-_test-utils}
+# Keep parity with other language test scripts: include _test-utils-bitcoind by default.
+PAYJOIN_FFI_FEATURES=${PAYJOIN_FFI_FEATURES:-_test-utils-bitcoind}
 GENERATOR_FEATURES="csharp"
 if [[ -n $PAYJOIN_FFI_FEATURES ]]; then
     GENERATOR_FEATURES="$GENERATOR_FEATURES,$PAYJOIN_FFI_FEATURES"

payjoin-ffi/dart/native/Cargo.toml

@@ -10,6 +10,7 @@ publish = false
 
 [features]
 _test-utils = ["payjoin-ffi/_test-utils"]
+_test-utils-bitcoind = ["payjoin-ffi/_test-utils-bitcoind"]
 
 [lib]
 name = "payjoin_ffi_wrapper"

payjoin-ffi/dart/scripts/generate_bindings.sh

@@ -19,7 +19,7 @@ fi
 
 cd ../
 echo "Generating payjoin dart..."
-cargo build --features dart,_test-utils --profile dev
-cargo run --features dart,_test-utils --profile dev --bin uniffi-bindgen -- --library ../target/debug/$LIBNAME --language dart --out-dir dart/lib/
+cargo build --features dart,_test-utils-bitcoind --profile dev
+cargo run --features dart,_test-utils-bitcoind --profile dev --bin uniffi-bindgen -- --library ../target/debug/$LIBNAME --language dart --out-dir dart/lib/
 
 echo "All done!"

payjoin-ffi/python/scripts/generate_bindings.sh

@@ -19,15 +19,15 @@ uv run python --version
 
 cd ../
 # This is a test script the actual release should not include the test utils feature
-cargo build --features _test-utils --profile dev
-cargo run --features _test-utils --profile dev --bin uniffi-bindgen generate --library ../target/debug/$LIBNAME --language python --out-dir python/src/payjoin/
+cargo build --features _test-utils-bitcoind --profile dev
+cargo run --features _test-utils-bitcoind --profile dev --bin uniffi-bindgen generate --library ../target/debug/$LIBNAME --language python --out-dir python/src/payjoin/
 
 if [[ $OS == "Darwin" ]]; then
     echo "Generating native binaries..."
     rustup target add aarch64-apple-darwin x86_64-apple-darwin
     # This is a test script the actual release should not include the test utils feature
-    cargo build --profile dev --target aarch64-apple-darwin --features _test-utils &
-    cargo build --profile dev --target x86_64-apple-darwin --features _test-utils &
+    cargo build --profile dev --target aarch64-apple-darwin --features _test-utils-bitcoind &
+    cargo build --profile dev --target x86_64-apple-darwin --features _test-utils-bitcoind &
     wait
 
     echo "Building macos fat library"
@@ -39,7 +39,7 @@ else
     echo "Generating native binaries..."
     rustup target add x86_64-unknown-linux-gnu
     # This is a test script the actual release should not include the test utils feature
-    cargo build --profile dev --target x86_64-unknown-linux-gnu --features _test-utils
+    cargo build --profile dev --target x86_64-unknown-linux-gnu --features _test-utils-bitcoind
 
     echo "Copying payjoin_ffi binary"
     cp ../target/x86_64-unknown-linux-gnu/debug/$LIBNAME python/src/payjoin/$LIBNAME

payjoin-ffi/src/test_utils.rs

@@ -2,11 +2,13 @@ use std::io;
 use std::sync::Arc;
 
 use lazy_static::lazy_static;
-use payjoin_test_utils::corepc_node::AddressType;
+#[cfg(feature = "_test-utils-bitcoind")]
+use payjoin_test_utils::corepc_node::{self, AddressType};
 use payjoin_test_utils::{
-    corepc_node, EXAMPLE_URL, INVALID_PSBT, ORIGINAL_PSBT, PAYJOIN_PROPOSAL,
-    PAYJOIN_PROPOSAL_WITH_SENDER_INFO, QUERY_PARAMS, RECEIVER_INPUT_CONTRIBUTION,
+    EXAMPLE_URL, INVALID_PSBT, ORIGINAL_PSBT, PAYJOIN_PROPOSAL, PAYJOIN_PROPOSAL_WITH_SENDER_INFO,
+    QUERY_PARAMS, RECEIVER_INPUT_CONTRIBUTION,
 };
+#[cfg(feature = "_test-utils-bitcoind")]
 use serde_json::Value;
 use tokio::runtime::Runtime;
 use tokio::sync::Mutex;
@@ -16,13 +18,15 @@ lazy_static! {
         Arc::new(std::sync::Mutex::new(Runtime::new().expect("Failed to create Tokio runtime")));
 }
 
+#[cfg(feature = "_test-utils-bitcoind")]
 #[derive(uniffi::Object)]
 pub struct BitcoindEnv {
     pub bitcoind: Arc<BitcoindInstance>,
     pub receiver: Arc<RpcClient>,
     pub sender: Arc<RpcClient>,
 }
 
+#[cfg(feature = "_test-utils-bitcoind")]
 #[uniffi::export]
 impl BitcoindEnv {
     pub fn get_receiver(&self) -> Arc<RpcClient> { self.receiver.clone() }
@@ -32,16 +36,19 @@ impl BitcoindEnv {
     pub fn get_bitcoind(&self) -> Arc<BitcoindInstance> { self.bitcoind.clone() }
 }
 
+#[cfg(feature = "_test-utils-bitcoind")]
 #[derive(uniffi::Object)]
 pub struct BitcoindInstance {
     _inner: corepc_node::Node,
 }
 
+#[cfg(feature = "_test-utils-bitcoind")]
 #[derive(uniffi::Object)]
 pub struct RpcClient {
     inner: corepc_node::Client,
 }
 
+#[cfg(feature = "_test-utils-bitcoind")]
 #[uniffi::export]
 impl RpcClient {
     pub fn call(&self, method: String, params: Vec<Option<String>>) -> Result<String, FfiError> {
@@ -63,6 +70,7 @@ impl RpcClient {
     }
 }
 
+#[cfg(feature = "_test-utils-bitcoind")]
 #[derive(Debug, thiserror::Error, PartialEq, Eq, uniffi::Error)]
 pub enum FfiError {
     #[error("Init error: {0}")]
@@ -73,6 +81,7 @@ pub enum FfiError {
     Message(String),
 }
 
+#[cfg(feature = "_test-utils-bitcoind")]
 impl FfiError {
     pub fn new(msg: impl Into<String>) -> Self { FfiError::Message(msg.into()) }
 }
@@ -177,6 +186,7 @@ impl TestServices {
     }
 }
 
+#[cfg(feature = "_test-utils-bitcoind")]
 #[uniffi::export]
 pub fn init_bitcoind_sender_receiver() -> Result<Arc<BitcoindEnv>, FfiError> {
     let (bitcoind, receiver, sender) = payjoin_test_utils::init_bitcoind_sender_receiver(