test(nodejs): use in-memory SQLite databases in e2e tests #796
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
:memory:databases in e2e testsbip39-identity-recovery.test.tsandremote-comms.test.tsNote
Low Risk
Low risk: changes are confined to NodeJS e2e test setup/fixtures and only affect how test databases are created, with no production logic impact. Main risk is subtle differences in SQLite
:memory:lifecycle causing unexpected test behavior if any test relied on persistence across connections.Overview
E2E test isolation improvement: updates NodeJS e2e tests to create SQLite databases with
dbFilename: ':memory:'instead of writing per-test.dbfiles.This applies to the BIP39 identity recovery and remote comms suites, reducing cross-run pollution/flakiness by ensuring each kernel/test uses ephemeral storage.
Written by Cursor Bugbot for commit 539e189. This will update automatically on new commits. Configure here.