-
-
Notifications
You must be signed in to change notification settings - Fork 61
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
[FEATURE] Have 100% of test coverage #28
Comments
Would you prefer to have these on one big PR? I took a quick glance at what's missing but I'm not sure how big the end change would be. Also, I would like to start working on these but I would also like to have a better control on what's taken and what's not in case more people wants to collaborate. |
@ab22 I guess one approach that we can do to split the work is to fix each file individually, doing PR's for each file adding a 100% coverage for only that one. what you think? |
Sounds good 👍. I will start with the |
@ab22 sorry for the delay, I've reviewed and commented there 👍 |
Is your feature request related to a problem? Please describe.
This is an improvement on our current test coverage.
Describe the solution you'd like
Recently we have added the test coverage report, considering that this project is focused in generate fake data for tests, is good to have 100% of test coverage.
Describe alternatives you've considered
For normal projects, it's common knowledge that 100% don't guarantee quality, but in this case this project is focused on tests, so is important to us to have at minimum a good test coverage.
Additional context
The confirmation that we reach this goal is having 100% in coveralls
Important! Use the
full-tests-coverage
branch as target of pull requests.The text was updated successfully, but these errors were encountered: