HomePhabricator

test: move logic error test to the end of the suite

Description

test: move logic error test to the end of the suite

Summary:
This test leaves cachedCoinsUsage in a corrupted state which is not supposed to happen because in a real world situation the logic error would cause a bitcoind crash.
Move it to the end of that test suite so it doesn't affect the following tests.

In a future commit the flush_all function will call CCoinsViewCache::SanityCheck which will fail an assertion after the logic error is suppressd.

Depends on D18611

Test Plan: ninja check

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Subscribers: Fabien

Differential Revision: https://reviews.bitcoinabc.org/D18612

Details

Provenance
PiRKAuthored on Sep 12 2025, 19:14
PiRKPushed on Sep 15 2025, 08:34
Reviewer
Restricted Project
Differential Revision
D18612: test: move logic error test to the end of the suite
Parents
rABCe1f6c05f8638: test: Use std::unique_ptr over manual delete in coins_tests
Branches
Unknown
Tags
Unknown