Merge lp:~jml/libdep-service/django-free-double into lp:libdep-service
Proposed by
Jonathan Lange
Status: | Merged |
---|---|
Approved by: | James Westby |
Approved revision: | 63 |
Merged at revision: | 61 |
Proposed branch: | lp:~jml/libdep-service/django-free-double |
Merge into: | lp:libdep-service |
Diff against target: |
111 lines (+26/-9) 3 files modified
djlibdep/stats.py (+6/-2) djlibdep/test_double.py (+5/-1) djlibdep/tests/test_interface.py (+15/-6) |
To merge this branch: | bzr merge lp:~jml/libdep-service/django-free-double |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
James Westby (community) | Approve | ||
Review via email: mp+129903@code.launchpad.net |
Commit message
Don't import Django in test double
Description of the change
The test double imports django settings in the stats module. This
a) fixes that, by wrapping settings usage in ImportError expection things
b) tests that, by removing DJANGO_
before running the test server
The reason "usage" rather than "import", is because Django is awesome.
To post a comment you must log in.