Repository navigation
test(checkpoints): verify migration from AppDB-written snapshots - #85
Open
eggtart-365 wants to merge 1 commit into
Open
eggtart-365 wants to merge 1 commit into
eggtart-365 wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
The native checkpoint store tests currently seed legacy records directly through IndexedDB. This adds coverage for snapshots actually written by the AppDB 1.2.0 client used in v1.3.2, so changes to the migration path cannot silently break compatibility with released installations.
The test creates a 4 MiB history with the legacy client, then checks that native storage can count and add without reading history values, page through only the requested records, and preserve every existing snapshot by SHA-256. It also includes Unicode document text and literal
true/falsenames, and forbidsgetAll()during native operations.This complements the memory fix in 12b265a; it does not duplicate that fix or change runtime behavior.
Validation:
npm run test:extension— 236 tests passed on Windows with Node 24. Live EasyEDA migration was not exercised by this test.