test: add functional test for bumpfee with enormous unconfirmed clusters #34961

pull cprkrn wants to merge 1 commits into bitcoin:master from cprkrn:test-feebumper-enormous-cluster changing 1 files +64 −0
  1. cprkrn commented at 3:34 am on March 31, 2026: none

    Adds functional test coverage for the CheckFeeRate() error path where calculateCombinedBumpFee() returns nullopt (line 84 of feebumper.cpp).

    Creates 9 independent 57-tx unconfirmed chains (each under MAX_CLUSTER_COUNT_LIMIT), has the wallet spend from all of them, and verifies bumpfee returns the expected error when GatherClusters() aggregates >500 total txs.

    Closes #34902

  2. DrahtBot added the label Tests on Mar 31, 2026
  3. DrahtBot commented at 3:35 am on March 31, 2026: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

  4. test: add functional test for bumpfee with enormous unconfirmed clusters
    Add test coverage for the CheckFeeRate() error path where
    calculateCombinedBumpFee() returns nullopt due to GatherClusters()
    finding >500 total transactions across input clusters.
    
    The test creates 9 independent unconfirmed chains (57 txs each, under
    the 64-tx cluster limit), has the wallet spend from all of them (which
    fails to broadcast but stays in the wallet), then verifies bumpfee
    returns the expected error.
    
    Closes #34902
    e991d5be3e
  5. cprkrn force-pushed on Mar 31, 2026
  6. cprkrn closed this on Mar 31, 2026


cprkrn DrahtBot

Labels
Tests


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-03-31 12:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me