test: add wallet creation SQLite lifecycle coverage#94
Conversation
|
This seems like a good idea I think. |
|
i'm still looking at this, though upon first glance I don't know if your intention is for this to be an integration test or simple offline persistence test... our test/integration/ suite are env-gated readme, this file lives there and carry the integration tag but isn't. I'd like some clarity on this before proceeding to review. |
08997e5 to
cb61e9e
Compare
|
Updated this based on the review feedback. The SQLite wallet creation lifecycle coverage has been moved out of I also removed the integration tag and updated the PR wording to describe this as offline SQLite lifecycle/persistence coverage rather than live env-gated integration coverage. Validation run:
|
|
Updated the SQLite lifecycle coverage with another cleanup pass. Changes:
The test remains deterministic and offline under the normal Validation:
|
Summary
test/Closes #93
Why
Existing tests cover wallet constructors, loading a prebuilt SQLite fixture, and broad randomized or live-network flows. This adds focused offline coverage for the create-to-persist-to-reload lifecycle against a previously nonexistent SQLite database.
Validation
dart format --output=none --set-exit-if-changed lib test exampledart analyze --fatal-infos --fatal-warnings lib test exampledart test test/wallet_creation_sqlite_lifecycle_test.dartdart test(26 passed, 5 live-network tests skipped by their environment gate)