Skip to content

Commit

Permalink
chore: use docker-compose for integration tests
Browse files Browse the repository at this point in the history
Signed-off-by: Gregory Hill <[email protected]>
  • Loading branch information
gregdhill committed Mar 28, 2023
1 parent 326909f commit 6145739
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 12 deletions.
7 changes: 2 additions & 5 deletions .github/workflows/cargo-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,6 @@ jobs:
test:
name: Test Suite
runs-on: [self-hosted, linux]
services:
bitcoind:
image: docker.io/interlayhq/bitcoin-core:22.0
ports:
- 18443:18443

strategy:
matrix:
Expand Down Expand Up @@ -62,7 +57,9 @@ jobs:
BITCOIN_RPC_URL: http://127.0.0.1:18443
BITCOIN_RPC_USER: user
BITCOIN_RPC_PASS: pass
ELECTRS_URL: http://localhost:3002
run: |
docker-compose up --detach
cargo test --release --workspace --features ${{ matrix.metadata }} --features uses-bitcoind
- name: build
run: |
Expand Down
10 changes: 5 additions & 5 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion bitcoin/tests/electrs.rs
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// #![cfg(feature = "uses-bitcoind")]
#![cfg(feature = "uses-bitcoind")]

use bitcoin::{
secp256k1::{constants::SECRET_KEY_SIZE, Secp256k1},
Expand Down
5 changes: 4 additions & 1 deletion vault/src/execution.rs
Original file line number Diff line number Diff line change
Expand Up @@ -563,7 +563,10 @@ pub async fn execute_open_requests(
// or just pay and execute all requests individually
for (id, request) in open_requests {
// try finding transaction directly (if supported)
match read_only_btc_rpc.get_tx_for_op_return(request.btc_address, request.amount, id).await {
match read_only_btc_rpc
.get_tx_for_op_return(request.btc_address, request.amount, id)
.await
{
Ok(Some(txid)) => {
create_payment_worker(
shutdown_tx.clone(),
Expand Down

0 comments on commit 6145739

Please sign in to comment.