test: Remove feature_txindex_compatibility.py in v27 #28421

issue stickies-v openend this issue on September 6, 2023
  1. stickies-v commented at 12:46 pm on September 6, 2023: contributor

    Since #28195 got merged, all non-test code relating to the legacy txindex (and its migration) has been removed from master.

    At this point, feature_txindex_compatibility.py is just testing whether -txindex works successfully when a legacy txindex exists, but as there is no overlap in data structures this feels like a redundant test and I think it can be removed from a future version. Since #28195 is included for v26, I’d suggest we remove this test for v27.

    Historical pulls for context: #13033: introduce new txindex db and migration code #22626: remove migration code #23046: add feature_txindex_compatibility.py

  2. maflcko added this to the milestone 27.0 on Sep 11, 2023
  3. maflcko added the label Tests on Sep 11, 2023
  4. maflcko added the label good first issue on Oct 13, 2023
  5. maflcko commented at 10:25 am on October 13, 2023: member

    Useful Skills

    • Compiling Bitcoin Core from source
    • Running the C++ unit tests and the Python functional tests
    • Bitcoin Core test framework understanding

    Guidance for new contributors

    Want to work on this issue?

    For guidance on contributing, please read CONTRIBUTING.md before opening your pull request.

  6. fanquake closed this on Oct 30, 2023


stickies-v maflcko

Labels
Tests good first issue

Milestone
27.0


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: 2024-09-28 22:12 UTC

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