-
-
Notifications
You must be signed in to change notification settings - Fork 407
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
Make all tests offline #593
Comments
I was thinking about the tests yesterday. Some of the tests I have done rely on |
can I take this up ? |
Yeah this issue is still open it would be great if you could work on it. Please let me know if you need any help |
Having just spent a good number of hours on a plane hacking on opsdroid, I can attest to this not being solved 😀 |
It seems there are at least 3 config tests which still try to git pull. We could consider adding a tox job which uses https://github.com/miketheman/pytest-socket to prevent future regressions on this. |
Sounds good. |
Hello, I would like to work on this if its available, thanks |
We have some tests which are checking things like pulling from GitHub, downloading files, installing deps from pypi, etc. This means the tests can break if there is no internet connection.
I've also noticed this happen from Travis where a GitHub url is denied, this may be some kind of rate limiting if lots of people on Travis are hitting GitHub.
We should mock out our tests better so they don't actually connect to the internet. This should also speed them up.
Update
Added list of failing tests so we can tick them off as they are fixed.
The text was updated successfully, but these errors were encountered: