Skip to content

statement-store: add concurrent and multi-peer propagation tests#11611

Merged
AndreiEres merged 9 commits intomasterfrom
denzelpenzel/statement-store-propagation-concurrent
Apr 7, 2026
Merged

statement-store: add concurrent and multi-peer propagation tests#11611
AndreiEres merged 9 commits intomasterfrom
denzelpenzel/statement-store-propagation-concurrent

Conversation

@DenzelPenzel
Copy link
Copy Markdown
Contributor

@DenzelPenzel DenzelPenzel commented Apr 1, 2026

Description

Implement unit tests for "Propagation under normal load" #11534

Summary

  • Add 3 multi-peer propagation tests for the network handler (all-peers delivery, known-statement filtering, same-statement deduplication)
  • Refactor 3 duplicate test builder functions into a single canonical build_handler_multi_peers(n) with thin wrappers

@DenzelPenzel
Copy link
Copy Markdown
Contributor Author

/cmd fmt

@DenzelPenzel DenzelPenzel added the T10-tests This PR/Issue is related to tests. label Apr 1, 2026
Comment thread substrate/client/network/statement/src/lib.rs Outdated
Comment thread substrate/client/network/statement/src/lib.rs Outdated
Comment thread substrate/client/network/statement/src/lib.rs Outdated
Comment thread substrate/client/network/statement/src/lib.rs Outdated
Comment thread substrate/client/network/statement/src/lib.rs
@DenzelPenzel
Copy link
Copy Markdown
Contributor Author

/cmd fmt

@DenzelPenzel
Copy link
Copy Markdown
Contributor Author

/cmd prdoc --audience node_dev --bump patch

@AndreiEres AndreiEres enabled auto-merge April 7, 2026 08:06
@AndreiEres AndreiEres added this pull request to the merge queue Apr 7, 2026
Merged via the queue into master with commit c9e8499 Apr 7, 2026
404 of 431 checks passed
@AndreiEres AndreiEres deleted the denzelpenzel/statement-store-propagation-concurrent branch April 7, 2026 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T10-tests This PR/Issue is related to tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants