kumar303/test-utils — comment on commit eae8041c0405021c8361d3a8ab5f9a6f31390f8c
File:  line 0
By peterbe:

From just looking at the code, this will continue to work as is. As a transition people will have `settings_test.py` and `settings/` in the root and the `settings_test.py` will just continue to do its magic. `setup_test_environment()` is surely called *after* the `django.conf.settings` object has been created.

One potential change could be something like this:

```python

try:
from settings import test as test_settings
except ImportError:
try:
import settings_test as test_settings
warnings.warn("Get your shit together! settings_test.py --> settings/test.py")
except ImportError:
test_settings = None

if test_settings:
for k in [x for x in dir(settings_test) if x.isupper()]:
setattr(settings, k, getattr(settings_test, k))
```

Notice also how I added an extra `isupper()` check. Maybe that's something to worry about some other time.