test: Remove feature_txindex_compatibility.py in v27 #28421

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

   ---

7. Julio-Rats referenced this in commit a3670b2273 on Jan 26, 2024
8. bitcoin locked this on Oct 29, 2024

Contributors
stickies-v maflcko

Labels
Tests good first issue

Milestone
27.0