Fixed #28856 -- Fixed a regression in caching of a GenericForeignKey pointing to a MTI model.

Regression in b9f8635f58ad743995cad2081b3dc395e55761e5.

4 files changed, 17 insertions, 11 deletions

diff --git a/tests/generic_relations_regress/tests.py b/tests/generic_relations_regress/tests.py
index 9add025a46..cfac484053 100644
--- a/tests/generic_relations_regress/tests.py
+++ b/tests/generic_relations_regress/tests.py
@@ -48,6 +48,12 @@ class GenericRelationTests(TestCase):
         TextLink.objects.create(content_object=oddrel)
         oddrel.delete()
 
+    def test_coerce_object_id_remote_field_cache_persistence(self):
+        restaurant = Restaurant.objects.create()
+        CharLink.objects.create(content_object=restaurant)
+        charlink = CharLink.objects.latest('pk')
+        self.assertIs(charlink.content_object, charlink.content_object)
+
     def test_q_object_or(self):
         """
         SQL query parameters for generic relations are properly
diff --git a/tests/model_fields/test_foreignkey.py b/tests/model_fields/test_foreignkey.py
index 1617f7f310..7c99d8e34a 100644
--- a/tests/model_fields/test_foreignkey.py
+++ b/tests/model_fields/test_foreignkey.py
@@ -93,3 +93,13 @@ class ForeignKeyTests(TestCase):
 
         assert_app_model_resolved('model_fields')
         assert_app_model_resolved('tests')
+
+    @isolate_apps('model_fields')
+    def test_to_python(self):
+        class Foo(models.Model):
+            pass
+
+        class Bar(models.Model):
+            fk = models.ForeignKey(Foo, models.CASCADE)
+
+        self.assertEqual(Bar._meta.get_field('fk').to_python('1'), 1)
diff --git a/tests/serializers/test_xml.py b/tests/serializers/test_xml.py
index ea9677d87f..b11cfdd864 100644
--- a/tests/serializers/test_xml.py
+++ b/tests/serializers/test_xml.py
@@ -30,12 +30,6 @@ class XmlSerializerTestCase(SerializersTestBase, TestCase):
 </django-objects>"""  # NOQA
 
     @staticmethod
-    def _comparison_value(value):
-        # The XML serializer handles everything as strings, so comparisons
-        # need to be performed on the stringified value
-        return str(value)
-
-    @staticmethod
     def _validate_output(serial_str):
         try:
             minidom.parseString(serial_str)
diff --git a/tests/serializers/tests.py b/tests/serializers/tests.py
index 0ea60f0f64..66fc2ef40c 100644
--- a/tests/serializers/tests.py
+++ b/tests/serializers/tests.py
@@ -90,10 +90,6 @@ class SerializerRegistrationTests(SimpleTestCase):
 class SerializersTestBase:
     serializer_name = None  # Set by subclasses to the serialization format name
 
-    @staticmethod
-    def _comparison_value(value):
-        return value
-
     def setUp(self):
         sports = Category.objects.create(name="Sports")
         music = Category.objects.create(name="Music")
@@ -193,7 +189,7 @@ class SerializersTestBase:
         self.assertFalse(self._get_field_values(serial_str, 'author'))
 
         for obj in serializers.deserialize(self.serializer_name, serial_str):
-            self.assertEqual(obj.object.pk, self._comparison_value(self.joe.pk))
+            self.assertEqual(obj.object.pk, self.joe.pk)
 
     def test_serialize_field_subset(self):
         """Output can be restricted to a subset of fields"""