From e2ae527f3a82c2855f7f4bacac8b72c1853bea49 Mon Sep 17 00:00:00 2001 From: benk10 Date: Tue, 11 Feb 2025 18:46:55 -0500 Subject: [PATCH] Add Keeper to default coordinators list Add Bitcoin Keeper to the list of coordinators enabled by default. --- src/seedsigner/models/settings_definition.py | 3 ++- tests/test_controller.py | 2 +- tests/test_flows_seed.py | 4 ++-- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/src/seedsigner/models/settings_definition.py b/src/seedsigner/models/settings_definition.py index 08652519..d28becb3 100644 --- a/src/seedsigner/models/settings_definition.py +++ b/src/seedsigner/models/settings_definition.py @@ -40,10 +40,10 @@ class SettingsConstants: COORDINATOR__KEEPER = "kpr" ALL_COORDINATORS = [ (COORDINATOR__BLUE_WALLET, "BlueWallet"), + (COORDINATOR__KEEPER, "Keeper"), (COORDINATOR__NUNCHUK, "Nunchuk"), (COORDINATOR__SPARROW, "Sparrow"), (COORDINATOR__SPECTER_DESKTOP, "Specter Desktop"), - (COORDINATOR__KEEPER, "Keeper"), ] LOCALE__ARABIC = "ar" @@ -410,6 +410,7 @@ class SettingsDefinition: selection_options=SettingsConstants.ALL_COORDINATORS, default_value=[ SettingsConstants.COORDINATOR__BLUE_WALLET, + SettingsConstants.COORDINATOR__KEEPER, SettingsConstants.COORDINATOR__NUNCHUK, SettingsConstants.COORDINATOR__SPARROW, SettingsConstants.COORDINATOR__SPECTER_DESKTOP, diff --git a/tests/test_controller.py b/tests/test_controller.py index d01a54cf..ba6db226 100644 --- a/tests/test_controller.py +++ b/tests/test_controller.py @@ -91,7 +91,7 @@ def test_missing_settings_get_defaults(self): assert controller.settings.get_value(SettingsConstants.SETTING__LOCALE) == SettingsConstants.LOCALE__ENGLISH assert controller.settings.get_value(SettingsConstants.SETTING__WORDLIST_LANGUAGE) == SettingsConstants.WORDLIST_LANGUAGE__ENGLISH assert controller.settings.get_value(SettingsConstants.SETTING__PERSISTENT_SETTINGS) == SettingsConstants.OPTION__DISABLED - assert controller.settings.get_value(SettingsConstants.SETTING__COORDINATORS) == [i for i,j in SettingsConstants.ALL_COORDINATORS if i!="kpr"] + assert controller.settings.get_value(SettingsConstants.SETTING__COORDINATORS) == [i for i,j in SettingsConstants.ALL_COORDINATORS] assert controller.settings.get_value(SettingsConstants.SETTING__BTC_DENOMINATION) == SettingsConstants.BTC_DENOMINATION__THRESHOLD # Advanced Settings defaults diff --git a/tests/test_flows_seed.py b/tests/test_flows_seed.py index c1bcc0bb..30deb7ba 100644 --- a/tests/test_flows_seed.py +++ b/tests/test_flows_seed.py @@ -317,8 +317,8 @@ def test_export_xpub_custom_derivation_flow(self): script_type = ButtonOption(self.settings.get_multiselect_value_display_names(SettingsConstants.SETTING__SCRIPT_TYPES)[2], return_data=custom_derivation) specter = SettingsConstants.COORDINATOR__SPECTER_DESKTOP - assert SettingsConstants.ALL_COORDINATORS[3][0] == specter - coordinator = ButtonOption(self.settings.get_multiselect_value_display_names(SettingsConstants.SETTING__COORDINATORS)[3], return_data=specter) + assert SettingsConstants.ALL_COORDINATORS[4][0] == specter + coordinator = ButtonOption(self.settings.get_multiselect_value_display_names(SettingsConstants.SETTING__COORDINATORS)[4], return_data=specter) self.run_sequence( initial_destination_view_args=dict(seed_num=0),