Fixed #34744 -- Prevented recreation of migration for constraints with a dict_keys.

Co-authored-by: Mariusz Felisiak <felisiak.mariusz@gmail.com>

2 files changed, 75 insertions, 0 deletions

diff --git a/tests/migrations/test_autodetector.py b/tests/migrations/test_autodetector.py
index 74892bbf3d..4c91659ca8 100644
--- a/tests/migrations/test_autodetector.py
+++ b/tests/migrations/test_autodetector.py
@@ -2793,6 +2793,43 @@ class AutodetectorTests(BaseAutodetectorTests):
             ["CreateModel", "AddField", "AddConstraint"],
         )
 
+    def test_add_constraints_with_dict_keys(self):
+        book_types = {"F": "Fantasy", "M": "Mystery"}
+        book_with_type = ModelState(
+            "testapp",
+            "Book",
+            [
+                ("id", models.AutoField(primary_key=True)),
+                ("type", models.CharField(max_length=1)),
+            ],
+            {
+                "constraints": [
+                    models.CheckConstraint(
+                        check=models.Q(type__in=book_types.keys()),
+                        name="book_type_check",
+                    ),
+                ],
+            },
+        )
+        book_with_resolved_type = ModelState(
+            "testapp",
+            "Book",
+            [
+                ("id", models.AutoField(primary_key=True)),
+                ("type", models.CharField(max_length=1)),
+            ],
+            {
+                "constraints": [
+                    models.CheckConstraint(
+                        check=models.Q(("type__in", tuple(book_types))),
+                        name="book_type_check",
+                    ),
+                ],
+            },
+        )
+        changes = self.get_changes([book_with_type], [book_with_resolved_type])
+        self.assertEqual(len(changes), 0)
+
     def test_add_index_with_new_model(self):
         book_with_index_title_and_pony = ModelState(
             "otherapp",
diff --git a/tests/queries/test_q.py b/tests/queries/test_q.py
index cdf40292b0..d3bab1f2a0 100644
--- a/tests/queries/test_q.py
+++ b/tests/queries/test_q.py
@@ -200,6 +200,44 @@ class QTests(SimpleTestCase):
         path, args, kwargs = q.deconstruct()
         self.assertEqual(Q(*args, **kwargs), q)
 
+    def test_equal(self):
+        self.assertEqual(Q(), Q())
+        self.assertEqual(
+            Q(("pk__in", (1, 2))),
+            Q(("pk__in", [1, 2])),
+        )
+        self.assertEqual(
+            Q(("pk__in", (1, 2))),
+            Q(pk__in=[1, 2]),
+        )
+        self.assertEqual(
+            Q(("pk__in", (1, 2))),
+            Q(("pk__in", {1: "first", 2: "second"}.keys())),
+        )
+        self.assertNotEqual(
+            Q(name__iexact=F("other_name")),
+            Q(name=Lower(F("other_name"))),
+        )
+
+    def test_hash(self):
+        self.assertEqual(hash(Q()), hash(Q()))
+        self.assertEqual(
+            hash(Q(("pk__in", (1, 2)))),
+            hash(Q(("pk__in", [1, 2]))),
+        )
+        self.assertEqual(
+            hash(Q(("pk__in", (1, 2)))),
+            hash(Q(pk__in=[1, 2])),
+        )
+        self.assertEqual(
+            hash(Q(("pk__in", (1, 2)))),
+            hash(Q(("pk__in", {1: "first", 2: "second"}.keys()))),
+        )
+        self.assertNotEqual(
+            hash(Q(name__iexact=F("other_name"))),
+            hash(Q(name=Lower(F("other_name")))),
+        )
+
     def test_flatten(self):
         q = Q()
         self.assertEqual(list(q.flatten()), [q])