Skip to content
This repository has been archived by the owner on Mar 14, 2023. It is now read-only.

Test esplora example #195

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

vladimirfomene
Copy link
Contributor

This PR implements basic test for bdk_esplora_example. The following test are implement in this PR:

  • Test for confirmed balance
  • Test for unconfirmed balance
  • Test for Reorg
  • Test for sending a tx

I had to implement a workaround for the reorg test to work for Esplora because after a reorg on bitcoind, electrs still shows the transactions in reorged blocks as confirmed. I followed @notmandatory answer to this stack exchange question

@vladimirfomene vladimirfomene marked this pull request as draft February 21, 2023 19:17
@codecov-commenter
Copy link

codecov-commenter commented Feb 21, 2023

Codecov Report

Merging #195 (40a89de) into master (2a11787) will not change coverage.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##           master     #195   +/-   ##
=======================================
  Coverage   56.74%   56.74%           
=======================================
  Files          10       10           
  Lines         252      252           
=======================================
  Hits          143      143           
  Misses        109      109           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@vladimirfomene vladimirfomene marked this pull request as ready for review February 22, 2023 08:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants