-
Notifications
You must be signed in to change notification settings - Fork 27
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
Increase test coverage #78
Comments
PR #76 adds tests for my functions which have routes. |
This is an ongoing process and a good practice for all new code. Most of the new PRs should have them. Is still relevant as an issue? |
I think out of the tasks set here, only one was worked on. This would be adding tests for functions which have routes. The other tasks are mainly security things. |
We should add two more files then: one requesting all our paths without authenticating and another requesting all our paths authenticated. I'll get into that |
Currently we're running into issues because we are not really testing much and end up making breaking changes without noticing. I think we should be working on improving this. I propose unit tests to be build for the following:
I think this would be most of what would be needed.
The text was updated successfully, but these errors were encountered: