test: add coverage for unknown value to -blockfilterindex #25192

pull brunoerg wants to merge 1 commits into bitcoin:master from brunoerg:2022-05-unknown-blockfilter changing 1 files +5 −0
  1. brunoerg commented at 9:07 PM on May 23, 2022: member

    This PR adds test coverage for the following init error: https://github.com/bitcoin/bitcoin/blob/44037a29129a830fd9c9580f0818387756cfd7d3/src/init.cpp#L844

    Passing an unknown value to -blockfilterindex should throw an error.

  2. test: add coverage for unknown -blockfilterindex 295ff61934
  3. DrahtBot commented at 9:48 PM on May 23, 2022: member

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--174a7506f384e20aa4161008e828411d-->

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #25193 (indexes: Read the locator's top block during init, allow interaction with reindex-chainstate by mzumsande)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  4. DrahtBot added the label Tests on May 23, 2022
  5. dunxen approved
  6. dunxen commented at 7:39 PM on May 24, 2022: contributor

    cr-ACK 295ff61

  7. MarcoFalke merged this on May 25, 2022
  8. MarcoFalke closed this on May 25, 2022

  9. sidhujag referenced this in commit 5ec4eea181 on May 28, 2022
  10. DrahtBot locked this on May 25, 2023
Labels

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-05-02 03:13 UTC

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