django-usda icon indicating copy to clipboard operation
django-usda copied to clipboard

Published 20 hours ago • verified publisher icon notanumber

Missing tests.py

Open notanumber opened this issue 15 years ago • 2 comments

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

notanumber avatar Mar 21 '10 16:03 notanumber

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

ckcollab avatar May 27 '15 00:05 ckcollab

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.

notanumber avatar Jun 16 '15 20:06 notanumber