Remove sys.path manipulation from tests

It was allowing the model to be imported twice and fail in the model registry since it would be registered to two different apps.
adamchainz · Jan 7, 2017 · 1ab3609 · 1ab3609
1 parent cb43168
commit 1ab3609
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/runtests.py b/runtests.py
@@ -9,8 +9,7 @@
 
 
 def main():
-    os.environ.setdefault('DJANGO_SETTINGS_MODULE', 'settings')
-    sys.path.insert(0, 'tests')
+    os.environ.setdefault('DJANGO_SETTINGS_MODULE', 'tests.settings')
     return pytest.main()
 
 

diff --git a/tests/settings.py b/tests/settings.py
@@ -5,7 +5,7 @@
 
 INSTALLED_APPS = [
     'corsheaders',
-    'testapp',
+    'tests.testapp',
 ]
 
 DATABASES = {
@@ -15,7 +15,7 @@
     },
 }
 
-ROOT_URLCONF = 'urls'
+ROOT_URLCONF = 'tests.urls'
 
 
 middlewares = list(global_settings.MIDDLEWARE_CLASSES)