• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

bitcoindevkit / bdk / 12891878608
79%

Build:
DEFAULT BRANCH: master
Ran 21 Jan 2025 04:59PM UTC
Jobs 1
Files 51
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

21 Jan 2025 04:54PM UTC coverage: 85.956% (+0.01%) from 85.942%
12891878608

push

github

ValuedMammal
Merge bitcoindevkit/bdk#1685: test(electrum): test for checking that fee calculation is correct

<a class=hub.com/bitcoindevkit/bdk/commit/f429973111c7fb21304f268b70c55d2d4f08eb2b">f42997311 test(electrum): test for checking that fee calculation is correct (f3r10)

Pull request description:

  ### Description

  This PR adds a unit test for checking that the fee calculation of `bdk_electrum` is correct.

  > a "preliminary" tx to an address in Core's wallet such that the output created becomes the input of the next tx. This preliminary tx is the prev_tx that needs to be fetched in order to calculate the fee of the tx sent to the receiver. This way is better because the required outpoint is less determined. Then we assert we have the right (outpoint, txout) in the update TxGraph  https://github.com/bitcoindevkit/bdk/issues/1444#issuecomment-2112751282

  ### Notes to the reviewers

  fixes: #1444

  ### Changelog notice

  ### Checklists

  #### All Submissions:

  * [x] I've signed all my commits
  * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md)
  * [x] I ran `cargo fmt` and `cargo clippy` before committing

ACKs for top commit:
  ValuedMammal:
    ACK f429973111c7fb21304f268b70c55d2d4f08eb2b

Tree-SHA512: f1f541989dd17d027d4b6f39f48dba8ab34

11966 of 13921 relevant lines covered (85.96%)

21296.34 hits per line

Jobs
ID Job ID Ran Files Coverage
1 12891878608.1 21 Jan 2025 04:59PM UTC 51
85.96
GitHub Action Run
Source Files on build 12891878608
  • Tree
  • List 51
  • Changed 23
  • Source Changed 0
  • Coverage Changed 1
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • b7d64c2f on github
  • Prev Build on master (#12809582414)
  • Next Build on master (#12894272707)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc