no need to modify it if it already equals that

Author: SalmonMode

Diff for: .travis.yml

@@ -1,5 +1,6 @@
 language: python
 dist: xenial
+cache: false
 
 jobs:
   fast_finish: true
@@ -97,7 +98,7 @@ install:
   - pip install tox==3.9.0
 
 script:
-  - tox
+  - tox --force-dep "pytest@git+https://github.com/blueyed/pytest@fixture-stack#egg=pytest"
 
 after_success:
   - |

Diff for: pytest_django/fixtures.py

@@ -36,28 +36,34 @@
 def django_db_modify_db_settings_tox_suffix():
     skip_if_no_django()
 
-    tox_environment = os.getenv("TOX_PARALLEL_ENV")
-    if tox_environment:
-        # Put a suffix like _py27-django21 on tox workers
-        _set_suffix_to_test_databases(suffix=tox_environment)
+    return os.getenv("TOX_PARALLEL_ENV")
 
 
 @pytest.fixture(scope="session")
 def django_db_modify_db_settings_xdist_suffix(request):
     skip_if_no_django()
 
-    xdist_suffix = getattr(request.config, "slaveinput", {}).get("slaveid")
-    if xdist_suffix:
-        # Put a suffix like _gw0, _gw1 etc on xdist processes
-        _set_suffix_to_test_databases(suffix=xdist_suffix)
+    return getattr(request.config, "slaveinput", {}).get("slaveid")
 
 
 @pytest.fixture(scope="session")
 def django_db_modify_db_settings_parallel_suffix(
     django_db_modify_db_settings_tox_suffix,
-    django_db_modify_db_settings_xdist_suffix,
+    django_db_modify_db_settings_xdist_suffix
 ):
     skip_if_no_django()
+    xdist_worker = django_db_modify_db_settings_xdist_suffix
+    tox_environment = django_db_modify_db_settings_tox_suffix
+    suffix_parts = []
+    if tox_environment:
+        # Put a suffix like _py27-django21 on tox workers
+        suffix_parts.append(tox_environment)
+    if xdist_worker:
+        # Put a suffix like _gw0, _gw1 etc on xdist processes
+        suffix_parts.append(xdist_worker)
+    suffix = "_".join(suffix_parts)
+    if suffix:
+        _set_suffix_to_test_databases(suffix=suffix)
 
 
 @pytest.fixture(scope="session")
@@ -182,10 +188,7 @@ def _set_suffix_to_test_databases(suffix):
             continue
 
         db_settings.setdefault("TEST", {})
-        test_db_name = test_name
-        if not test_name.endswith("_{}".format(suffix)):
-            test_db_name = "{}_{}".format(test_name, suffix)
-        db_settings["TEST"]["NAME"] = test_db_name
+        db_settings["TEST"]["NAME"] = "{}_{}".format(test_name, suffix)
 
 
 # ############### User visible fixtures ################

Diff for: pytest_django_test/settings_base.py

@@ -1,5 +1,3 @@
-import os
-
 import django
 
 ROOT_URLCONF = "pytest_django_test.urls"
@@ -14,10 +12,6 @@
 STATIC_URL = "/static/"
 SECRET_KEY = "foobar"
 
-# Used to construct unique test database names to allow detox to run multiple
-# versions at the same time
-db_suffix = "_%s" % os.getuid()
-
 MIDDLEWARE = [
     "django.contrib.sessions.middleware.SessionMiddleware",
     "django.middleware.common.CommonMiddleware",

Diff for: pytest_django_test/settings_mysql_innodb.py

@@ -1,9 +1,9 @@
-from .settings_base import *  # noqa: F403
+from .settings_base import *  # noqa: F401 F403
 
 DATABASES = {
     "default": {
         "ENGINE": "django.db.backends.mysql",
-        "NAME": "pytest_django" + db_suffix,  # noqa: F405
+        "NAME": "pytest_django_should_never_get_accessed",
         "HOST": "localhost",
         "USER": "root",
         "OPTIONS": {"init_command": "SET default_storage_engine=InnoDB"},

Diff for: pytest_django_test/settings_mysql_myisam.py

@@ -1,9 +1,9 @@
-from pytest_django_test.settings_base import *  # noqa: F403
+from .settings_base import *  # noqa: F401 F403
 
 DATABASES = {
     "default": {
         "ENGINE": "django.db.backends.mysql",
-        "NAME": "pytest_django" + db_suffix,  # noqa: F405
+        "NAME": "pytest_django_should_never_get_accessed",
         "HOST": "localhost",
         "USER": "root",
         "OPTIONS": {"init_command": "SET default_storage_engine=MyISAM"},

Diff for: pytest_django_test/settings_postgres.py

@@ -1,4 +1,4 @@
-from pytest_django_test.settings_base import *  # noqa
+from .settings_base import *  # noqa: F401 F403
 
 # PyPy compatibility
 try:
@@ -12,7 +12,7 @@
 DATABASES = {
     "default": {
         "ENGINE": "django.db.backends.postgresql_psycopg2",
-        "NAME": "pytest_django" + db_suffix,  # noqa
+        "NAME": "pytest_django_should_never_get_accessed",
         "HOST": "localhost",
         "USER": "",
     }

Diff for: pytest_django_test/settings_sqlite.py

@@ -1,4 +1,4 @@
-from .settings_base import *  # noqa
+from .settings_base import *  # noqa: F401 F403
 
 DATABASES = {
     "default": {

Diff for: pytest_django_test/settings_sqlite_file.py

@@ -1,6 +1,6 @@
 import tempfile
 
-from pytest_django_test.settings_base import *  # noqa
+from .settings_base import *  # noqa: F401 F403
 
 # This is a SQLite configuration, which uses a file based database for
 # tests (via setting TEST_NAME / TEST['NAME']).
@@ -11,7 +11,7 @@
 DATABASES = {
     "default": {
         "ENGINE": "django.db.backends.sqlite3",
-        "NAME": "/should_never_be_accessed",
+        "NAME": "/pytest_django_should_never_get_accessed",
         "TEST": {"NAME": _filename},
     }
 }

Diff for: tests/test_db_setup.py

@@ -159,32 +159,34 @@ def test_xdist_with_reuse(django_testdir):
 
         from .app.models import Item
 
-        def _check(settings):
+        def _check(settings, worker_id):
             # Make sure that the database name looks correct
             db_name = settings.DATABASES['default']['NAME']
-            assert db_name.endswith('_gw0') or db_name.endswith('_gw1')
-
+            assert db_name == (
+                'test_pytest_django_should_never_get_accessed_inner_inner_{}'
+                .format(worker_id)
+            )
             assert Item.objects.count() == 0
             Item.objects.create(name='foo')
             assert Item.objects.count() == 1
 
 
         @pytest.mark.django_db
-        def test_a(settings):
-            _check(settings)
+        def test_a(settings, worker_id):
+            _check(settings, worker_id)
 
 
         @pytest.mark.django_db
-        def test_b(settings):
-            _check(settings)
+        def test_b(settings, worker_id):
+            _check(settings, worker_id)
 
         @pytest.mark.django_db
-        def test_c(settings):
-            _check(settings)
+        def test_c(settings, worker_id):
+            _check(settings, worker_id)
 
         @pytest.mark.django_db
-        def test_d(settings):
-            _check(settings)
+        def test_d(settings, worker_id):
+            _check(settings, worker_id)
     """
     )
 
@@ -270,7 +272,7 @@ def test_sqlite_database_renamed(self, django_testdir):
             from django.db import connections
 
             @pytest.mark.django_db
-            def test_a():
+            def test_a(worker_id):
                 (conn_db2, conn_default) = sorted(
                     connections.all(),
                     key=lambda conn: conn.alias,
@@ -288,7 +290,7 @@ def test_a():
 
                 assert conn_db2.vendor == 'sqlite'
                 db_name = conn_db2.creation._get_test_db_name()
-                assert db_name.startswith('test_custom_db_name_gw')
+                assert db_name == 'test_custom_db_name_{}'.format(worker_id)
         """
         )
 
@@ -377,13 +379,15 @@ def test_db_with_tox_suffix(self, django_testdir, monkeypatch):
             from django.db import connections
 
             @pytest.mark.django_db
-            def test_inner():
+            def test_inner(worker_id):
 
                 (conn, ) = connections.all()
 
                 assert conn.vendor == 'sqlite'
                 db_name = conn.creation._get_test_db_name()
-                assert db_name.startswith('test_custom_db_name_py37-django22_gw')
+                assert db_name == 'test_custom_db_name_py37-django22_{}'.format(
+                    worker_id,
+                )
         """
         )
 

Diff for: tox.ini

@@ -27,6 +27,8 @@ deps =
     postgres: psycopg2-binary
     coverage: coverage-enable-subprocess
 
+    pytest
+
     pytest41: pytest>=4.1,<4.2
     pytest41: attrs==17.4.0
     xdist: pytest-xdist>=1.15