2 files changed, 34 insertions, 36 deletions
diff --git a/tests/model_fields/models.py b/tests/model_fields/models.py
index 4208454b68..cca2b463a0 100644
--- a/tests/model_fields/models.py
+++ b/tests/model_fields/models.py
@@ -373,14 +373,3 @@ class PrimaryKeyUUIDModel(models.Model):
 
 class RelatedToUUIDModel(models.Model):
     uuid_fk = models.ForeignKey('PrimaryKeyUUIDModel')
-
-
-###############################################################################
-
-# See ticket #24215.
-class AbstractForeignFieldsModel(models.Model):
-    fk = models.ForeignKey('missing.FK')
-    m2m = models.ManyToManyField('missing.M2M', through='missing.Through')
-
-    class Meta:
-        abstract = True
diff --git a/tests/model_fields/tests.py b/tests/model_fields/tests.py
index 31f2cee337..6b63b8f115 100644
--- a/tests/model_fields/tests.py
+++ b/tests/model_fields/tests.py
@@ -5,6 +5,7 @@ import unittest
 from decimal import Decimal
 
 from django import forms, test
+from django.apps import apps
 from django.core import checks, validators
 from django.core.exceptions import ValidationError
 from django.db import IntegrityError, connection, models, transaction
@@ -21,12 +22,11 @@ from django.utils import six
 from django.utils.functional import lazy
 
 from .models import (
-    AbstractForeignFieldsModel, Bar, BigD, BigIntegerModel, BigS, BooleanModel,
-    DataModel, DateTimeModel, Document, FksToBooleans, FkToChar, FloatModel,
-    Foo, GenericIPAddress, IntegerModel, NullBooleanModel,
-    PositiveIntegerModel, PositiveSmallIntegerModel, Post, PrimaryKeyCharModel,
-    RenamedField, SmallIntegerModel, VerboseNameField, Whiz, WhizIter,
-    WhizIterEmpty,
+    Bar, BigD, BigIntegerModel, BigS, BooleanModel, DataModel, DateTimeModel,
+    Document, FksToBooleans, FkToChar, FloatModel, Foo, GenericIPAddress,
+    IntegerModel, NullBooleanModel, PositiveIntegerModel,
+    PositiveSmallIntegerModel, Post, PrimaryKeyCharModel, RenamedField,
+    SmallIntegerModel, VerboseNameField, Whiz, WhizIter, WhizIterEmpty,
 )
 
 
@@ -202,37 +202,46 @@ class ForeignKeyTests(test.TestCase):
         rel_name = Bar._meta.get_field('a').remote_field.related_name
         self.assertIsInstance(rel_name, six.text_type)
 
-    def test_abstract_model_pending_lookups(self):
+    def test_abstract_model_pending_operations(self):
         """
         Foreign key fields declared on abstract models should not add lazy relations to
         resolve relationship declared as string. refs #24215
         """
-        opts = AbstractForeignFieldsModel._meta
-        to_key = ('missing', 'FK')
-        fk_lookup = (AbstractForeignFieldsModel, opts.get_field('fk'))
-        self.assertFalse(
-            any(lookup[0:2] == fk_lookup for lookup in opts.apps._pending_lookups.get(to_key, [])),
-            'Pending lookup added for the abstract model foreign key `to` parameter'
+        pending_ops_before = list(apps._pending_operations.items())
+
+        class AbstractForeignKeyModel(models.Model):
+            fk = models.ForeignKey('missing.FK')
+
+            class Meta:
+                abstract = True
+
+        self.assertIs(AbstractForeignKeyModel._meta.apps, apps)
+        self.assertEqual(
+            pending_ops_before,
+            list(apps._pending_operations.items()),
+            "Pending lookup added for a foreign key on an abstract model"
         )
 
 
 class ManyToManyFieldTests(test.TestCase):
-    def test_abstract_model_pending_lookups(self):
+    def test_abstract_model_pending_operations(self):
         """
         Many-to-many fields declared on abstract models should not add lazy relations to
         resolve relationship declared as string. refs #24215
         """
-        opts = AbstractForeignFieldsModel._meta
-        to_key = ('missing', 'M2M')
-        fk_lookup = (AbstractForeignFieldsModel, opts.get_field('m2m'))
-        self.assertFalse(
-            any(lookup[0:2] == fk_lookup for lookup in opts.apps._pending_lookups.get(to_key, [])),
-            'Pending lookup added for the abstract model many-to-many `to` parameter.'
-        )
-        through_key = ('missing', 'Through')
-        self.assertFalse(
-            any(lookup[0:2] == fk_lookup for lookup in opts.apps._pending_lookups.get(through_key, [])),
-            'Pending lookup added for the abstract model many-to-many `through` parameter.'
+        pending_ops_before = list(apps._pending_operations.items())
+
+        class AbstractManyToManyModel(models.Model):
+            fk = models.ForeignKey('missing.FK')
+
+            class Meta:
+                abstract = True
+
+        self.assertIs(AbstractManyToManyModel._meta.apps, apps)
+        self.assertEqual(
+            pending_ops_before,
+            list(apps._pending_operations.items()),
+            "Pending lookup added for a many-to-many field on an abstract model"
         )