frost/bdk
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Disable test-hardware-signer

Browse Source 
I suspect the latest version of HWI just broke everything
This commit is contained in:
Daniela Brozzoni 2023-03-02 16:14:27 +01:00
parent 303a1703c9
commit 3f5a78ae3b
No known key found for this signature in database
GPG Key ID: 7DE4F1FDCED0AB87
2 changed files with 34 additions and 63 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
.github/workflows/cont_integration.yml vendored
View File

@ -96,32 +96,3 @@ jobs:
run: rustup update run: rustup update
- name: Check fmt - name: Check fmt
run: cargo fmt --all -- --config format_code_in_doc_comments=true --check run: cargo fmt --all -- --config format_code_in_doc_comments=true --check
test_hardware_wallet:
runs-on: ubuntu-20.04
strategy:
matrix:
rust:
- version: 1.65.0 # STABLE
- version: 1.57.0 # MSRV
steps:
- name: Checkout
uses: actions/checkout@v3
- name: Build simulator image
run: docker build -t hwi/ledger_emulator ./ci -f ci/Dockerfile.ledger
- name: Run simulator image
run: docker run --name simulator --network=host hwi/ledger_emulator &
- name: Install Python
uses: actions/setup-python@v4
with:
python-version: '3.9'
- name: Install python dependencies
run: pip install hwi==2.1.1 protobuf==3.20.1
- name: Set default toolchain
run: rustup default ${{ matrix.rust.version }}
- name: Set profile
run: rustup set profile minimal
- name: Update toolchain
run: rustup update
- name: Test
run: cargo test --features test-hardware-signer

68
crates/bdk/tests/wallet.rs
View File

@ -3189,40 +3189,40 @@ fn test_fee_rate_sign_grinding_low_r() {
assert_fee_rate!(psbt, details.fee.unwrap_or(0), fee_rate); assert_fee_rate!(psbt, details.fee.unwrap_or(0), fee_rate);
} }
#[cfg(feature = "test-hardware-signer")] // #[cfg(feature = "test-hardware-signer")]
#[test] // #[test]
fn test_create_signer() { // fn test_hardware_signer() {
use std::sync::Arc; // use std::sync::Arc;
//
use bdk::signer::SignerOrdering; // use bdk::signer::SignerOrdering;
use bdk::wallet::hardwaresigner::HWISigner; // use bdk::wallet::hardwaresigner::HWISigner;
use hwi::types::HWIChain; // use hwi::types::HWIChain;
use hwi::HWIClient; // use hwi::HWIClient;
//
let mut devices = HWIClient::enumerate().unwrap(); // let mut devices = HWIClient::enumerate().unwrap();
if devices.is_empty() { // if devices.is_empty() {
panic!("No devices found!"); // panic!("No devices found!");
} // }
let device = devices.remove(0).unwrap(); // let device = devices.remove(0).unwrap();
let client = HWIClient::get_client(&device, true, HWIChain::Regtest).unwrap(); // let client = HWIClient::get_client(&device, true, HWIChain::Regtest).unwrap();
let descriptors = client.get_descriptors::<String>(None).unwrap(); // let descriptors = client.get_descriptors::<String>(None).unwrap();
let custom_signer = HWISigner::from_device(&device, HWIChain::Regtest).unwrap(); // let custom_signer = HWISigner::from_device(&device, HWIChain::Regtest).unwrap();
//
let (mut wallet, _) = get_funded_wallet(&descriptors.internal[0]); // let (mut wallet, _) = get_funded_wallet(&descriptors.internal[0]);
wallet.add_signer( // wallet.add_signer(
KeychainKind::External, // KeychainKind::External,
SignerOrdering(200), // SignerOrdering(200),
Arc::new(custom_signer), // Arc::new(custom_signer),
); // );
//
let addr = wallet.get_address(LastUnused); // let addr = wallet.get_address(LastUnused);
let mut builder = wallet.build_tx(); // let mut builder = wallet.build_tx();
builder.drain_to(addr.script_pubkey()).drain_wallet(); // builder.drain_to(addr.script_pubkey()).drain_wallet();
let (mut psbt, _) = builder.finish().unwrap(); // let (mut psbt, _) = builder.finish().unwrap();
//
let finalized = wallet.sign(&mut psbt, Default::default()).unwrap(); // let finalized = wallet.sign(&mut psbt, Default::default()).unwrap();
assert!(finalized); // assert!(finalized);
} // }
#[test] #[test]
fn test_taproot_load_descriptor_duplicated_keys() { fn test_taproot_load_descriptor_duplicated_keys() {