-
Notifications
You must be signed in to change notification settings - Fork 10
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
Testing #54
Comments
👍 👍 👍 👍 👍 👍 👍 👍 👍 👍 👍 👍 👍 👍 👍 👍 👍 👍 👍 👍 if you promise to write tests I would do the setup for you |
Hm...I'm not sure how to structure our tests. Ideally, the Flask-side should be a pretty transparent API for querying our database. Results really depend on what's in the database. Not sure how useful tests are for something like this. I think testing is most needed for the front-end react, but I'm not sure how to write tests for that. Any ideas? |
A common strategy that I use with Ci is keeping a database dump so that tests can be run against data. I don't have a ton of experience with frontend testing but I think looking through this would be a good start: https://facebook.github.io/react/docs/test-utils.html |
Alright. I'll start working on this. |
If we could set up tests, we can set up Travis to run some automated tests for us to catch any bugs before merging into master.
The text was updated successfully, but these errors were encountered: