bitcoindevkit / bdk / 9035910760

79%

Build Type

push

github

Committed by notmandatory

Commit Message

Merge bitcoindevkit/bdk#1403: Update `bdk_electrum` crate to use sync/full-scan structs

b45897e6f feat(electrum): update docs and simplify logic of `ElectrumExt` (志宇)
92fb6cb37 chore(electrum): do not use `anyhow::Result` directly (志宇)
b2f3cacce feat(electrum): include option for previous `TxOut`s for fee calculation (Wei Chen)
c0d7d60a5 feat(chain)!: use custom return types for `ElectrumExt` methods (志宇)
2945c6be8 fix(electrum): fixed `sync` functionality (Wei Chen)
9ed33c25e docs(electrum): fixed `full_scan`, `sync`, and crate documentation (Wei Chen)
b1f861b93 feat: update logging of electrum examples (志宇)
a6fdfb2ae feat(electrum)!: use new sync/full-scan structs for `ElectrumExt` (志宇)
653e4fed6 feat(wallet): cache txs when constructing full-scan/sync requests (志宇)
58f27b38e feat(chain): introduce `TxCache` to `SyncRequest` and `FullScanRequest` (志宇)
721bb7f51 fix(chain): Make `Anchor` type in `FullScanResult` generic (志宇)
e3cfb8489 feat(chain): `TxGraph::insert_tx` reuses `Arc` (志宇)
2ffb65618 refactor(electrum): remove `RelevantTxids` and track txs in `TxGraph` (Wei Chen)

Pull request description:

  Fixes #1265
  Possibly fixes #1419

  ### Context

  Previous changes such as

  * Universal structures for full-scan/sync (PR #1413)
  * Making `CheckPoint` linked list query-able (PR #1369)
  * Making `Transaction`s cheaply-clonable (PR #1373)

  has allowed us to simplify the interaction between chain-source and receiving-structures (`bdk_chain`).

  The motivation is to accomplish something like this ([as mentioned here](https://github.com/bitcoindev... (continued)

Run Details

152 of 206 new or added lines in 4 files covered. (73.79%)

5 existing lines in 3 files now uncovered.

10653 of 12922 relevant lines covered (82.44%)

16610.3 hits per line