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

Add dedicated checkList unit tests #60

Open
Sean-Der opened this issue Jun 1, 2019 · 1 comment
Open

Add dedicated checkList unit tests #60

Sean-Der opened this issue Jun 1, 2019 · 1 comment
Labels

Comments

@Sean-Der
Copy link
Member

Sean-Der commented Jun 1, 2019

@hugoArregui

This might not be needed, but would be nice to add some basic tests for this (n time should select this pair etc...)

Just a few checks to make sure we don't take relay when we have good host candidates etc..

@hugoArregui
Copy link
Member

Yes, I agree it will be nice to add some test, however in regards to make sure we don't take relay when we have good host candidates that's probably covered, since we only that the best candidate available, no matter the case. Which means we probable need to test the srflx wait time by rejecting host candidates (for example), it seems to me we could use vnet for this.

@stv0g stv0g added the testing label Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants