]> git.ipfire.org Git - thirdparty/sqlalchemy/sqlalchemy.git/commitdiff
adjust update/delete test for non-ordering
authorMike Bayer <mike_mp@zzzcomputing.com>
Tue, 8 Nov 2022 23:28:13 +0000 (18:28 -0500)
committerMike Bayer <mike_mp@zzzcomputing.com>
Tue, 8 Nov 2022 23:28:13 +0000 (18:28 -0500)
this test sometimes has different ordering when running under
CI

Change-Id: I6dc3e24c6e23dc00a0ee5ba53e489ad813f12c58

test/orm/dml/test_update_delete_where.py

index 9c8809eef952e08c9d1d76add7bef01fae574a89..0dd769be144b2bf30fc4663173749a23b9a1c6ac 100644 (file)
@@ -1230,10 +1230,10 @@ class UpdateDeleteTest(fixtures.MappedTest):
                 .execution_options(synchronize_session="fetch")
                 .returning(User, User.name)
             )
-            expected = [
+            expected = {
                 (User(age=37), "jack"),
                 (User(age=27), "jane"),
-            ]
+            }
         elif crud_type == "delete":
             stmt = (
                 delete(User)
@@ -1241,16 +1241,18 @@ class UpdateDeleteTest(fixtures.MappedTest):
                 .execution_options(synchronize_session="fetch")
                 .returning(User, User.name)
             )
-            expected = [
+            expected = {
                 (User(age=47), "jack"),
                 (User(age=37), "jane"),
-            ]
+            }
         else:
             assert False
 
         result = sess.execute(stmt)
 
-        eq_(result.all(), expected)
+        # note that ComparableEntity sets up __hash__ for mapped objects
+        # to point to the class, so you can test eq with sets
+        eq_(set(result.all()), expected)
 
     @testing.combinations(True, False, argnames="implicit_returning")
     def test_delete_fetch_returning(self, implicit_returning):