4
63%
master: 63%

Ran 15 May 2025 07:46PM UTC

Files 87

Run time 2s

Badge

Embed ▾

Committed 15 May 2025 07:35PM UTC coverage: 59.68% (-0.02%) from 59.698%

Job # 4829117322887168.4

Build Type

push

CirrusCI

Committed by

SomberNight

Commit Message

synchronizer: get_transaction should discard tx_height as it can change

tx_height comes from the `get_history` RPC, we then call the `get_transaction` RPC.
By the time the `get_transaction` RPC returns, we might have received another
scripthash status update, called `get_history` again, and updated height for the txid.
Synchronizer._get_transaction() should not call adb.receive_tx_callback() with
the old tx_height (but it was doing exactly that).

Patch to trigger, e.g. regtest failures:
(e.g. for tests.regtest.TestLightningAB.test_extract_preimage)
```
diff --git a/electrum/interface.py b/electrum/interface.py
index 8649652b9c..fce7a1c6de 100644
--- a/electrum/interface.py
+++ b/electrum/interface.py
@@ -991,6 +991,7 @@ class Interface(Logger):
         return res

     async def get_transaction(self, tx_hash: str, *, timeout=None) -> str:
+        await asyncio.sleep(3)
         if not is_hash256_str(tx_hash):
             raise Exception(f"{repr(tx_hash)} is not a txid")
         raw = await self.session.send_request('blockchain.transaction.get', [tx_hash], timeout=timeout)

```

Run Details

21533 of 36081 relevant lines covered (59.68%)

0.6 hits per line

spesmilo / electrum / 4829117322887168 / 4
63%
master: 63%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4829117322887168.4

spesmilo / electrum / 4829117322887168 / 4 63% master: 63%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4829117322887168.4

spesmilo / electrum / 4829117322887168 / 4
63%
master: 63%

README BADGES
x