1
59%
master: 63%

Ran 26 May 2025 04:50PM UTC

Files 87

Run time 2s

Badge

Embed ▾

Committed 26 May 2025 04:43PM UTC coverage: 59.458% (-0.004%) from 59.462%

Job # 5908696527536128.1

Build Type

Pull #9871

CirrusCI

Committed by

SomberNight

Commit Message

adb.get_tx_height: return "LOCAL" height if missing full signed tx

get_tx_height previously did not consider whether the walletDB has the full tx for
the corresponding txid, and could consider a tx even "mined" and spv-ed, even if
we were missing the raw tx.
Now if the full tx is missing or the tx in the db is partial,
get_tx_height considers it to be LOCAL.

In particular the txbatcher, in `run_iteration`,
- first saves a tx as local *unsigned* (to reserve UTXOs),
- then signs it and tries to broadcast it.

The history tx will later transition to local and signed,
after we get back the broadcasted tx via the Synchronizer dance.
In the meantime there is a race where we have an unsigned tx in the history,
but the txid could already transition to mempool or even to mined,
before we download the full raw tx from the server.
During that time window, it makes the adb state more consistent
to just consider the history tx to be local, as done here.

Pull Request Pull Request #9871: adb.get_tx_height: return "LOCAL" height if missing full signed tx

Run Details

21641 of 36397 relevant lines covered (59.46%)

0.59 hits per line

spesmilo / electrum / 5908696527536128 / 1
59%
master: 63%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 5908696527536128.1

spesmilo / electrum / 5908696527536128 / 1 59% master: 63%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 5908696527536128.1

spesmilo / electrum / 5908696527536128 / 1
59%
master: 63%

README BADGES
x