Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

p2p sync tests #2394

Open
wants to merge 25 commits into
base: main
Choose a base branch
from
Open

p2p sync tests #2394

wants to merge 25 commits into from

Conversation

CHr15F0x
Copy link
Member

@CHr15F0x CHr15F0x commented Nov 20, 2024 •
edited
Loading

Fixes: #2300

  • fake block generation now includes tries, respective commitments, and signatures
  • some fixtures removed
  • added an integration test with 3 cases:
    • full sync, recoverable errors triggered in each stage (ie. headers, transactions, ...) separately for checkpoint and track sync
    • full sync, interrupted by a fatal error during checkpoint sync
    • full sync, interrupted by a fatal error during tracking sync
  • removed a redundant test
  • other minor fixes and refactors

@CHr15F0x CHr15F0x force-pushed the chris/sync-tests2 branch 7 times, most recently from cd46651 to 26e6870 Compare November 22, 2024 18:34
@CHr15F0x CHr15F0x changed the title Chris/sync tests2 p2p sync tests Nov 22, 2024
@CHr15F0x
feat(state_update): allow StateUpdateRef from mut ref
9be2532
@CHr15F0x
fix(transactions): CalculateHashes reports placeholder peer id
7ca8280
@CHr15F0x
feat(storage): add in_tempdir for specific test cases
52bfa09
@CHr15F0x
chore(bloom): suppress dead code warning in bloom tests
2706d0f
@CHr15F0x
test(fake): add trie generation to fake storage, refactor, update api
8a0c95c
@CHr15F0x
test(sync_handlers): update proptests
3adeb25
@CHr15F0x
test(checkpoint): update tests, remove fixture
7b32c42
@CHr15F0x
chore(sync/state_updates): remove dead code
a2524b9
@CHr15F0x
test(sync/storage_adapters): update tests
7b79858
@CHr15F0x
feat(sync/track): advance next and parent_hash, update tests
41adfef
@CHr15F0x
test(connection/block): update tests
83663a6
@CHr15F0x
test(rpc): update tests
7ba11c4
@CHr15F0x
test(checkpoint/transactions): fix test timeout
608adc6
@CHr15F0x
test(sync): enable verify_tree_hashes in checkpoint and track tests
7e93e7c
@CHr15F0x
refactor(checkpoint): make checkpoint::Sync generic over p2p client
a096c31
@CHr15F0x
refactor(sync): make Sync generic over p2p client
ca97d86
@CHr15F0x
test(sync): add mock
fdeaa48
@CHr15F0x
refactor(sync): make a few items generic over feeder gateway client
360209b
@CHr15F0x
test(checkpoint): update fixture with correct signatures
dbac2ad
@CHr15F0x
test(storage/fake): add signature generation to fake block data gener…
509932f 
…ation
@CHr15F0x
feat(sync): reenable block header signature verification
ec960a3
@CHr15F0x
test(sync): deduplicate fake blocks generation
c48b773
@CHr15F0x CHr15F0x marked this pull request as ready for review November 22, 2024 19:14
@CHr15F0x CHr15F0x requested a review from a team as a code owner November 22, 2024 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Tests for tracking and checkpoint sync
1 participant
@CHr15F0x