From 0b1786c113253c234443c905979e24e13c861aee Mon Sep 17 00:00:00 2001 From: James Rouzier Date: Mon, 21 Jul 2025 09:52:38 -0400 Subject: [PATCH 1/2] Don't show custom_aliases as a user --- linodecli/configuration/config.py | 2 +- tests/unit/test_configuration.py | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/linodecli/configuration/config.py b/linodecli/configuration/config.py index 969adeb00..4cd54bcfe 100644 --- a/linodecli/configuration/config.py +++ b/linodecli/configuration/config.py @@ -126,7 +126,7 @@ def print_users(self): default_user = self.default_username() for sec in self.config.sections(): - if sec != "DEFAULT": + if sec != "DEFAULT" and sec != "custom_aliases": print(f'{"*" if sec == default_user else " "} {sec}') sys.exit(ExitCodes.SUCCESS) diff --git a/tests/unit/test_configuration.py b/tests/unit/test_configuration.py index 474b083f3..14b704d0c 100644 --- a/tests/unit/test_configuration.py +++ b/tests/unit/test_configuration.py @@ -113,6 +113,7 @@ def test_print_users(self): Test CLIConfig.print_users() """ conf = self._build_test_config() + conf.config.add_section("custom_aliases") f = io.StringIO() @@ -122,6 +123,7 @@ def test_print_users(self): assert err.value.code == 0 assert "* cli-dev" in f.getvalue() + assert "custom_aliases" not in f.getvalue() def test_set_default_user(self): """ From 5880e156efc787a7c135804d5b6294e2f8e71213 Mon Sep 17 00:00:00 2001 From: James Rouzier Date: Mon, 21 Jul 2025 10:22:51 -0400 Subject: [PATCH 2/2] Fix linting errors --- linodecli/configuration/config.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/linodecli/configuration/config.py b/linodecli/configuration/config.py index 4cd54bcfe..0094e00b1 100644 --- a/linodecli/configuration/config.py +++ b/linodecli/configuration/config.py @@ -126,7 +126,7 @@ def print_users(self): default_user = self.default_username() for sec in self.config.sections(): - if sec != "DEFAULT" and sec != "custom_aliases": + if sec not in ("DEFAULT", "custom_aliases"): print(f'{"*" if sec == default_user else " "} {sec}') sys.exit(ExitCodes.SUCCESS)