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

The Dispersy tracker stress test could be part of our tests #542

Open
devos50 opened this issue May 6, 2017 · 1 comment
Open

The Dispersy tracker stress test could be part of our tests #542

devos50 opened this issue May 6, 2017 · 1 comment

Comments

@devos50
Copy link
Contributor

devos50 commented May 6, 2017

When refactoring the bootstrap tests, I noticed that we have a stress test that sends a bunch of introduction requests to a specific tracker. I think this test can be a valuable addition to our testing suite. I'm talking about this test: https://github.com/Tribler/dispersy/blob/devel/tests/test_bootstrap.py#L242. Currently, this test is disabled and requires some work to make it functional again.

If we can reuse this test, we could modify the code such that it starts a local tracker (instead of using one of our live trackers) and sends request to that tracker. We already have some code to start a tracker in the same file.

@qstokkink
Copy link
Contributor

I agree with the reason for disabling it though (The stress test is not actually a unittest). I think this is more of a Gumby integration test than a Dispersy unit test. In fact, I believe the entirety of test_bootstap.py should be moved to Gumby.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants