Handle list of str properly

parsec/commands/folders/set_permissions.py

@@ -12,15 +12,21 @@
 )
 @click.option(
     "--add_ids",
-    help="list of role IDs which can add datasets to the folder"
+    help="list of role IDs which can add datasets to the folder",
+    type=str,
+    multiple=True
 )
 @click.option(
     "--manage_ids",
-    help="list of role IDs which can manage datasets in the folder"
+    help="list of role IDs which can manage datasets in the folder",
+    type=str,
+    multiple=True
 )
 @click.option(
     "--modify_ids",
-    help="list of role IDs which can modify datasets in the folder"
+    help="list of role IDs which can modify datasets in the folder",
+    type=str,
+    multiple=True
 )
 
 @pass_context

parsec/commands/quotas/create_quota.py

@@ -15,17 +15,21 @@
 )
 @click.option(
     "--in_users",
-    help="A list of user IDs or user emails."
+    help="A list of user IDs or user emails.",
+    type=str,
+    multiple=True
 )
 @click.option(
     "--in_groups",
-    help="A list of group IDs or names."
+    help="A list of group IDs or names.",
+    type=str,
+    multiple=True
 )
 
 @pass_context
 @bioblend_exception
 @dict_output
-def cli(ctx, name, description, amount, operation, default="", in_users="", in_groups=""):
+def cli(ctx, name, description, amount, operation, default="", in_users=None, in_groups=None):
     """Create a new quota
     """
     return ctx.gi.quotas.create_quota(name, description, amount, operation, default=default, in_users=in_users, in_groups=in_groups)

parsec/commands/quotas/update_quota.py

@@ -32,17 +32,21 @@
 )
 @click.option(
     "--in_users",
-    help="A list of user IDs or user emails."
+    help="A list of user IDs or user emails.",
+    type=str,
+    multiple=True
 )
 @click.option(
     "--in_groups",
-    help="A list of group IDs or names."
+    help="A list of group IDs or names.",
+    type=str,
+    multiple=True
 )
 
 @pass_context
 @bioblend_exception
 @dict_output
-def cli(ctx, quota_id, name="", description="", amount="", operation="", default="", in_users="", in_groups=""):
+def cli(ctx, quota_id, name="", description="", amount="", operation="", default="", in_users=None, in_groups=None):
     """Update an existing quota
     """
     return ctx.gi.quotas.update_quota(quota_id, name=name, description=description, amount=amount, operation=operation, default=default, in_users=in_users, in_groups=in_groups)

requirements.txt

@@ -1,6 +1,6 @@
 Click
 pyyaml
 # pre-release version
-git+https://github.com/erasche/bioblend.git@4fc0803ffc86bab14bf36eeb30bb055c7ef6cd94
+git+https://github.com/erasche/bioblend.git@2e92cbcc22d2e1d9dec6554f3ad21b12112925d5
 docutils
 wrapt

scripts/autobuilder.py

@@ -52,6 +52,10 @@
         'type=str', # TODO
         'multiple=True',
     ],
+    'list of str': [
+        'type=str', # TODO
+        'multiple=True',
+    ],
     'file': [
         'type=click.File(\'rb+\')'
     ],