Merge pull request #194 from Kakadus/main

tests: use `sorted(...)` instead of `.sort()`

Author: bogdanp05

tests/test_asyncio/test_cluster.py

@@ -2620,7 +2620,7 @@ async def test_init_slots_dynamic_startup_nodes(self, dynamic_startup_nodes):
         ]
         startup_nodes = list(rc.nodes_manager.startup_nodes.keys())
         if dynamic_startup_nodes is True:
-            assert startup_nodes.sort() == discovered_nodes.sort()
+            assert sorted(startup_nodes) == sorted(discovered_nodes)
         else:
             assert startup_nodes == ["my@DNS.com:7000"]
 

tests/test_asyncio/test_search.py

@@ -827,7 +827,7 @@ async def test_tags(decoded_r: valkey.Valkey):
         assert 1 == res.total
 
         q2 = await decoded_r.ft().tagvals("tags")
-        assert (tags.split(",") + tags2.split(",")).sort() == q2.sort()
+        assert sorted(tags.split(",") + tags2.split(",")) == sorted(q2)
     else:
         res = await decoded_r.ft().search(q)
         assert 1 == res["total_results"]

tests/test_cluster.py

@@ -2815,7 +2815,7 @@ def test_init_slots_dynamic_startup_nodes(self, dynamic_startup_nodes):
         ]
         startup_nodes = list(rc.nodes_manager.startup_nodes.keys())
         if dynamic_startup_nodes is True:
-            assert startup_nodes.sort() == discovered_nodes.sort()
+            assert sorted(startup_nodes) == sorted(discovered_nodes)
         else:
             assert startup_nodes == ["my@DNS.com:7000"]