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

network/tests: Add conformance testing for litep2p and libp2p #7361

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

network/tests: Add conformance testing for litep2p and libp2p #7361

wants to merge 18 commits into from

Conversation

lexnv
Copy link
Contributor

@lexnv lexnv commented Jan 28, 2025

This PR implements conformance tests between our network backends (litep2p and libp2p).

The PR creates a setup for extending testing in the future, while implementing the following tests:

  • connectivity check: Connect litep2p -> libp2p and libp2p -> litep2p
  • request response check: Send 32 requests from one backend to the other
  • notification check: Send 128 ping pong notifications and 128 from one backend to the other

cc @paritytech/networking

lexnv added 7 commits January 28, 2025 14:34
@lexnv
network/tests: Add conformance testing for litep2p and libp2p
1a3434c 
connectivity

Signed-off-by: Alexandru Vasile <[email protected]>
@lexnv
network/tests: Connect two generic backends together
97c9745 
Signed-off-by: Alexandru Vasile <[email protected]>
@lexnv
network/tests: Add high level connectivity test
165e027 
Signed-off-by: Alexandru Vasile <[email protected]>
@lexnv
network/tests: Check high level req-resp protocol from both ways
427e015 
Signed-off-by: Alexandru Vasile <[email protected]>
@lexnv
network/tests: Connect notifications and ping pong between backends
5ec0fba 
Signed-off-by: Alexandru Vasile <[email protected]>
@lexnv
network/tests: Cleanup imports
89ba9da 
Signed-off-by: Alexandru Vasile <[email protected]>
@lexnv
network/tests: Disable tracing for tests
47b0601 
Signed-off-by: Alexandru Vasile <[email protected]>
@lexnv lexnv added R0-silent Changes should not be mentioned in any release notes T10-tests This PR/Issue is related to tests. labels Jan 28, 2025
@lexnv lexnv self-assigned this Jan 28, 2025
@lexnv lexnv enabled auto-merge February 4, 2025 16:41
@paritytech-workflow-stopper
Copy link

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/13154495481
Failed job name: test-linux-stable-no-try-runtime

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dmitry-markin dmitry-markin dmitry-markin approved these changes

@bkchr bkchr bkchr approved these changes

Assignees

@lexnv lexnv

Labels
R0-silent Changes should not be mentioned in any release notes T10-tests This PR/Issue is related to tests.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lexnv @bkchr @dmitry-markin