test(storage-engine): add integration tests for WAL, compaction, CDC, and gossip/sharding#3
Open
poyrazK wants to merge 3 commits into
Open
test(storage-engine): add integration tests for WAL, compaction, CDC, and gossip/sharding#3poyrazK wants to merge 3 commits into
poyrazK wants to merge 3 commits into
Conversation
Add wal_compaction_recovery_tests.zig with 5 tests covering: - Committed writes surviving full compaction and restart - Uncommitted writes replaying from WAL on restart - Delete/tombestone survival through compaction cycles - Multi-key data surviving compaction and recovery - Overwritten keys recovering correct latest value Add compaction_transaction_tests.zig with 4 tests covering: - Concurrent reads during compaction - Concurrent writes during compaction - Data correctness after multiple compaction cycles - Tombestone isolation during compaction Add cdc_transaction_tests.zig with 4 tests covering: - Multiple puts emitting multiple CDC events - Delete operations emitting DELETE event type - Update operations producing PUT events - CDC events having incrementing sequence numbers
Add gossip_sharding_rebalance_tests.zig with 5 tests covering: - Node join triggering ring updates - Concurrent writes during node rebalance - Node failure detection and data availability - Multi-node gossip discovery - Data availability during ownership changes Also update run_tests.zig to import all new test files.
…tion - Add trailing newline to cdc_transaction_tests.zig - Add trailing newline to compaction_transaction_tests.zig - Add trailing newline to gossip_sharding_rebalance_tests.zig - Add trailing newline to wal_compaction_recovery_tests.zig - Remove unused getEngineCallbacks() function from compaction_transaction_tests.zig
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
1 participant
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.
Summary
Add 4 new integration test files with 18 total tests covering component interactions:
New Test Files
**** (5 tests)
**** (4 tests)
**** (4 tests)
**** (5 tests)
Modified Files
All 18 new tests pass alongside the existing 120+ tests.