Skip to content

feat: Added transport test for test-plans#27

Merged
turuslan merged 33 commits into
qdrvm:mainfrom
bismuth01:test-plans
Feb 17, 2026
Merged

feat: Added transport test for test-plans#27
turuslan merged 33 commits into
qdrvm:mainfrom
bismuth01:test-plans

Conversation

@bismuth01
Copy link
Copy Markdown
Contributor

@bismuth01 bismuth01 commented Feb 9, 2026 •
edited
Loading

Wrote a ping script that utilises the current available quic-v1 transport protocol with the current test-plans standard.

Tested on machine, through manual docker containers and also after integration in test-plans.
Please wait till I do a final run, there seems to be some yaml parsing issue

bismuth01 and others added 23 commits December 20, 2025 14:58
@bismuth01
fix: Update leanp2p commit hash to working version
a7f4fff
@bismuth01
feat: Add redis feature to vcpkg for test-plans build
8f1365b
@bismuth01
feat: Add ping interop test for dialer mode
193a176
@bismuth01
Merge branch 'qdrvm:main' into test-plans
22875c9
@bismuth01
feat: finished dialer in test plans
97f6d77
@bismuth01
feat: complete basic test-plans transport test
3aada1b
@bismuth01
feat: add dockerfile for test-plans debugging
352659c
@bismuth01
fix: added nasm to dockerfile
1e6e342
@bismuth01
fix: fixed address pushing and dockerfile
7e96fd6
@bismuth01
fix: fixed redis interaction and redis port parsing
52261b5
@bismuth01
refactor: decoupled preset in dockerfile for faster testing
3a9e3d0
@bismuth01
fix: dialer reply status
6529bc9
@bismuth01
feat: tested plaintext quic-v1 test-plans script
bada3ed
@bismuth01
refactor: testing smaller dockerfile
cd7b61a
@bismuth01
fix: fixed builder container
3bc375f
@bismuth01
fix: redis host parsing added
cb78831
@bismuth01
fix: Adding ip in multiaddr
6d377a8
@bismuth01
fix: testTimout to seconds for dialer
8c81fc2
@bismuth01
feat: upgraded transport test script to new test-plans standard
799b2c6
@bismuth01
fix: Converting redisKey to c_str
a99a53e
@bismuth01
fix: moved redis key logging
12df304
@bismuth01
fix: return statements
6d3b52a
@bismuth01
fix: Added automated IP discovery
f54837c
@bismuth01
Copy link
Copy Markdown
Contributor Author

bismuth01 commented Feb 12, 2026

Yes, now it's successfully working, can be merged.

turuslan
turuslan requested changes Feb 13, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@turuslan turuslan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for integrating leanp2p and test-plans

Comment thread example/00-vcpkg-install/vcpkg-overlay/leanp2p/portfile.cmake Outdated
Comment thread test-plans/ping/CMakeLists.txt Outdated
Comment thread test-plans/CMakeLists.txt Outdated
@@ -0,0 +1 @@
add_subdirectory(ping) No newline at end of file
Copy link
Copy Markdown
Contributor

@turuslan turuslan Feb 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add newline at the end of file (github shows them with this red sign, git gui mark them too)

Comment thread CMakeLists.txt Outdated
Comment thread vcpkg.json Outdated
Comment thread test-plans/transport_interop/Dockerfile
&& rm -rf $VCPKG_ROOT/.git

WORKDIR /build
COPY . .
Copy link
Copy Markdown
Contributor

@turuslan turuslan Feb 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does source . expand to project root ".../leanp2p", or test-plans ".../leanp2p/test-plans/ping"?

Copy link
Copy Markdown
Contributor Author

@bismuth01 bismuth01 Feb 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is project root. Need to copy the whole library and then build from root.
I tried creating a minimal project setup of leanp2p in test-plans, that might have been better as it wouldn't have added anymore dependencies like redis or extra code but the I made the decision due to build time.

Vcpkg takes approx 17 to 19 minutes to install and build dependencies when the library is copied inside the docker image. But when leanp2p is also added inside vcpkg, the time shoots to 25+ minutes as it also pulls in leanp2p and builds it.

Comment thread test-plans/ping/ping.cpp Outdated
Comment thread test-plans/ping/ping.cpp Outdated
turuslan and others added 5 commits February 13, 2026 17:09
@turuslan
refactor
3a8a3a1 
- move reusable code to test-plans/common.hpp
- unique_ptr for redis pointers
- redis connect and commands helper functions
- TRY_OR_SL_FATAL to simplify test code
- TestTimeout accounts for elapsed time
@bismuth01
fix: reference update to leanp2p
3e0206d
@bismuth01
refactor: changing test-plans names for readability
6aa3e5a
@bismuth01
refactor: Updating CMake presets for test-plans
bdea41d
@bismuth01
fix: update CMake list option
a28bc4a
@bismuth01
Copy link
Copy Markdown
Contributor Author

bismuth01 commented Feb 17, 2026

I have updated the repo as per your requirements, kindly check if anything else is needed.

@turuslan turuslan enabled auto-merge (squash) February 17, 2026 16:43
@turuslan turuslan merged commit 344c66e into qdrvm:main Feb 17, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@turuslan turuslan turuslan approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bismuth01 @turuslan