-
-
Notifications
You must be signed in to change notification settings - Fork 508
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 more test coverage #70
Labels
Comments
Yes, I was thinking the same thing to be honest. |
I agree . |
Not closing this until coverage is at least 80% |
@shivankar-madaan is currently working on this. |
This is no longer being worked on if anyone wants to take a look. |
This can be broken up into a couple of tasks.
|
Majority of crawling logic has been moved to https://github.com/KingAkeem/gotor |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We should add more checks to each test case to cover cases more thoroughly. We should add coverage for edge cases, unexpected input and etc.
This is being addressed with #95 but we still need more test for network utilities and we should still cover more test cases.
The text was updated successfully, but these errors were encountered: