Merge pull request #171 from aiven/rdpo-fix-service-user-create-regre…

…ssion cli: fix user creation regression #171
aiven · Dec 17, 2020 · 90b91c7 · 90b91c7
2 parents 8b7c034 + 47cf89c
commit 90b91c7
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/aiven/client/cli.py b/aiven/client/cli.py
@@ -1148,7 +1148,7 @@ def service__user_create(self):
         acl_params = {
             key: arg_vars[key].split()
             for key in {"redis_acl_keys", "redis_acl_commands", "redis_acl_categories"}
-            if key in arg_vars
+            if arg_vars[key]
         }
         if acl_params:
             extra_params = {"access_control": acl_params}

diff --git a/tests/test_cli.py b/tests/test_cli.py
@@ -30,6 +30,10 @@ def test_service_types_v():
     AivenCLI().run(args=["service", "types", "-v"])
 
 
+def test_service_user_create():
+    AivenCLI().run(args=["service", "user-create", "service", "--username", "username"])
+
+
 def test_help():
     AivenCLI().run(args=["help"])