Skip to content

Conversation

@jsonbailey
Copy link
Contributor

@jsonbailey jsonbailey commented Jan 22, 2026

Note

Strengthens test coverage around data store behavior, key handling, persistence, and concurrency.

  • InMemoryFeatureStoreV2: Adds tests for initialized?, string/symbol key compatibility in get, filtering in all, full replacement via set_basis, incremental updates via apply_delta (incl. deletes and error handling), and thread-safety under concurrent reads/writes.
  • DataStore::Store: New spec validates applying change sets (TRANSFER_FULL/TRANSFER_CHANGES/TRANSFER_NONE), DELETE handling, change set broadcasting, and commit semantics with/without persistent store (READ_WRITE vs READ_ONLY), including encoding, upsert usage, and failure propagation.
  • FDv2 persistence: Extends tests to verify delta updates are persisted in READ_WRITE but not READ_ONLY, combined initializer+synchronizer persistence behavior, and presence/behavior of the data store status provider (with or without a persistent store).

Written by Cursor Bugbot for commit af1931b. This will update automatically on new commits. Configure here.

@jsonbailey jsonbailey marked this pull request as ready for review January 22, 2026 20:58
@jsonbailey jsonbailey requested a review from a team as a code owner January 22, 2026 20:58
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.

3 participants