From 762788efa3428f02520076cf7fcb1ad4d2523554 Mon Sep 17 00:00:00 2001 From: Ayush Baid Date: Sun, 29 Dec 2019 15:30:38 -0500 Subject: [PATCH] Fixing the test for merging two sets in the DSF I am not super sure with the code, but I think the intention behind the test is to check pair1 with pair2 after it has been merged. Hence changing it. --- cython/gtsam/tests/test_dsf_map.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/cython/gtsam/tests/test_dsf_map.py b/cython/gtsam/tests/test_dsf_map.py index 6eb5510342..b2ede57bf0 100644 --- a/cython/gtsam/tests/test_dsf_map.py +++ b/cython/gtsam/tests/test_dsf_map.py @@ -30,8 +30,10 @@ def key(index_pair): pair1 = gtsam.IndexPair(1, 18) self.assertEqual(key(dsf.find(pair1)), key(pair1)) pair2 = gtsam.IndexPair(2, 2) + + # testing the merge feature of dsf dsf.merge(pair1, pair2) - self.assertTrue(dsf.find(pair1), dsf.find(pair1)) + self.assertEquals(key(dsf.find(pair1)), key(dsf.find(pair2))) if __name__ == '__main__':