Skip to content

test: add wallet creation SQLite lifecycle integration coverage#94

Open
j-kon wants to merge 4 commits into
bitcoindevkit:mainfrom
j-kon:codex/wallet-creation-integration
Open

test: add wallet creation SQLite lifecycle integration coverage#94
j-kon wants to merge 4 commits into
bitcoindevkit:mainfrom
j-kon:codex/wallet-creation-integration

Conversation

@j-kon

@j-kon j-kon commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a deterministic offline integration test for fresh SQLite wallet creation
  • persist external and internal derivation state, dispose the original resources, and reload with a new persister
  • verify network, zero balance, derivation indexes, and address scripts after reopening
  • reuse the repository's retry cleanup pattern for cross-platform SQLite file handling

Closes #93

Why

Existing tests cover wallet constructors, loading a prebuilt SQLite fixture, and broad randomized or network-sync flows. This adds focused coverage for the create-to-persist-to-reload lifecycle against a previously nonexistent database.

Validation

  • red check: without wallet.persist, reload restored derivation index 0 instead of the expected 1
  • dart format --output=none --set-exit-if-changed lib test example
  • dart analyze --fatal-infos --fatal-warnings lib test example
  • dart test test/integration/wallet_creation_test.dart
  • dart test (23 passed, 5 live-network tests skipped by their environment gate)

@j-kon j-kon marked this pull request as ready for review June 23, 2026 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test: add wallet creation SQLite lifecycle integration coverage

1 participant