The mentioned bug (issue #26245) has been fixed in PR #30666, so the workaround is not needed anymore.
test: remove unnecessary m_best_header setting hack in feature_assumeutxo.py #32746
pull theStack wants to merge 1 commits into bitcoin:master from theStack:202506-test-assumeutxo_remove_hack_for_mbestheader changing 1 files +0 −6-
theStack commented at 12:50 PM on June 13, 2025: contributor
-
206bc05e62
test: remove unnecessary m_best_header setting hack in feature_assumeutxo.py
The mentioned bug has been fixed in PR #30666, so the workaround is not needed anymore.
-
DrahtBot commented at 12:50 PM on June 13, 2025: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage & Benchmarks
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32746.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
- DrahtBot added the label Tests on Jun 13, 2025
- theStack renamed this:
test: remove unnecessary m_best_header setting hack in feature_assume_utxo.py
test: remove unnecessary m_best_header setting hack in feature_assumeutxo.py
on Jun 13, 2025 -
fjahr commented at 2:16 PM on June 13, 2025: contributor
ACK 206bc05e62c6a71c328eaab90fd865f46328642f
-
janb84 commented at 2:26 PM on June 13, 2025: contributor
ACK 206bc05e62c6a71c328eaab90fd865f46328642f
The issue is resolved, the "hack" is not needed anymore.
- Validated that the test still runs / passed ✅
-
mzumsande commented at 2:37 PM on June 13, 2025: contributor
utACK 206bc05e62c6a71c328eaab90fd865f46328642f
- fanquake merged this on Jun 13, 2025
- fanquake closed this on Jun 13, 2025
- theStack deleted the branch on Jun 13, 2025
- bitcoin deleted a comment on Jun 13, 2025
- stringintech referenced this in commit f19e7819e9 on Jul 19, 2025
- alexanderwiederin referenced this in commit 9997c3d993 on Jul 25, 2025
- alexanderwiederin referenced this in commit 8d6ebb9c4e on Jul 28, 2025
- alexanderwiederin referenced this in commit 533a4585b5 on Jul 28, 2025
- yuvicc referenced this in commit 22f55cf11d on Aug 26, 2025
- bug-castercv502 referenced this in commit d4bd8a1259 on Sep 28, 2025
- stickies-v referenced this in commit a19c56cd7c on Nov 4, 2025
- Kino1994 referenced this in commit b1a5eb957a on Jun 28, 2026
- BigcoinBGC referenced this in commit 6e3be238de on Jun 30, 2026
- PastaPastaPasta referenced this in commit 78d88083d8 on Jul 12, 2026
- PastaPastaPasta referenced this in commit f359fdbce6 on Jul 12, 2026
- PastaPastaPasta referenced this in commit 7459dadf6f on Jul 12, 2026
- PastaPastaPasta referenced this in commit f8520dae66 on Jul 12, 2026
- PastaPastaPasta referenced this in commit d945b2b411 on Jul 14, 2026
- PastaPastaPasta referenced this in commit 4a0fcf61f1 on Jul 14, 2026
- PastaPastaPasta referenced this in commit 69236c78fc on Jul 14, 2026
- PastaPastaPasta referenced this in commit db6bac79f6 on Jul 14, 2026
- PastaPastaPasta referenced this in commit 930546d6d1 on Jul 20, 2026
- PastaPastaPasta referenced this in commit 70728886be on Jul 20, 2026
- bitcoin locked this on Jul 30, 2026