Fixed #35402 -- Fixed crash when DatabaseFeatures.django_test_skips references a class in another test module

Author: jonmcfee03

AUTHORS

@@ -521,6 +521,7 @@ answer newbie questions, and generally made Django that much better:
     Jonathan Buchanan <[email protected]>
     Jonathan Daugherty (cygnus) <http://www.cprogrammer.org/>
     Jonathan Feignberg <[email protected]>
+    Jonathan McFee <[email protected]>
     Jonathan Slenders
     Jonny Park <[email protected]>
     Jordan Bae <[email protected]>

django/db/backends/base/creation.py

@@ -347,12 +347,19 @@ def mark_expected_failures_and_skips(self):
         for reason, tests in self.connection.features.django_test_skips.items():
             for test_name in tests:
                 test_case_name, _, test_method_name = test_name.rpartition(".")
+                if not test_method_name.startswith("test"):
+                    test_case_name = test_name
+                    test_method_name = None
                 test_app = test_name.split(".")[0]
                 # Importing a test app that isn't installed raises RuntimeError.
                 if test_app in settings.INSTALLED_APPS:
                     test_case = import_string(test_case_name)
-                    test_method = getattr(test_case, test_method_name)
-                    setattr(test_case, test_method_name, skip(reason)(test_method))
+                    if test_method_name:
+                        test_method = getattr(test_case, test_method_name)
+                        setattr(test_case, test_method_name, skip(reason)(test_method))
+                    else:
+                        setattr(test_case, "__unittest_skip__", True)
+                        setattr(test_case, "__unittest_skip_why__", reason)
 
     def sql_table_creation_suffix(self):
         """

tests/backends/base/test_creation.py

@@ -269,11 +269,11 @@ def test_serialize_db_to_string_base_manager_with_prefetch_related(self):
 
 
 class SkipTestClass:
-    def skip_function(self):
+    def test_skip_function(self):
         pass
 
 
-def skip_test_function():
+def test_skip_test_function():
     pass
 
 
@@ -293,7 +293,7 @@ def test_mark_expected_failures_and_skips(self):
                 "backends.base.test_creation.SkipTestClass",
             },
             "skip test function": {
-                "backends.base.test_creation.skip_test_function",
+                "backends.base.test_creation.test_skip_test_function",
             },
         }
         creation.mark_expected_failures_and_skips()
@@ -306,8 +306,8 @@ def test_mark_expected_failures_and_skips(self):
             SkipTestClass.__unittest_skip_why__,
             "skip test class",
         )
-        self.assertIs(skip_test_function.__unittest_skip__, True)
+        self.assertIs(test_skip_test_function.__unittest_skip__, True)
         self.assertEqual(
-            skip_test_function.__unittest_skip_why__,
+            test_skip_test_function.__unittest_skip_why__,
             "skip test function",
         )