zcash_client_sqlite::testing contains a TestBuilder that can be used to produce a TestState, on which a test can operate. The test framework is generic over a block cache, that can optionally be provided with a TestCache implementation for tests that require block scanning. However, it currently hard-codes zcash_client_sqlite::WalletDb as the wallet that is tested.
In order to make the tests reusable with other wallet backends, we should rework TestBuilder etc to be generic over a TestWallet trait, that exposes the necessary operations that we currently depend on concrete WalletDb methods for. The new generic TestBuilder parameter should also be bound on WalletRead, WalletWrite, etc. as appropriate (i.e. only when the test framework requires a given bound).
Once that refactor is done, we can then move the test framework to zcash_client_backend behind the test-dependencies feature flag, and depend on it from zcash_client_sqlite. This will then free us up to subsequently genericise and migrate individual tests from zcash_client_sqlite to zcash_client_backend as we wish.
zcash_client_sqlite::testingcontains aTestBuilderthat can be used to produce aTestState, on which a test can operate. The test framework is generic over a block cache, that can optionally be provided with aTestCacheimplementation for tests that require block scanning. However, it currently hard-codeszcash_client_sqlite::WalletDbas the wallet that is tested.In order to make the tests reusable with other wallet backends, we should rework
TestBuilderetc to be generic over aTestWallettrait, that exposes the necessary operations that we currently depend on concreteWalletDbmethods for. The new genericTestBuilderparameter should also be bound onWalletRead,WalletWrite, etc. as appropriate (i.e. only when the test framework requires a given bound).Once that refactor is done, we can then move the test framework to
zcash_client_backendbehind thetest-dependenciesfeature flag, and depend on it fromzcash_client_sqlite. This will then free us up to subsequently genericise and migrate individual tests fromzcash_client_sqlitetozcash_client_backendas we wish.