ttests: try to eliminate random failures from mpp_split tests - electrum - Electrum Bitcoin wallet HTML git clone https://git.parazyd.org/electrum DIR Log DIR Files DIR Refs DIR Submodules --- DIR commit a9d0e3fca9dd756b7629890f04429a056d893a88 DIR parent 228c4b45973f266ebcc06d3ebf3d633b41f3f702 HTML Author: SomberNight <somber.night@protonmail.com> Date: Wed, 24 Feb 2021 12:54:43 +0100 ttests: try to eliminate random failures from mpp_split tests closes: #7062 Diffstat: M electrum/mpp_split.py | 2 +- M electrum/tests/test_mpp_split.py | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-) --- DIR diff --git a/electrum/mpp_split.py b/electrum/mpp_split.py t@@ -24,7 +24,7 @@ def unique_hierarchy(hierarchy: Dict[int, List[Dict[bytes, int]]]) -> Dict[int, for config in configs: # config dict can be out of order, so sort, otherwise not unique unique_configs.add(tuple((c, config[c]) for c in sorted(config.keys()))) - for unique_config in unique_configs: + for unique_config in sorted(unique_configs): new_hierarchy[number_parts].append( {t[0]: t[1] for t in unique_config}) return new_hierarchy DIR diff --git a/electrum/tests/test_mpp_split.py b/electrum/tests/test_mpp_split.py t@@ -1,3 +1,5 @@ +import random + import electrum.mpp_split as mpp_split # side effect for PART_PENALTY from electrum.lnutil import NoPathFound t@@ -9,6 +11,8 @@ PART_PENALTY = mpp_split.PART_PENALTY class TestMppSplit(ElectrumTestCase): def setUp(self): super().setUp() + # to make tests reproducible: + random.seed(0) # undo side effect mpp_split.PART_PENALTY = PART_PENALTY self.channels_with_funds = {