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.

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

test that multiple clients can connect #44

Open
Augry2 opened this issue Feb 8, 2024 · 2 comments · May be fixed by #64
Open

test that multiple clients can connect #44

Augry2 opened this issue Feb 8, 2024 · 2 comments · May be fixed by #64
Assignees
Labels
enhancement New feature or request

Comments

@Augry2
Copy link
Contributor

Augry2 commented Feb 8, 2024

Problem Description

create a unit test for making sure that multiple clients can connect to the server simultaneously.

@Augry2 Augry2 added the enhancement New feature or request label Feb 8, 2024
@aeldin aeldin self-assigned this Feb 8, 2024
@kappsegla
Copy link
Contributor

Testing connections from multiple clients means that we will have to start threads and also involve handling incoming connections on a server socket. That might be a bit problematic to implement as unit tests and are more of type integration tests.

@aeldin aeldin removed their assignment Feb 13, 2024
@Rishad11 Rishad11 self-assigned this Feb 22, 2024
@Rishad11 Rishad11 linked a pull request Mar 3, 2024 that will close this issue
@Rishad11
Copy link

Rishad11 commented Mar 4, 2024

We could, perhaps, simulate client connections and avoid the complexities of actual network interactions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants