test: add test for corrupt wallet bdb logs#20974
test: add test for corrupt wallet bdb logs#20974ryanofsky wants to merge 3 commits intobitcoin:masterfrom
Conversation
|
Opened as draft PR to see if tests will pass now. If they do then #19034 can be closed and this can be reviewed and merged. Otherwise some more debugging will be required. |
|
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ReviewsSee the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update. ConflictsNo conflicts as of last run. |
|
Approach ACK |
|
I'm not sure that the issue this is trying to fix is really an issue worth spending time on since we have BDB slated for removal. |
|
Closing as this has been open as a draft for more than 2 years with basically no activity. The tests do not pass, nor does it appear that the issue has been fixed or will be fixed. The planned removal of BDB will also obsolete this test. |
5 participants
This PR adds a wallet test testing handling of corrupt BDB logs. It restores commit c6a2c265f5d0b424fc8226c2f3426e82856779d0 from #18907 which was dropped because it triggered a memory leak, which is reported as #19034