-
Notifications
You must be signed in to change notification settings - Fork 378
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 testing #146
Comments
@coreyd303 thanks for submitting this issue! Could you please re-format to something like the following template on #83? |
I could help with this. Current test coverage is only 48%, which is pretty dismal. Anything special I need to know to start? |
Hi - what is the status of this issue? I would be happy to implement Coveralls and write controller/ feature tests, it doesn't look like we have either at the moment. |
We'd love some unit tests if you have some time to jump in. We don't currently have any overall testing plan; we've just been doing feature-based testing on individual release.
|
I think coverall is a great idea. It would provide a nice feedback and motivation. Be sure to add the badge to the README |
Great, I just added Coveralls and have a pull request open. This is my first open source contribution, so I hope I did everything correctly. Will start writing some tests later this week. |
Could use feature testing to help bolster test coverage
The text was updated successfully, but these errors were encountered: