Commit f95c642
committed
Merge #300: Add utilities to test persistence
abcf868 refactor: remove bdk_testenv dependency (codingp110)
f539909 docs: document persist_test_utils module (codingp110)
a03d3e3 test: use utils to test file_store and rusqlite (codingp110)
62060fd feat: add utilities to test persistence of wallet (codingp110)
Pull request description:
<!-- You can erase any parts of this template not applicable to your Pull Request. -->
### Description
Added some basic functions to test `WalletPersister` impl. Fixes #14 and might help with #234 .
### Changelog notice
```
Added
- functions to test `WalletPersister` impl of custom persistence backends.
- tests for file_store and rusqlite based on persistence testing functions added
```
### Checklists
#### All Submissions:
* [x] I've signed all my commits
* [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md)
* [x] I ran `just p` before pushing
#### New Features:
* [x] I've added tests for the new feature
* [x] I've added docs for the new feature
ACKs for top commit:
ValuedMammal:
ACK abcf868
Tree-SHA512: 854e24ace5366c9bf744db602c99e98849654dc162cd5689b8de49bcdf608a11834875cc9d11086cc5c67ef802eb6ce79c380e56386862644471fbc699fc297aFile tree
4 files changed
+470
-1
lines changed- wallet
- src
- tests
4 files changed
+470
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
| 25 | + | |
| 26 | + | |
25 | 27 | | |
26 | 28 | | |
27 | 29 | | |
| |||
35 | 37 | | |
36 | 38 | | |
37 | 39 | | |
38 | | - | |
| 40 | + | |
39 | 41 | | |
40 | 42 | | |
41 | 43 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
31 | 31 | | |
32 | 32 | | |
33 | 33 | | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
34 | 38 | | |
35 | 39 | | |
36 | 40 | | |
| |||
0 commit comments