test: refactor: make chain_dir consistent with CreateBaseChainParams #36320

pull ekzyis wants to merge 1 commits into bitcoin:master from ekzyis:test-mainnet-chain-dir changing 5 files +10 −5
  1. ekzyis commented at 5:57 PM on September 23, 2026: contributor

    chain_dir returned the chain name for every non-signet chain, but mainnet stores its data in the top-level data directory. Mainnet tests worked around this by setting the chain to the empty string, which doubles as the datadir name.

    Instead of this workaround, map "main" to an empty datadir, and translate it to no chain argument and no config section in write_config, so tests can name the chain the way the rest of the codebase does.

    This was mentioned in #34566 (review).

  2. test: refactor: make chain_dir consistent with CreateBaseChainParams
    chain_dir returned the chain name for every non-signet chain, but
    mainnet stores its data in the top-level data directory. Mainnet tests
    worked around this by setting the chain to the empty string, which
    doubles as the datadir name.
    
    Instead of this workaround, map "main" to an empty datadir, and
    translate it to no chain argument and no config section in write_config,
    so tests can name the chain the way the rest of the codebase does.
    0cea924d33
  3. DrahtBot added the label Tests on Sep 23, 2026
  4. DrahtBot commented at 5:58 PM on September 23, 2026: 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/36320.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

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

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

  5. DrahtBot added the label CI failed on Sep 23, 2026
  6. DrahtBot removed the label CI failed on Sep 23, 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-09-24 13:51 UTC

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