django-migration-zero 2.0.3__tar.gz → 2.3.2__tar.gz

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/.ambient-package-update/metadata.py

@@ -8,10 +8,11 @@ from ambient_package_update.metadata.constants import (
 from ambient_package_update.metadata.maintainer import PackageMaintainer
 from ambient_package_update.metadata.package import PackageMetadata
 from ambient_package_update.metadata.readme import ReadmeContent
+from ambient_package_update.metadata.ruff_ignored_inspection import RuffIgnoredInspection
 
 METADATA = PackageMetadata(
     package_name="django_migration_zero",
-    module_name="django_migration_zero",
+    github_package_group="ambient-innovation",
     authors=[
         PackageAuthor(
             name="Ambient Digital",
@@ -26,12 +27,14 @@ METADATA = PackageMetadata(
     has_migrations=True,
     readme_content=ReadmeContent(uses_internationalisation=True),
     dependencies=[
-        "Django>=3.2",
+        f"Django>={SUPPORTED_DJANGO_VERSIONS[0]}",
     ],
     supported_django_versions=SUPPORTED_DJANGO_VERSIONS,
     supported_python_versions=SUPPORTED_PYTHON_VERSIONS,
     optional_dependencies={
         "dev": [*DEV_DEPENDENCIES, "unittest-parametrize~=1.3"],
     },
-    ruff_ignore_list=[],
+    ruff_ignore_list=[
+        RuffIgnoredInspection(key="TRY003", comment="Avoid specifying long messages outside the exception class"),
+    ],
 )

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/.github/workflows/ci.yml

@@ -42,14 +42,14 @@ jobs:
     runs-on: ubuntu-22.04
     strategy:
       matrix:
-        python-version: ['3.8', '3.9', '3.10', '3.11', '3.12', ]
-        django-version: ['42', '50', ]
+        python-version: ['3.9', '3.10', '3.11', '3.12', ]
+        django-version: ['42', '50', '51', ]
 
         exclude:
-          - python-version: '3.8'
-            django-version: 50
           - python-version: '3.9'
             django-version: 50
+          - python-version: '3.9'
+            django-version: 51
 
     steps:
       - uses: actions/checkout@v4
@@ -64,9 +64,9 @@ jobs:
           TOXENV: django${{ matrix.django-version }}
         run: tox
       - name: Upload coverage data
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
-          name: coverage-data
+          name: coverage-data-${{ matrix.python-version }}-${{ matrix.django-version }}
           path: '.coverage*'
 
   coverage:
@@ -84,11 +84,12 @@ jobs:
         run: python -m pip install --upgrade coverage[toml]
 
       - name: Download data
-        uses: actions/download-artifact@v3
+        uses: actions/download-artifact@v4
         with:
-          name: coverage-data
+          pattern: coverage-data*
+          merge-multiple: true
 
-      - name: Combine coverage and fail if it's <100%
+      - name: Combine coverage and fail if it's <100.0%
         run: |
           python -m coverage html --skip-covered --skip-empty
           python -m coverage report --fail-under=100.0

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/.pre-commit-config.yaml

@@ -3,7 +3,7 @@
 
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.4.8
+    rev: v0.6.4
     hooks:
       # Run the Ruff linter.
       - id: ruff
@@ -12,7 +12,7 @@ repos:
       - id: ruff-format
 
   - repo: https://github.com/adamchainz/blacken-docs
-    rev: 1.16.0
+    rev: 1.18.0
     hooks:
       - id: blacken-docs
         additional_dependencies:
@@ -20,14 +20,14 @@ repos:
         files: '(?:README\.md|docs\/.*\.(?:md|rst))'
 
   - repo: https://github.com/asottile/pyupgrade
-    rev: v3.16.0
+    rev: v3.17.0
     hooks:
       - id: pyupgrade
-        args: [ --py38-plus ]
+        args: [ --py39-plus ]
         stages: [ push ]
 
   - repo: https://github.com/adamchainz/django-upgrade
-    rev: 1.18.0
+    rev: 1.21.0
     hooks:
       - id: django-upgrade
         args: [--target-version, "4.2"]

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/CHANGES.md

@@ -1,5 +1,28 @@
 # Changelog
 
+* *2.3.2* (2024-09-11)
+  * Fixed package name
+
+* *2.3.1* (2024-08-12)
+  * Fixed test matrix
+
+* *2.3.0* (2024-08-12)
+  * Added Django 5.1 support
+
+* *2.2.0* (2024-07-18)
+  * Use ORM to reset `django_migrations` table
+  * Lock rows to enable parallel deployments
+  * Dropped Python 3.8 support
+  * Added multiple ruff linters and updated packages
+  * Updated GitHub actions
+  * Added SECURITY.md
+  * Internal updates via `ambient-package-update`
+
+* *2.1.0* (2024-07-09)
+  * Discover apps in nested directories
+  * Use `BASE_DIR` instead of `MIGRATION_ZERO_APPS_DIR`
+  * Fixed bug for migrations with > 4 leading digits
+
 * *2.0.3* (2024-06-21)
   * Linted docs with `blacken-docs` via `ambient-package-update`
 

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: django-migration-zero
-Version: 2.0.3
+Version: 2.3.2
 Summary: Holistic implementation of "migration zero" pattern for Django covering local changes and in-production database adjustments.
 Author-email: Ambient Digital <hello@ambient.digital>
 Description-Content-Type: text/markdown
@@ -9,32 +9,33 @@ Classifier: Environment :: Web Environment
 Classifier: Framework :: Django
 Classifier: Framework :: Django :: 4.2
 Classifier: Framework :: Django :: 5.0
+Classifier: Framework :: Django :: 5.1
 Classifier: Intended Audience :: Developers
 Classifier: License :: OSI Approved :: MIT License
 Classifier: Natural Language :: English
 Classifier: Operating System :: OS Independent
 Classifier: Programming Language :: Python
 Classifier: Programming Language :: Python :: 3
-Classifier: Programming Language :: Python :: 3.8
 Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Topic :: Utilities
-Requires-Dist: Django>=3.2
+Requires-Dist: Django>=4.2
 Requires-Dist: typer~=0.9 ; extra == "dev"
 Requires-Dist: freezegun~=1.3 ; extra == "dev"
 Requires-Dist: pytest-django~=4.7 ; extra == "dev"
 Requires-Dist: pytest-mock~=3.12 ; extra == "dev"
 Requires-Dist: coverage~=7.3 ; extra == "dev"
-Requires-Dist: pre-commit~=3.5 ; extra == "dev"
-Requires-Dist: ruff~=0.1.7 ; extra == "dev"
+Requires-Dist: pre-commit~=3.7 ; extra == "dev"
+Requires-Dist: ruff~=0.4 ; extra == "dev"
 Requires-Dist: sphinx~=7.1 ; extra == "dev"
 Requires-Dist: sphinx-rtd-theme~=2.0 ; extra == "dev"
 Requires-Dist: m2r2==0.3.3.post2 ; extra == "dev"
 Requires-Dist: mistune<2.0.0 ; extra == "dev"
 Requires-Dist: flit~=3.9 ; extra == "dev"
-Requires-Dist: ambient-package-update~=24.6.4 ; extra == "dev"
+Requires-Dist: keyring~=25.2 ; extra == "dev"
+Requires-Dist: ambient-package-update ; extra == "dev"
 Requires-Dist: unittest-parametrize~=1.3 ; extra == "dev"
 Project-URL: Bugtracker, https://github.com/ambient-innovation/django-migration-zero/issues
 Project-URL: Changelog, https://django-migration-zero.readthedocs.io/en/latest/features/changelog.html

django_migration_zero-2.3.2/SECURITY.md

@@ -0,0 +1,18 @@
+# Reporting Security Issues
+
+The maintainers of this package take security bugs seriously. We appreciate your efforts to responsibly
+disclose your findings, and will make every effort to acknowledge your contributions.
+
+To report a security issue, please use the GitHub Security
+Advisory ["Report a Vulnerability"](https://github.com/ambient-innovation/django-migration-zero/security/advisories/new) 
+tab.
+
+The maintainers will send a response indicating the next steps in handling your report. After the initial reply to
+your report, the security team will keep you informed of the progress towards a fix and full announcement, and may ask
+for additional information or guidance.
+
+Report security bugs in third-party modules to the person or team maintaining the module.
+
+## Source
+
+This file was inspired by: https://github.com/electron/electron/blob/main/SECURITY.md

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/django_migration_zero/__init__.py

@@ -3,4 +3,4 @@
 Holistic implementation of "migration zero" pattern for Django covering local changes and in-production database adjustments.
 """
 
-__version__ = "2.0.3"
+__version__ = "2.3.2"

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/django_migration_zero/helpers/file_system.py

@@ -2,62 +2,62 @@ import os
 import re
 from os.path import isdir
 from pathlib import Path
-from typing import List
 
 from django.apps import apps
+from django.apps.config import AppConfig
+from django.conf import settings
 
 from django_migration_zero.helpers.logger import get_logger
-from django_migration_zero.settings import get_migration_zero_apps_dir
 
 logger = get_logger()
 
 
-def build_migration_directory_path(*, app_label: str) -> Path:
+def build_migration_directory_path(*, app_path: Path) -> Path:
     """
     Get directory to the migration directory of a given local Django app
     """
-    return get_migration_zero_apps_dir() / app_label / "migrations"
+    return app_path / "migrations"
 
 
-def get_local_django_apps() -> List[str]:
+def get_local_django_apps() -> list[AppConfig]:
     """
     Iterate all installed Django apps and detect local ones.
     """
     local_apps = []
+    local_path = str(settings.BASE_DIR).replace("\\", "/")
     logger.info("Getting local Django apps...")
     for app_config in apps.get_app_configs():
-        local_path = str(get_migration_zero_apps_dir())
-        app_path = str(Path(app_config.path))
-        if local_path in app_path and "site-packages" not in app_path:
+        app_path = str(app_config.path).replace("\\", "/")
+        if app_path.startswith(local_path) and "site-packages" not in app_path:
             logger.info(f"Local app {app_config.label!r} discovered.")
-            local_apps.append(app_config.label)
+            local_apps.append(app_config)
         else:
             logger.debug(f"App {app_config.label!r} ignored since it's not local.")
 
     return local_apps
 
 
-def has_migration_directory(*, app_label: str) -> bool:
+def has_migration_directory(*, app_path: Path) -> bool:
     """
     Determines if the given Django app has a migrations directory and therefore migrations
     """
-    possible_migration_dir = build_migration_directory_path(app_label=app_label)
+    possible_migration_dir = build_migration_directory_path(app_path=app_path)
     return True if isdir(possible_migration_dir) else False
 
 
-def get_migration_files(*, app_label: str, exclude_initials: bool = False) -> List[str]:
+def get_migration_files(*, app_label: str, app_path: Path, exclude_initials: bool = False) -> list[str]:
     """
     Returns a list of all migration files detected in the given Django app.
     """
     migration_file_list = []
 
     logger.info(f"Getting migration files from app {app_label!r}...")
-    migration_dir = build_migration_directory_path(app_label=app_label)
-    file_pattern = r"^\d{4}_\w+\.py$"
+    migration_dir = build_migration_directory_path(app_path=app_path)
+    file_pattern = r"^\d{4,}_\w+\.py$"
     for filename in os.listdir(migration_dir):
         if re.match(file_pattern, filename):
             if exclude_initials:
-                initial_pattern = r"^\d{4}_initial.py$"
+                initial_pattern = r"^\d{4,}_initial.py$"
                 if re.match(initial_pattern, filename):
                     logger.debug(f"File {filename!r} ignored since it's an initial migration.")
                     continue
@@ -70,11 +70,11 @@ def get_migration_files(*, app_label: str, exclude_initials: bool = False) -> Li
     return migration_file_list
 
 
-def delete_file(*, filename: str, app_label: str, dry_run: bool = False) -> None:
+def delete_file(*, filename: str, app_path: Path, dry_run: bool = False) -> None:
     """
     Physically delete the given file
     """
-    file_path = build_migration_directory_path(app_label=app_label) / filename
+    file_path = build_migration_directory_path(app_path=app_path) / filename
     if not dry_run:
         try:
             os.unlink(file_path)

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/django_migration_zero/managers.py

@@ -1,35 +1,26 @@
+from django.core.exceptions import MultipleObjectsReturned, ObjectDoesNotExist
 from django.db import ProgrammingError, models
 
 from django_migration_zero.exceptions import MissingMigrationZeroConfigRecordError
 from django_migration_zero.helpers.logger import get_logger
 
 
-class MigrationZeroConfigurationQuerySet(models.QuerySet):
-    pass
-
-
 class MigrationZeroConfigurationManager(models.Manager):
     def fetch_singleton(self) -> None:
         logger = get_logger()
         try:
-            number_records = self.count()
+            config_singleton = self.select_for_update().get()
         except ProgrammingError:
             logger.warning(
                 "The migration zero table is missing. This might be ok for the first installation of "
                 '"django-migration-zero" but if you see this warning after that point, something went sideways.'
             )
-            return None
-
-        if number_records > 1:
+            config_singleton = None
+        except MultipleObjectsReturned as e:
             raise MissingMigrationZeroConfigRecordError(
                 "Too many configuration records detected. There can only be one."
-            )
-
-        config_singleton = self.all().first()
-        if not config_singleton:
-            raise MissingMigrationZeroConfigRecordError("No configuration record found in the database.")
+            ) from e
+        except ObjectDoesNotExist as e:
+            raise MissingMigrationZeroConfigRecordError("No configuration record found in the database.") from e
 
         return config_singleton
-
-
-MigrationZeroConfigurationManager = MigrationZeroConfigurationManager.from_queryset(MigrationZeroConfigurationQuerySet)

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/django_migration_zero/services/deployment.py

@@ -1,7 +1,8 @@
 from logging import Logger
 
 from django.core.management import call_command
-from django.db import connections
+from django.db import transaction
+from django.db.migrations.recorder import MigrationRecorder
 
 from django_migration_zero.exceptions import InvalidMigrationTreeError
 from django_migration_zero.helpers.logger import get_logger
@@ -20,6 +21,7 @@ class DatabasePreparationService:
 
         self.logger = get_logger()
 
+    @transaction.atomic
     def process(self):
         self.logger.info("Starting migration zero database adjustments...")
 
@@ -34,8 +36,7 @@ class DatabasePreparationService:
         # records of the other ones
         self.logger.info("Resetting migration history for all apps...")
 
-        with connections["default"].cursor() as cursor:
-            cursor.execute("DELETE FROM `django_migrations`")
+        MigrationRecorder.Migration.objects.all().delete()
 
         # Apply migrations via fake because the database is already up-to-date
         self.logger.info("Populating migration history.")

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/django_migration_zero/services/local.py

@@ -1,3 +1,5 @@
+from pathlib import Path
+
 from django.core.management import call_command
 
 from django_migration_zero.helpers.file_system import (
@@ -23,17 +25,23 @@ class ResetMigrationFiles:
 
     def process(self):
         logger = get_logger()
-        local_app_list = get_local_django_apps()
+        local_apps = get_local_django_apps()
+
+        for app_config in local_apps:
+            app_path = Path(app_config.path)
 
-        for app_label in local_app_list:
-            if not has_migration_directory(app_label=app_label):
-                logger.debug(f"Skipping app {app_label!r}. No migration package detected.")
+            if not has_migration_directory(app_path=app_path):
+                logger.debug(f"Skipping app {app_config.label!r}. No migration package detected.")
                 continue
 
-            migration_file_list = get_migration_files(app_label=app_label, exclude_initials=self.exclude_initials)
+            migration_file_list = get_migration_files(
+                app_label=app_config.label,
+                app_path=app_path,
+                exclude_initials=self.exclude_initials,
+            )
 
             for migration_file in migration_file_list:
-                delete_file(filename=migration_file, app_label=app_label, dry_run=self.dry_run)
+                delete_file(filename=migration_file, app_path=app_path, dry_run=self.dry_run)
 
         logger.info("Recreating new initial migration files...")
         call_command("makemigrations")

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/docs/conf.py

@@ -67,8 +67,6 @@ exclude_patterns = ["_build", "Thumbs.db", ".DS_Store"]
 
 # The theme to use for HTML and HTML Help pages.  See the documentation for
 # a list of builtin themes.
-#
-# html_theme = "alabaster"
 html_theme = "sphinx_rtd_theme"
 html_theme_options = {
     "display_version": False,

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/docs/features/02_configuration.md

@@ -2,17 +2,11 @@
 
 # Settings
 
-This package requires all local Django apps to be inside a single directory. This directory needs to be defined in the
-main django settings.
-
-Usually, you can use the `BASE_DIR` variable from Django's default setup and add a path.
+This package requires all local Django apps to be inside a `BASE_DIR` directory. This directory should be already
+defined in the main django settings.
 
 Note that this variable has to be of type `pathlib.Path`.
 
-```python
-MIGRATION_ZERO_APPS_DIR = BASE_DIR / "apps"
-```
-
 ## Logging
 
 The scripts are quite chatty. If you want to see what's going on under the hood, just configure a quite default-looking

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/manage.py

@@ -7,7 +7,7 @@ import sys
 
 def main():
     """Run administrative tasks."""
-    os.environ.setdefault("DJANGO_SETTINGS_MODULE", "django_migration_zero.settings")
+    os.environ.setdefault("DJANGO_SETTINGS_MODULE", "settings")
     try:
         from django.core.management import execute_from_command_line
     except ImportError as exc:

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/pyproject.toml

@@ -14,13 +14,13 @@ classifiers = [
     "Framework :: Django",
     "Framework :: Django :: 4.2",
     "Framework :: Django :: 5.0",
+    "Framework :: Django :: 5.1",
     "Intended Audience :: Developers",
     "License :: OSI Approved :: MIT License",
     "Natural Language :: English",
     "Operating System :: OS Independent",
     "Programming Language :: Python",
     "Programming Language :: Python :: 3",
-    "Programming Language :: Python :: 3.8",
     "Programming Language :: Python :: 3.9",
     "Programming Language :: Python :: 3.10",
     "Programming Language :: Python :: 3.11",
@@ -30,7 +30,7 @@ classifiers = [
 dynamic = ["version", "description"]
 license = {"file" = "LICENSE.md"}
 dependencies = [
-    'Django>=3.2',
+    'Django>=4.2',
 ]
 
 
@@ -41,14 +41,15 @@ dev = [
    'pytest-django~=4.7',
    'pytest-mock~=3.12',
    'coverage~=7.3',
-   'pre-commit~=3.5',
-   'ruff~=0.1.7',
+   'pre-commit~=3.7',
+   'ruff~=0.4',
    'sphinx~=7.1',
    'sphinx-rtd-theme~=2.0',
    'm2r2==0.3.3.post2',
    'mistune<2.0.0',
    'flit~=3.9',
-   'ambient-package-update~=24.6.4',
+   'keyring~=25.2',
+   'ambient-package-update',
    'unittest-parametrize~=1.3',
 ]
 
@@ -74,18 +75,24 @@ lint.select = [
     "DTZ",     # flake8-datetimez
     "DJ",      # flake8-django
     "TD",      # flake8-to-do
-    "RUF100",  # Removes unnecessary "#noqa" comments
+    "RUF",     # Ruff-specific rules
     "YTT",     # Avoid non-future-prove usages of "sys"
-    # "FBT",     # Protects you from the "boolean trap bug"
     "C4",      # Checks for unnecessary conversions
     "PIE",     # Bunch of useful rules
-    # "SIM",     # Simplifies your code
     "INT",     # Validates your gettext translation strings
     "PERF",    # PerfLint
     "PGH",     # No all-purpose "# noqa" and eval validation
     "PL",      # PyLint
+    "LOG",     # flake8-logging
+    "TID",     # flake8-tidy-imports
+    "PLR2004", # Magic numbers
+    "BLE",     # Checks for except clauses that catch all exceptions
+    "ANN401",  # Checks that function arguments are annotated with a more specific type than Any
+    "TRY",     # Clean try/except
+    "ERA",     # Commented out code
 ]
 lint.ignore = [
+    'TRY003', # Avoid specifying long messages outside the exception class
 ]
 
 # Allow autofix for all enabled rules (when `--fix`) is provided.
@@ -100,16 +107,21 @@ lint.fixable = [
     "DTZ",     # flake8-datetimez
     "DJ",      # flake8-django
     "TD",      # flake8-to-do
-    "RUF100",  # Removes unnecessary "#noqa" comments
+    "RUF",     # Ruff-specific rules
     "YTT",     # Avoid non-future-prove usages of "sys"
-    # "FBT",     # Protects you from the "boolean trap bug"
     "C4",      # Checks for unnecessary conversions
     "PIE",     # Bunch of useful rules
-    # "SIM",     # Simplifies your code
     "INT",     # Validates your gettext translation strings
     "PERF",    # PerfLint
     "PGH",     # No all-purpose "# noqa" and eval validation
     "PL",      # PyLint
+    "LOG",     # flake8-logging
+    "TID",     # flake8-tidy-imports
+    "PLR2004", # Magic numbers
+    "BLE",     # Checks for except clauses that catch all exceptions
+    "ANN401",  # Checks that function arguments are annotated with a more specific type than Any
+    "TRY",     # Clean try/except
+    "ERA",     # Commented out code
 ]
 lint.unfixable = []
 
@@ -141,7 +153,7 @@ exclude = [
 line-length = 120
 
 # Allow unused variables when underscore-prefixed.
-dummy-variable-rgx = "^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$"
+lint.dummy-variable-rgx = "^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$"
 
 # Assume Python 3.12
 target-version = "py312"
@@ -161,22 +173,18 @@ line-ending = "auto"
 
 [tool.tox]
 legacy_tox_ini = """
-[tox]
-envlist = py{38,39,310,311}-django{32,41,42}
-isolated_build = True
-
 [testenv]
 # Django deprecation overview: https://www.djangoproject.com/download/
 deps =
     django42: Django==4.2.*
     django50: Django==5.0.*
+    django51: Django==5.1.*
 extras = dev,
 commands =
     coverage run -m pytest --ds settings tests
 
 [gh-actions]
 python =
-    3.8: py38
     3.9: py39
     3.10: py310
     3.11: py311

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/settings.py

@@ -42,6 +42,7 @@ INSTALLED_APPS = [
     "django_migration_zero",
     # Local
     "testapp",
+    "testapp.nested_app",
 ]
 
 MIDDLEWARE = [
@@ -81,8 +82,6 @@ DATABASES = {
     }
 }
 
-MIGRATION_ZERO_APPS_DIR = BASE_DIR / "django-migration-zero"
-
 # Password validation
 # https://docs.djangoproject.com/en/4.2/ref/settings/#auth-password-validators
 

django_migration_zero-2.3.2/tests/helpers/test_file_system.py

@@ -0,0 +1,81 @@
+from pathlib import Path
+from unittest import mock
+
+from django.apps import apps
+from django.test import TestCase
+
+from django_migration_zero.helpers.file_system import (
+    build_migration_directory_path,
+    delete_file,
+    get_local_django_apps,
+    get_migration_files,
+    has_migration_directory,
+)
+
+
+class HelperFileSystemTest(TestCase):
+    def setUp(self):
+        self.django_migration_zero_config = apps.get_app_config("django_migration_zero")
+        self.testapp_config = apps.get_app_config("testapp")
+        self.nested_app_config = apps.get_app_config("nested_app")
+
+    def test_build_migration_directory_path_regular(self, *args):
+        path = Path("/user/workspace/migration_zero/ilyta")
+        self.assertEqual(
+            build_migration_directory_path(app_path=path), Path("/user/workspace/migration_zero/ilyta/migrations")
+        )
+
+    def test_get_local_django_apps_regular(self):
+        self.assertEqual(
+            get_local_django_apps(),
+            [
+                self.django_migration_zero_config,
+                self.testapp_config,
+                self.nested_app_config,
+            ],
+        )
+
+    def test_has_migration_directory_positive_case(self):
+        app_path = Path(self.django_migration_zero_config.path)
+        self.assertTrue(has_migration_directory(app_path=app_path))
+
+    def test_has_migration_directory_negative_case(self):
+        app_path = Path(self.testapp_config.path)
+        self.assertFalse(has_migration_directory(app_path=app_path))
+
+    def test_has_migration_directory_positive_case_no_migrations(self):
+        app_path = Path(self.nested_app_config.path)
+        self.assertTrue(has_migration_directory(app_path=app_path))
+
+    def test_get_migration_files_regular(self):
+        self.assertEqual(
+            get_migration_files(
+                app_label=self.django_migration_zero_config.label,
+                app_path=Path(self.django_migration_zero_config.path),
+            ),
+            ["0001_initial.py"],
+        )
+
+    def test_get_migration_files_exclude_initials(self):
+        self.assertEqual(
+            get_migration_files(
+                app_label=self.django_migration_zero_config.label,
+                app_path=Path(self.django_migration_zero_config.path),
+                exclude_initials=True,
+            ),
+            [],
+        )
+
+    @mock.patch("django_migration_zero.helpers.file_system.os.unlink")
+    def test_delete_file_regular(self, mocked_unlink):
+        delete_file(app_path=Path(self.testapp_config.path), filename="my_file.py")
+        mocked_unlink.assert_called_once()
+
+    @mock.patch("django_migration_zero.helpers.file_system.os.unlink")
+    def test_delete_file_dry_run(self, mocked_unlink):
+        delete_file(app_path=Path(self.testapp_config.path), filename="my_file.py", dry_run=True)
+        mocked_unlink.assert_not_called()
+
+    @mock.patch("django_migration_zero.helpers.file_system.os.unlink", side_effect=OSError)
+    def test_delete_file_os_error(self, *args):
+        self.assertIsNone(delete_file(app_path=Path(self.testapp_config.path), filename="my_file.py", dry_run=False))

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/tests/services/test_deployment.py

@@ -1,7 +1,9 @@
 from logging import Logger
+from threading import Thread
 from unittest import mock
+from unittest.mock import Mock, call
 
-from django.test import TestCase
+from django.test import TestCase, TransactionTestCase
 from django.utils import timezone
 from freezegun import freeze_time
 
@@ -81,3 +83,56 @@ class DatabasePreparationServiceTest(TestCase):
 
         self.assertEqual(calls[1].args, ("migrate",))
         self.assertEqual(calls[1].kwargs, {"check": True})
+
+
+class DatabasePreparationServiceTestParallelDeployment(TransactionTestCase):
+    @mock.patch("django_migration_zero.services.deployment.get_logger")
+    def test_process_multiple_threads(self, mock_get_logger):
+        """Test parallel deployments to multiple pods."""
+        # Setup
+        mock_logger_info = Mock(return_value=None)
+        mock_get_logger.return_value = Mock(info=mock_logger_info)
+        config, _ = MigrationZeroConfiguration.objects.update_or_create(
+            defaults={
+                "migration_imminent": True,
+                "migration_date": timezone.now().date(),
+            }
+        )
+
+        # Testable
+        number_of_pods = 1
+        threads = [Thread(target=DatabasePreparationService().process) for _ in range(number_of_pods)]
+
+        for thread in threads:
+            thread.start()
+
+        for thread in threads:
+            thread.join()
+
+        # Assertions
+        self.assertEqual(mock_logger_info.call_count, 6 + number_of_pods, msg=mock_logger_info.call_args_list)
+        mock_logger_info.assert_has_calls(
+            [
+                call("Starting migration zero database adjustments..."),
+                call("Resetting migration history for all apps..."),
+                call("Populating migration history."),
+                call("Checking migration integrity."),
+                call("All good."),
+                call("Deactivating migration zero switch in database."),
+                call("Process successfully finished."),
+            ],
+            any_order=True,
+        )
+        self.assertEqual(
+            len(
+                [
+                    mock_call
+                    for mock_call in mock_logger_info.call_args_list
+                    if mock_call == call("Starting migration zero database adjustments...")
+                ]
+            ),
+            number_of_pods,
+        )
+
+        config.refresh_from_db()
+        self.assertFalse(config.migration_imminent)

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/tests/services/test_local.py

@@ -1,5 +1,7 @@
+from pathlib import Path
 from unittest import mock
 
+from django.apps import apps
 from django.test import TestCase
 
 from django_migration_zero.services.local import ResetMigrationFiles
@@ -7,26 +9,39 @@ from django_migration_zero.services.local import ResetMigrationFiles
 
 @mock.patch("django_migration_zero.services.local.delete_file")
 class ResetMigrationFilesTest(TestCase):
+    def setUp(self):
+        self.django_migration_zero_config = apps.get_app_config("django_migration_zero")
+        self.testapp_config = apps.get_app_config("testapp")
+        self.nested_app_config = apps.get_app_config("nested_app")
+
     def test_init_regular(self, *args):
         service = ResetMigrationFiles(dry_run=True, exclude_initials=True)
 
         self.assertTrue(service.dry_run)
         self.assertTrue(service.exclude_initials)
 
-    @mock.patch("django_migration_zero.services.local.get_local_django_apps", return_value=["django_migration_zero"])
     @mock.patch("django_migration_zero.services.local.has_migration_directory", return_value=True)
     def test_process_case_app_with_migration_found(self, mocked_has_migration_directory, *args):
         service = ResetMigrationFiles()
-        service.process()
+
+        with mock.patch(
+            "django_migration_zero.services.local.get_local_django_apps",
+            return_value=[self.django_migration_zero_config],
+        ):
+            service.process()
 
         # Assertion
         mocked_has_migration_directory.assert_called()
 
-    @mock.patch("django_migration_zero.services.local.get_local_django_apps", return_value=["django_migration_zero"])
     @mock.patch("django_migration_zero.services.local.has_migration_directory", return_value=False)
     def test_process_case_app_with_no_migration_found(self, mocked_has_migration_directory, *args):
         service = ResetMigrationFiles()
-        service.process()
+
+        with mock.patch(
+            "django_migration_zero.services.local.get_local_django_apps",
+            return_value=[self.testapp_config, self.nested_app_config],
+        ):
+            service.process()
 
         # Assertion
         mocked_has_migration_directory.assert_called()
@@ -37,7 +52,7 @@ class ResetMigrationFilesTest(TestCase):
 
         # Assertion
         mocked_delete_file.assert_called_with(
-            filename="0001_initial.py", app_label="django_migration_zero", dry_run=False
+            filename="0001_initial.py", app_path=Path(self.django_migration_zero_config.path), dry_run=False
         )
 
     def test_process_delete_file_case_dry_run(self, mocked_delete_file):
@@ -46,7 +61,7 @@ class ResetMigrationFilesTest(TestCase):
 
         # Assertion
         mocked_delete_file.assert_called_with(
-            filename="0001_initial.py", app_label="django_migration_zero", dry_run=True
+            filename="0001_initial.py", app_path=Path(self.django_migration_zero_config.path), dry_run=True
         )
 
     def test_process_delete_file_case_exclude_initials(self, mocked_delete_file):

{django_migration_zero-2.0.3 → django_migration_zero-2.3.2}/tests/test_managers.py

@@ -5,6 +5,7 @@ from django.db import ProgrammingError
 from django.test import TestCase
 
 from django_migration_zero.exceptions import MissingMigrationZeroConfigRecordError
+from django_migration_zero.managers import MigrationZeroConfigurationManager
 from django_migration_zero.models import MigrationZeroConfiguration
 
 
@@ -18,7 +19,7 @@ class MigrationZeroConfigurationManagerTest(TestCase):
     def test_fetch_singleton_singleton_exists_via_migration(self):
         self.assertEqual(MigrationZeroConfiguration.objects.all().count(), 1)
 
-    @mock.patch.object(MigrationZeroConfiguration.objects, "count", side_effect=ProgrammingError)
+    @mock.patch.object(MigrationZeroConfigurationManager, "select_for_update", side_effect=ProgrammingError)
     def test_fetch_singleton_database_error(self, *args):
         self.assertIsNone(MigrationZeroConfiguration.objects.fetch_singleton())
 

django_migration_zero-2.0.3/django_migration_zero/settings.py

@@ -1,19 +0,0 @@
-from pathlib import Path
-
-from django.conf import settings
-
-from django_migration_zero.exceptions import InvalidMigrationAppsDirPathError
-
-
-def get_migration_zero_apps_dir() -> Path:
-    """
-    Helper method to get the settings variable and validate the type
-    """
-    migration_zero_apps_dir = getattr(settings, "MIGRATION_ZERO_APPS_DIR", Path("/"))
-
-    if not isinstance(migration_zero_apps_dir, Path):
-        raise InvalidMigrationAppsDirPathError(
-            'Settings variable "MIGRATION_ZERO_APPS_DIR" has to be of type pathlib.Path.'
-        )
-
-    return migration_zero_apps_dir

django_migration_zero-2.0.3/tests/helpers/test_file_system.py

@@ -1,49 +0,0 @@
-from pathlib import Path
-from unittest import mock
-
-from django.test import TestCase, override_settings
-
-from django_migration_zero.helpers.file_system import (
-    build_migration_directory_path,
-    delete_file,
-    get_local_django_apps,
-    get_migration_files,
-    has_migration_directory,
-)
-
-
-class HelperFileSystemTest(TestCase):
-    @override_settings(MIGRATION_ZERO_APPS_DIR=Path("/user/workspace/migration_zero/"))
-    def test_build_migration_directory_path_regular(self, *args):
-        self.assertEqual(
-            build_migration_directory_path(app_label="ilyta"), Path("/user/workspace/migration_zero/ilyta/migrations")
-        )
-
-    def test_get_local_django_apps_regular(self):
-        self.assertEqual(get_local_django_apps(), ["django_migration_zero", "testapp"])
-
-    def test_has_migration_directory_positive_case(self):
-        self.assertTrue(has_migration_directory(app_label="django_migration_zero"))
-
-    def test_has_migration_directory_negative_case(self):
-        self.assertFalse(has_migration_directory(app_label="testapp"))
-
-    def test_get_migration_files_regular(self):
-        self.assertEqual(get_migration_files(app_label="django_migration_zero"), ["0001_initial.py"])
-
-    def test_get_migration_files_exclude_initials(self):
-        self.assertEqual(get_migration_files(app_label="django_migration_zero", exclude_initials=True), [])
-
-    @mock.patch("django_migration_zero.helpers.file_system.os.unlink")
-    def test_delete_file_regular(self, mocked_unlink):
-        delete_file(app_label="testapp", filename="my_file.py")
-        mocked_unlink.assert_called_once()
-
-    @mock.patch("django_migration_zero.helpers.file_system.os.unlink")
-    def test_delete_file_dry_run(self, mocked_unlink):
-        delete_file(app_label="testapp", filename="my_file.py", dry_run=True)
-        mocked_unlink.assert_not_called()
-
-    @mock.patch("django_migration_zero.helpers.file_system.os.unlink", side_effect=OSError)
-    def test_delete_file_os_error(self, *args):
-        self.assertIsNone(delete_file(app_label="testapp", filename="my_file.py", dry_run=False))

django_migration_zero-2.0.3/tests/test_settings.py

@@ -1,18 +0,0 @@
-from django.conf import settings
-from django.test import TestCase, override_settings
-
-from django_migration_zero.exceptions import InvalidMigrationAppsDirPathError
-from django_migration_zero.settings import get_migration_zero_apps_dir
-
-
-class SettingsTest(TestCase):
-    def test_get_migration_zero_apps_dir_regular(self):
-        self.assertEqual(get_migration_zero_apps_dir(), settings.MIGRATION_ZERO_APPS_DIR)
-
-    @override_settings(MIGRATION_ZERO_APPS_DIR="/usr/path/to/apps")
-    def test_get_migration_zero_apps_dir_wrong_type(self):
-        with self.assertRaisesMessage(
-            InvalidMigrationAppsDirPathError,
-            'Settings variable "MIGRATION_ZERO_APPS_DIR" has to be of type pathlib.Path.',
-        ):
-            get_migration_zero_apps_dir()