search

mozilla / django-browserid

Django application for adding BrowserID support.
Mozilla Public License 2.0
180 stars 80 forks source link

Refactor all tests to be class-based #199

Closed Osmose closed 10 years ago

Osmose commented 10 years ago

For consistency's sake, we should ensure that all of our tests are defined on a TestCase class. This also opens up the opportunity to use TestCase classes that speed up test runs that require the database, so we should probably use a base TestCase in django_browserid.tests that all of our tests will subclass.

Osmose commented 10 years ago

As noted by @willkg, we should probably take this opportunity to use docstrings instead of comments to describe our tests, and use this nifty trick from @andymckay to disable replacing the test names with the docstring.