Merge pull request #139 from DanGould/test-cli-v2-build

Test payjoin-cli v2 builds
payjoin · Dec 13, 2023 · b0d530e · b0d530e
2 parents 624ab7f + 1f61d31
commit b0d530e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml
@@ -36,8 +36,11 @@ jobs:
         run: |
           cargo test --package payjoin --verbose --features=send,receive,v2 --test integration v2_to_v2
           cargo test --package payjoin --verbose --features=send,receive,v2 --test integration v1_to_v2
-      - name: test payjoin-cli bin
+      - name: test payjoin-cli bin v1
         run: cargo test --package payjoin-cli --verbose --features=danger-local-https
+      - name: build payjoin-cli bin v2
+        if: matrix.rust != '1.63.0'
+        run: cargo build --package payjoin-cli --verbose --features=v2
 
   fmt:
     runs-on: ubuntu-latest

diff --git a/payjoin-cli/src/app.rs b/payjoin-cli/src/app.rs
@@ -539,7 +539,7 @@ impl App {
             )?;
 
         // Receive Check 1: Can Broadcast
-        let proposal = proposal.check_can_broadcast(|tx| {
+        let proposal = proposal.check_broadcast_suitability(None, |tx| {
             let raw_tx = bitcoin::consensus::encode::serialize_hex(&tx);
             let mempool_results =
                 bitcoind.test_mempool_accept(&[raw_tx]).map_err(|e| Error::Server(e.into()))?;