tests: add concise docstrings in chain_tiebreaks #34167
pull Olly60 wants to merge 2 commits into bitcoin:master from Olly60:master changing 1 files +2 −0-
Olly60 commented at 9:45 pm on December 28, 2025: noneAdded concise docstrings for test_chain_split_in_memory and test_chain_split_from_disk
-
tests: add concise docstrings in chain_tiebreaks 7f177280fd
-
DrahtBot added the label Tests on Dec 28, 2025
-
DrahtBot commented at 9:45 pm on December 28, 2025: contributor
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
Code Coverage & Benchmarks
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/34167.
Reviews
See the guideline for information on the review process. A summary of reviews will appear here.
-
in test/functional/feature_chain_tiebreaks.py:107 in 7f177280fd
103@@ -103,6 +104,7 @@ def test_chain_split_in_memory(self): 104 node.invalidateblock(blocks[0].hash_hex) 105 106 def test_chain_split_from_disk(self): 107+ """Test active chain selection with in-memory forks and out-of-order blocks."""
billymcbip commented at 3:09 pm on December 30, 2025:Shouldn’t it be “on-disk forks” rather than “in-memory forks”?
bensig commented at 6:25 am on January 1, 2026:+1 on @billymcbip’s observation, looks like copypasta
Olly60 commented at 1:07 pm on January 1, 2026:+1 on @billymcbip’s observation, looks like copypasta
My bad I should I thought it was that. I’ll make sure I double check next time
Fix docstring: on-disk forks (not in-memory) 87ca084518DrahtBot added the label CI failed on Jan 2, 2026DrahtBot removed the label CI failed on Jan 2, 2026maflcko commented at 8:25 am on January 5, 2026: memberPlease squash your commits according to https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md#squashing-commits
Though, I think the code is already self-documenting. So thx, but i think i’ll close this for now.
maflcko closed this on Jan 5, 2026
github-metadata-mirror
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-01-07 03:13 UTC
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-01-07 03:13 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me
More mirrored repositories can be found on mirror.b10c.me