search

sandiegopython / test-driven-django-development

This repository contains materials for a San Diego Python workshop on learning Django with test inspired development
http://test-driven-django-development.readthedocs.org/
Other
102 stars 35 forks source link

Pluralize "Entry" to "Entries" #77

Closed treyhunner closed 10 years ago

treyhunner commented 10 years ago

It looks like #73 contains a fix for this also, but doesn't include tests. I think we should add a test while fixing the problem.

Thoughts?

macro1 commented 10 years ago

As much as I want to say it's excessive... Shouldn't every test tutorial have 100% coverage?

Another option would be to demonstrate how we can solve a minor annoiance correctly with an outside package such as inflect: https://pypi.python.org/pypi/inflect

I just did a quick search, and that's what came up... So don't count that as an endorsement.

treyhunner commented 10 years ago

@macro1 I'd prefer to solve this one manually. That package seems like a general case solution, but this is easy to fix as a one-off.

Yes I agree that all appropriate tests should be written for every tutorial.

treyhunner commented 10 years ago

Merge-ready?