flowmade-one/aether
3
0
Fork 0
Code Issues 5 Pull Requests 7 Actions Packages Projects Releases Wiki Activity

[Issue #4] Add SnapshotStore unit tests #31

Merged
HugoNijhuis merged 2 commits from issue-4-snapshotstore-tests into main 2026-01-09 16:37:24 +00:00
Conversation 2 Commits 2 Files Changed 2
+686 -4
2 Commits
Author SHA1 Message Date
HugoNijhuisandClaude Opus 4.5 185c645238 Merge main and address review feedback
CI / build (pull_request) Successful in 17s
Details 
- Add mutex protection to SaveSnapshot and GetLatestSnapshot
- Add nil validation in SaveSnapshot
- Fix key collision in TestSnapshotDataIntegrity_LargeState (use fmt.Sprintf)
- Add concurrent access tests for snapshots

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
 2026-01-09 17:34:22 +01:00
HugoNijhuisandClaude Opus 4.5 0b2b6a3125 Add comprehensive unit tests for SnapshotStore
CI / build (pull_request) Successful in 17s
Details 
Implement SaveSnapshot and GetLatestSnapshot methods on InMemoryEventStore
to satisfy the SnapshotStore interface, and add comprehensive tests covering:

- SaveSnapshot persists snapshots correctly
- GetLatestSnapshot returns most recent snapshot by version
- Behavior when no snapshot exists (returns nil, nil)
- Snapshot versioning is respected across actors
- Version ordering (higher version wins, not insertion order)
- Data integrity for complex, nested, and special character states
- Edge cases: zero version, large version, empty/nil state

Closes #4

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
 2026-01-09 17:13:02 +01:00