django-migration-zero 2.0.2__tar.gz → 2.1.0__tar.gz

{django_migration_zero-2.0.2 → django_migration_zero-2.1.0}/.ambient-package-update/templates/snippets/installation.tpl

@@ -26,9 +26,13 @@
 - Add this block to your loggers in your main Django `settings.py` to show logs in your console.
 
 ```python
-"django_migration_zero": {
-    "handlers": ["console"],
-    "level": "INFO",
-    "propagate": True,
-},
+LOGGING = {
+    "loggers": {
+        "django_migration_zero": {
+            "handlers": ["console"],
+            "level": "INFO",
+            "propagate": True,
+        },
+    },
+}
 ```

{django_migration_zero-2.0.2 → django_migration_zero-2.1.0}/.pre-commit-config.yaml

@@ -11,6 +11,14 @@ repos:
       # Run the Ruff formatter.
       - id: ruff-format
 
+  - repo: https://github.com/adamchainz/blacken-docs
+    rev: 1.16.0
+    hooks:
+      - id: blacken-docs
+        additional_dependencies:
+        - black==24.4.2
+        files: '(?:README\.md|docs\/.*\.(?:md|rst))'
+
   - repo: https://github.com/asottile/pyupgrade
     rev: v3.16.0
     hooks:

{django_migration_zero-2.0.2 → django_migration_zero-2.1.0}/CHANGES.md

@@ -1,5 +1,13 @@
 # Changelog
 
+* *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`
+
 * *2.0.2* (2024-06-20)
   * Internal updates via `ambient-package-update`
 

{django_migration_zero-2.0.2 → django_migration_zero-2.1.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: django-migration-zero
-Version: 2.0.2
+Version: 2.1.0
 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
@@ -34,7 +34,7 @@ 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.3 ; extra == "dev"
+Requires-Dist: ambient-package-update~=24.6.4 ; 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
@@ -120,11 +120,15 @@ fixing squashed migrations you won't need, you can use the "migration zero" patt
 - Add this block to your loggers in your main Django `settings.py` to show logs in your console.
 
 ```python
-"django_migration_zero": {
-    "handlers": ["console"],
-    "level": "INFO",
-    "propagate": True,
-},
+LOGGING = {
+    "loggers": {
+        "django_migration_zero": {
+            "handlers": ["console"],
+            "level": "INFO",
+            "propagate": True,
+        },
+    },
+}
 ```
 
 ## Contribute

{django_migration_zero-2.0.2 → django_migration_zero-2.1.0}/README.md

@@ -75,11 +75,15 @@ fixing squashed migrations you won't need, you can use the "migration zero" patt
 - Add this block to your loggers in your main Django `settings.py` to show logs in your console.
 
 ```python
-"django_migration_zero": {
-    "handlers": ["console"],
-    "level": "INFO",
-    "propagate": True,
-},
+LOGGING = {
+    "loggers": {
+        "django_migration_zero": {
+            "handlers": ["console"],
+            "level": "INFO",
+            "propagate": True,
+        },
+    },
+}
 ```
 
 ## Contribute

{django_migration_zero-2.0.2 → django_migration_zero-2.1.0}/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.2"
+__version__ = "2.1.0"

{django_migration_zero-2.0.2 → django_migration_zero-2.1.0}/django_migration_zero/helpers/file_system.py

@@ -5,59 +5,60 @@ 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 +71,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.2 → django_migration_zero-2.1.0}/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.2 → django_migration_zero-2.1.0}/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
@@ -28,7 +22,6 @@ LOGGING = {
             "level": "INFO",
             "propagate": True,
         },
-        ...
     },
 }
 ```

{django_migration_zero-2.0.2 → django_migration_zero-2.1.0}/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.2 → django_migration_zero-2.1.0}/pyproject.toml

@@ -48,7 +48,7 @@ dev = [
    'm2r2==0.3.3.post2',
    'mistune<2.0.0',
    'flit~=3.9',
-   'ambient-package-update~=24.6.3',
+   'ambient-package-update~=24.6.4',
    'unittest-parametrize~=1.3',
 ]
 

{django_migration_zero-2.0.2 → django_migration_zero-2.1.0}/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.1.0/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.2 → django_migration_zero-2.1.0}/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.2/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.2/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.2/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()