fanquake 6737331c4c Merge bitcoin/bitcoin#29363: test: Fix CPartialMerkleTree.nTransactions signedness ...

facafa90f7 test: Fix CPartialMerkleTree.nTransactions signedness (MarcoFalke)

Pull request description:

  It is unsigned in Bitcoin Core, so the tests should match it:

  aa9231fafe/src/merkleblock.h (L59)

  Large positive values, or "negative" values, are rejected anyway, but it still seems fine to fix this.

  The bug was introduced when the code was written in d280617bf5.

  (Lowercase `i` means signed, see https://docs.python.org/3/library/struct.html#format-characters)

ACKs for top commit:
  theStack:
    LGTM ACK facafa90f7
  Empact:
    ACK facafa90f7

Tree-SHA512: 35ac11bb5382dffe132bfae6097efc343ef6c06b1b4b1545130ca27b228ca6894679004862fee921b095172abaddbef5972c24d9bc195ce970f35643bd4a0f09

2024-02-07 15:08:02 +00:00

..

crypto

…

__init__.py

…

address.py

…

authproxy.py

…

bdb.py

…

bip340_test_vectors.csv

…

blockfilter.py

Merge bitcoin/bitcoin#28605: Fix typos

2023-11-16 10:35:49 +00:00

blocktools.py

…

coverage.py

…

descriptors.py

…

key.py

…

messages.py

test: Fix CPartialMerkleTree.nTransactions signedness

2024-02-01 13:18:40 +01:00

netutil.py

…

p2p.py

Merge bitcoin/bitcoin#29356: test: make v2transport arg in addconnection mandatory and few cleanups

2024-02-06 11:02:36 +00:00

psbt.py

…

script_util.py

…

script.py

Merge bitcoin/bitcoin#28725: test: refactor: use built-in collection types for type hints (Python 3.9 / PEP 585)

2023-11-17 11:19:17 +00:00

segwit_addr.py

…

socks5.py

…

test_framework.py

test: Use blocks_path where possible

2024-01-17 16:48:47 +01:00

test_node.py

net: enable v2transport by default

2024-01-29 22:48:01 -05:00

test_shell.py

…

util.py

test: detect OS consistently using platform.system()

2023-12-08 18:16:24 +01:00

v2_p2p.py

[test] make v2transport arg in addconnection mandatory and few cleanups

2024-01-31 22:37:54 +05:30

wallet_util.py

…

wallet.py

…