HSMAUTH: change command name for change management key

Yubico · Aug 7, 2023 · 88191b7 · 88191b7
1 parent 1d5770d
commit 88191b7
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 12 deletions.
diff --git a/tests/device/cli/test_hsmauth.py b/tests/device/cli/test_hsmauth.py
@@ -240,7 +240,7 @@ def test_change_management_key(self, ykman_cli):
         ykman_cli(
             "hsmauth",
             "access",
-            "change",
+            "change-management-key",
             "-m",
             DEFAULT_MANAGEMENT_KEY,
             "-n",
@@ -252,7 +252,7 @@ def test_change_management_key(self, ykman_cli):
             ykman_cli(
                 "hsmauth",
                 "access",
-                "change",
+                "change-management-key",
                 "-m",
                 DEFAULT_MANAGEMENT_KEY,
                 "-n",
@@ -263,7 +263,7 @@ def test_change_management_key(self, ykman_cli):
         ykman_cli(
             "hsmauth",
             "access",
-            "change",
+            "change-management-key",
             "-m",
             NON_DEFAULT_MANAGEMENT_KEY,
             "-n",
@@ -272,16 +272,14 @@ def test_change_management_key(self, ykman_cli):
 
     def test_change_management_key_generate(self, ykman_cli):
         output = ykman_cli(
-            "hsmauth", "access", "change", "-m", DEFAULT_MANAGEMENT_KEY, "-g"
+            "hsmauth",
+            "access",
+            "change-management-key",
+            "-m",
+            DEFAULT_MANAGEMENT_KEY,
+            "-g",
         ).output
 
         assert re.match(
             r"^Generated management key: [a-f0-9]{16}", output, re.MULTILINE
         )
-
-    def test_get_management_key_retries(self, ykman_cli):
-        output = ykman_cli("hsmauth", "access", "retries").output
-
-        assert re.match(
-            r"^Retries left for Management Key: [0-9]", output, re.MULTILINE
-        )
diff --git a/ykman/_cli/hsmauth.py b/ykman/_cli/hsmauth.py
@@ -600,7 +600,7 @@ def access():
     help="generate a random management key "
     "(can't be used with --new-management-key)",
 )
-def change(ctx, management_key, new_management_key, generate):
+def change_management_key(ctx, management_key, new_management_key, generate):
     """
     Change the management key.