Added DatabaseFeatures.supports_inspectdb.

Needed by MongoDB.

Author: timgraham

django/core/management/commands/inspectdb.py

@@ -44,10 +44,10 @@ def add_arguments(self, parser):
         )
 
     def handle(self, **options):
-        try:
+        if connections[options["database"]].features.supports_inspectdb:
             for line in self.handle_inspection(options):
                 self.stdout.write(line)
-        except NotImplementedError:
+        else:
             raise CommandError(
                 "Database inspection isn't supported for the currently selected "
                 "database backend."

django/db/backends/base/features.py

@@ -400,6 +400,9 @@ class BaseDatabaseFeatures:
     supports_on_delete_db_default = True
     supports_on_delete_db_null = True
 
+    # Does the backend support the inspectdb management command?
+    supports_inspectdb = True
+
     # Collation names for use by the Django test suite.
     test_collations = {
         "ci": None,  # Case-insensitive.

docs/releases/6.1.txt

@@ -367,6 +367,9 @@ backends.
   ``db_on_delete``) as values. ``db_on_delete`` is one of the database-level
   delete options e.g. :attr:`~django.db.models.DB_CASCADE`.
 
+* Set the new ``DatabaseFeatures.supports_inspectdb`` attribute to ``False``
+  if the management command isn't supported.
+
 :mod:`django.contrib.gis`
 -------------------------
 

tests/gis_tests/inspectapp/tests.py

@@ -14,6 +14,7 @@
 from .models import AllOGRFields
 
 
+@skipUnlessDBFeature("supports_inspectdb")
 class InspectDbTests(TestCase):
     def test_geom_columns(self):
         """

tests/inspectdb/tests.py

@@ -2,11 +2,16 @@
 from io import StringIO
 from unittest import mock, skipUnless
 
-from django.core.management import call_command
+from django.core.management import CommandError, call_command
 from django.core.management.commands import inspectdb
 from django.db import connection
 from django.db.backends.base.introspection import TableInfo
-from django.test import TestCase, TransactionTestCase, skipUnlessDBFeature
+from django.test import (
+    TestCase,
+    TransactionTestCase,
+    skipIfDBFeature,
+    skipUnlessDBFeature,
+)
 
 from .models import PeopleMoreData, test_collation
 
@@ -40,6 +45,7 @@ def cursor_execute(*queries):
     return results
 
 
+@skipUnlessDBFeature("supports_inspectdb")
 class InspectDBTestCase(TestCase):
     unique_re = re.compile(r".*unique_together = \((.+),\).*")
 
@@ -519,6 +525,7 @@ def test_same_relations(self):
         )
 
 
+@skipUnlessDBFeature("supports_inspectdb")
 class InspectDBTransactionalTests(TransactionTestCase):
     available_apps = ["inspectdb"]
 
@@ -671,3 +678,14 @@ def test_composite_primary_key_not_unique_together(self):
         out = StringIO()
         call_command("inspectdb", "inspectdb_compositepkmodel", stdout=out)
         self.assertNotIn("unique_together", out.getvalue())
+
+
+@skipIfDBFeature("supports_inspectdb")
+class InspectDBNotSupportedTests(TestCase):
+    def test_not_supported(self):
+        msg = (
+            "Database inspection isn't supported for the currently selected "
+            "database backend."
+        )
+        with self.assertRaisesMessage(CommandError, msg):
+            call_command("inspectdb")