#22945 closed Bug (invalid)
Creating a tests directory with __init__ breaks the testrunner
Reported by: | Owned by: | nobody | |
---|---|---|---|
Component: | Testing framework | Version: | 1.7-rc-1 |
Severity: | Normal | Keywords: | testrunner, tests, test |
Cc: | mmitar@… | Triage Stage: | Unreviewed |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
In any given app, creating a directory called 'tests' with an init.py file and a number of test_*.py files which contain TestCases, the testrunner errors when trying to load the 'tests' directory
Traceback (most recent call last): File "/Applications/PyCharm.app/helpers/pycharm/django_test_manage.py", line 127, in <module> utility.execute() File "/Applications/PyCharm.app/helpers/pycharm/django_test_manage.py", line 102, in execute PycharmTestCommand().run_from_argv(self.argv) File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/core/management/commands/test.py", line 50, in run_from_argv super(Command, self).run_from_argv(argv) File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/core/management/base.py", line 288, in run_from_argv self.execute(*args, **options.__dict__) File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/core/management/commands/test.py", line 71, in execute super(Command, self).execute(*args, **options) File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/core/management/base.py", line 337, in execute output = self.handle(*args, **options) File "/Applications/PyCharm.app/helpers/pycharm/django_test_manage.py", line 89, in handle failures = TestRunner(test_labels, verbosity=verbosity, interactive=interactive, failfast=failfast) File "/Applications/PyCharm.app/helpers/pycharm/django_test_runner.py", line 228, in run_tests extra_tests=extra_tests, **options) File "/Applications/PyCharm.app/helpers/pycharm/django_test_runner.py", line 128, in run_tests return super(DjangoTeamcityTestRunner, self).run_tests(test_labels, extra_tests, **kwargs) File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/test/runner.py", line 146, in run_tests suite = self.build_suite(test_labels, extra_tests) File "/Applications/PyCharm.app/helpers/pycharm/django_test_runner.py", line 92, in build_suite suite = super(DjangoTeamcityTestRunner, self).build_suite(*args, **kwargs) File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/test/runner.py", line 95, in build_suite tests = self.test_loader.discover(start_dir=label, **kwargs) File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/loader.py", line 204, in discover tests = list(self._find_tests(start_dir, pattern)) File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/loader.py", line 265, in _find_tests raise ImportError(msg % (mod_name, module_dir, expected_dir)) ImportError: 'tests' module incorrectly imported from '/Users/gene/Python/dd_django/pacer/tests'. Expected '/Users/gene/Python/dd_django/pacer'. Is this module globally installed?
Change History (13)
comment:1 by , 10 years ago
comment:2 by , 10 years ago
Description: | modified (diff) |
---|
The problem seems to be that:
- you have a structure like pacer/tests/tests and both tests directories are on PYTHONPATH, or
- you have a structure like pacer/tests/tests.py and the the tests directory and the tests.py module are on PYTHONPATH.
Can you check that?
comment:3 by , 10 years ago
The folder structure is the following: ~/dd_django/pacer/tests/ where pacer is an app directory with all of the normal stuff (models, admin, views, etc.)
Within /tests there are: init.py and test_models.py
The former is empty. The latter is a number of TestCase subclasses to test models within /pacer
settingy.py is, other than adding the pacer app and a few niceties, out-of-the-box standard.
I'm running in a virtualenv as well, created using virtualenvwrapper. This is standard as well, with no modifications to activate or activate_this.py.
The test runner is being initialized as:
/Users/gene/Python/venvs/dd_django/bin/python /Applications/PyCharm.app/helpers/pycharm/django_test_manage.py test pacer /Users/gene/Python/dd_django
which makes me think that it may be a pycharm problem...
comment:4 by , 10 years ago
Can you reproduce the issue without PyCharm?
Have you tried reporting it to the PyCharm developers?
comment:5 by , 10 years ago
Reproduced bug without any PyCharm interaction, running 'test' admin task from manage.py. Same error.
$ python manage.py test
Traceback (most recent call last):
File "manage.py", line 10, in <module>
execute_from_command_line(sys.argv)
File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/core/management/init.py", line 385, in execute_from_command_line
utility.execute()
File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/core/management/init.py", line 377, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/core/management/commands/test.py", line 50, in run_from_argv
super(Command, self).run_from_argv(argv)
File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/core/management/base.py", line 288, in run_from_argv
self.execute(*args, options.dict)
File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/core/management/commands/test.py", line 71, in execute
super(Command, self).execute(*args, options)
File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/core/management/base.py", line 337, in execute
output = self.handle(*args, options)
File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/core/management/commands/test.py", line 88, in handle
failures = test_runner.run_tests(test_labels)
File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/test/runner.py", line 146, in run_tests
suite = self.build_suite(test_labels, extra_tests)
File "/Users/gene/Python/venvs/dd_django/lib/python2.7/site-packages/django/test/runner.py", line 95, in build_suite
tests = self.test_loader.discover(start_dir=label, kwargs)
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/loader.py", line 204, in discover
tests = list(self._find_tests(start_dir, pattern))
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/loader.py", line 285, in _find_tests
for test in self._find_tests(full_path, pattern):
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/loader.py", line 265, in _find_tests
raise ImportError(msg % (mod_name, module_dir, expected_dir))
ImportError: 'tests' module incorrectly imported from '/Users/gene/Python/dd_django/pacer/tests'. Expected '/Users/gene/Python/dd_django/pacer'. Is this module globally installed?
comment:6 by , 10 years ago
Please use formatting for tracebacks (Aymeric and I edited your previous posts).
I couldn't reproduce this by modifying the tutorial like this:
mysite/ polls/ tests/ __init__.py test_foo.py (originally polls/tests.py)
$ python manage.py test Creating test database for alias 'default'... .......... ---------------------------------------------------------------------- Ran 10 tests in 0.030s
comment:7 by , 10 years ago
Do you have a tests.py or a tests.pyc in the pacer directory by any chance?
I can recreate the error with this structure:
tmp/ __init__.py tests.py tests/ __init__.py test_foo.py
And by running python -m unittest discover tmp
.
comment:8 by , 10 years ago
Yes! Very embarrassed. Can't believe I missed that in the directory tree!
Ran it again and was able to reproduce, and solve the problem.
So, not sure if this is then a bug or not, but that seems to be the issue.
Sorry to send this to the bug tracker without doing enough digging on my own.
comment:9 by , 10 years ago
Resolution: | → invalid |
---|---|
Status: | new → closed |
Closing this issue; prestontimmons' ability to reproduce using python -m unittest discover tmp
reveals that this is a behavior of Python's unittest module discovery, not an issue in Django.
comment:10 by , 9 years ago
Should there perhaps be a ticket for improving the error message this results in?
I just spent a lot of time figuring this out. I won't do that again, but to new developers, this could be a tremendous help the first time.
Perhaps this should be a general check every time Django tries to load models, test, or other. In debug mode only of course.
comment:11 by , 9 years ago
The error message comes from Python, not Django. Without diving into the unittest discovery code, it's not obvious to me if the error message could be improved.
comment:12 by , 9 years ago
I had the same problem, with the following structure:
missing/ __init__.py test.py tests/ __init__.py test_missing.py
comment:13 by , 9 years ago
Cc: | added |
---|
One more note -- changing the 'tests' directory to anything other than 'tests' resolves the issue.