Skip to content

Commit ca8d6c8

Browse files
committed
fix more tests
1 parent 59db7d0 commit ca8d6c8

1 file changed

Lines changed: 9 additions & 5 deletions

File tree

irods/test/data_obj_test.py

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1253,7 +1253,10 @@ def test_replica_number(self):
12531253

12541254
# assertions on replicas
12551255
self.assertEqual(len(obj.replicas), number_of_replicas)
1256-
self.assertEqual({repl.number for repl in obj.replicas}, {*range(4)})
1256+
self.assertEqual(
1257+
{repl.number for repl in obj.replicas},
1258+
{*range(len(obj.replicas))}
1259+
)
12571260

12581261
# now trim odd-numbered replicas
12591262
# note (see irods/irods#4861): COPIES_KW might disappear in the future
@@ -1727,11 +1730,12 @@ def test_get_data_objects(self):
17271730
self.assertIsNotNone(obj.replicas[1].__getattribute__(i))
17281731

17291732
# ensure replica info is sensible
1733+
replicas = sorted(obj.replicas, key=lambda repl: repl.number)
17301734
for i in range(2):
1731-
self.assertEqual(obj.replicas[i].number, i)
1732-
self.assertEqual(obj.replicas[i].status, "1")
1733-
self.assertEqual(obj.replicas[i].path.split("/")[-1], filename)
1734-
self.assertEqual(obj.replicas[i].resc_hier.split(";")[-1], ufs_resources[i].name)
1735+
self.assertEqual(replicas[i].number, i)
1736+
self.assertEqual(replicas[i].status, "1")
1737+
self.assertEqual(replicas[i].path.split("/")[-1], filename)
1738+
self.assertEqual(replicas[i].resc_hier.split(";")[-1], ufs_resources[i].name)
17351739

17361740
self.assertEqual(obj.replicas[0].resource_name, ufs_resources[0].name)
17371741
if self.sess.server_version < (4, 2, 0):

0 commit comments

Comments
 (0)