Fixed #21015 -- Fixed MigrationLoader when importlib.import_module returns a file module or an empty directory.

4 files changed, 16 insertions, 0 deletions

diff --git a/tests/migrations/faulty_migrations/__init__.py b/tests/migrations/faulty_migrations/__init__.py
new file mode 100644
index 0000000000..e69de29bb2
--- /dev/null
+++ b/tests/migrations/faulty_migrations/__init__.py
diff --git a/tests/migrations/faulty_migrations/file.py b/tests/migrations/faulty_migrations/file.py
new file mode 100644
index 0000000000..e69de29bb2
--- /dev/null
+++ b/tests/migrations/faulty_migrations/file.py
diff --git a/tests/migrations/faulty_migrations/namespace/foo/__init__.py b/tests/migrations/faulty_migrations/namespace/foo/__init__.py
new file mode 100644
index 0000000000..e69de29bb2
--- /dev/null
+++ b/tests/migrations/faulty_migrations/namespace/foo/__init__.py
diff --git a/tests/migrations/test_loader.py b/tests/migrations/test_loader.py
index 6810089603..c7fe16cf9a 100644
--- a/tests/migrations/test_loader.py
+++ b/tests/migrations/test_loader.py
@@ -1,8 +1,11 @@
+from unittest import skipIf
+
 from django.test import TestCase
 from django.test.utils import override_settings
 from django.db import connection
 from django.db.migrations.loader import MigrationLoader, AmbiguityError
 from django.db.migrations.recorder import MigrationRecorder
+from django.utils import six
 
 
 class RecorderTests(TestCase):
@@ -84,3 +87,16 @@ class LoaderTests(TestCase):
         with override_settings(MIGRATION_MODULES={"migrations": "migrations.faulty_migrations.import_error"}):
             with self.assertRaises(ImportError):
                 migration_loader.load_disk()
+
+    def test_load_module_file(self):
+        migration_loader = MigrationLoader(connection)
+
+        with override_settings(MIGRATION_MODULES={"migrations": "migrations.faulty_migrations.file"}):
+            migration_loader.load_disk()
+
+    @skipIf(six.PY2, "PY2 doesn't load empty dirs.")
+    def test_load_empty_dir(self):
+        migration_loader = MigrationLoader(connection)
+
+        with override_settings(MIGRATION_MODULES={"migrations": "migrations.faulty_migrations.namespace"}):
+            migration_loader.load_disk()