Skip to content

test(basichost): add a stress test for reachability logic - #3533

Merged
sukunrt merged 1 commit into
masterfrom
sukun/stress-test-reachability
Jul 29, 2026
Merged

test(basichost): add a stress test for reachability logic#3533
sukunrt merged 1 commit into
masterfrom
sukun/stress-test-reachability

Conversation

@sukunrt

@sukunrt sukunrt commented Jul 28, 2026

Copy link
Copy Markdown
Member

This does catch the bug fixed by #3526 and is exhaustive.

I'd like to add one end to end test with running basichosts with address discovery but making webrtc work with simnet is a lot more work so that'll take more time.

@sukunrt
sukunrt requested a review from lidel July 28, 2026 15:05
@sukunrt
sukunrt force-pushed the sukun/stress-test-reachability branch from d18dbc6 to 36e5b8c Compare July 28, 2026 16:13

@lidel lidel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, fine to merge once CI is green.

@sukunrt
sukunrt force-pushed the sukun/stress-test-reachability branch from 36e5b8c to f032c08 Compare July 29, 2026 14:54
@sukunrt
sukunrt merged commit 726accf into master Jul 29, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants