user list with -a

Author: MenxLi

lfss/cli/user.py

@@ -43,7 +43,7 @@ def parse_bool(s):
     sp_list = sp.add_parser('list', help="List specified users, or detailed info with -l")
     sp_list.add_argument("username", nargs='*', type=str, default=None)
     sp_list.add_argument("-l", "--long", action="store_true", help="Show detailed information, including credential and peer users")
-    sp_list.add_argument("--hidden", action="store_true", help="Include hidden users (virtual users) in the listing")
+    sp_list.add_argument("-a", '--all', action="store_true", dest="show_all", help="Show all users, include hidden users (virtual users) in the listing")
 
     sp_peer = sp.add_parser('set-peer', help="Set peer user relationship")
     sp_peer.add_argument('src_username', type=str)
@@ -107,7 +107,7 @@ async def get_uconn():
         async with get_uconn() as uconn:
             term_width = os.get_terminal_size().columns
             async def __iter_users():
-                if args.hidden:
+                if args.show_all:
                     async for user in uconn.iter_all(): yield user
                     async for user in uconn.iter_hidden(): yield user
                 else:

test/cases/test_14_virtual_user.py

@@ -27,6 +27,11 @@ def test_init_user_creation(server):
     v1_token = v1.credential
     v1_name = v1.username
 
+def test_u1_create(server):
+    u1 = get_conn('u1')
+    with pytest.raises(Exception, match="403"):
+        u1.add_virtual_user(tag="baduser", peers={AccessLevel.READ: ['u0']})
+
 def test_v1_put(server):
     v1 = get_conn_bytoken(v1_token)