Skip to content
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

Missing tests.py #2

Open
notanumber opened this issue Mar 21, 2010 · 2 comments
Open

Missing tests.py #2

notanumber opened this issue Mar 21, 2010 · 2 comments
Labels

Comments

@notanumber
Copy link
Owner

Should include tests.py so that users can run automated tests.

@ckcollab
Copy link

Thanks so much for releasing this, I was dreading having to parse all of these complex formatted/related CSV files... I may pick this task up for you!

I haven't used this quite yet, but could you help me identify some quick-and-easy things to test that will give us some nice coverage? Here's my list after a little code review:

Test ideas

Management commands

  • Test importing food group creates appropriate models with proper attributes
  • Test importing nutrient definition creates appropriate models with proper attributes
  • Etc. etc. from here

Views (super basic)

  • Smoke test (at least 200 response) food list
  • Smoke test (at least 200 response) food detail

Models

  • Not sure there's anything that'd be really useful to test here?

Extra

  • Setup circleCI, put nice tests passing logo at top of README

@notanumber
Copy link
Owner Author

Hey, I'd love to see someone pick up the slack on this project. In all honesty, I haven't touched it in years. At this point, I'm not even sure if it runs on a modern Django installation.

You're more than welcome to take over! If you want, feel free to branch the code or I can set you up with commit access if you'd prefer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants