Merge pull request #2 from wd-mgreynolds/curtlh-patch-02

Format using Black

Author: CurtLH

prism/cli.py

@@ -28,18 +28,10 @@ def param_fixup(value, config, config_name, option):
 @click.group(help="CLI for interacting with Workday’s Prism API")
 # Tenant specific parameters
 @click.option(
-    "--base_url",
-    envvar="workday_base_url",
-    type=str,
-    required=False,
-    help="The base URL for the API client",
+    "--base_url", envvar="workday_base_url", type=str, required=False, help="The base URL for the API client",
 )
 @click.option(
-    "--tenant_name",
-    envvar="workday_tenant_name",
-    type=str,
-    required=False,
-    help="The name of your Workday tenant",
+    "--tenant_name", envvar="workday_tenant_name", type=str, required=False, help="The name of your Workday tenant",
 )
 @click.option(
     "--client_id",
@@ -94,16 +86,7 @@ def param_fixup(value, config, config_name, option):
 )
 @click.pass_context
 def cli(
-    ctx,
-    base_url,
-    tenant_name,
-    client_id,
-    client_secret,
-    refresh_token,
-    log_level,
-    log_file,
-    config_file,
-    config_name,
+    ctx, base_url, tenant_name, client_id, client_secret, refresh_token, log_level, log_file, config_file, config_name,
 ):
     # Attempt to locate a configuration file - this is not required and config
     # parameters are only used if the configuration values are not passed on

prism/commands/buckets_commands.py

@@ -8,11 +8,7 @@
 
 @click.command("get")
 @click.option(
-    "-n",
-    "--isName",
-    is_flag=True,
-    default=False,
-    help="Flag to treat the bucket or table argument as a name.",
+    "-n", "--isName", is_flag=True, default=False, help="Flag to treat the bucket or table argument as a name.",
 )
 @click.option(
     "-l",
@@ -37,12 +33,7 @@
     help="How much information to be returned in response JSON.",
 )
 @click.option(
-    "-s",
-    "--search",
-    is_flag=True,
-    show_default=True,
-    default=False,
-    help="Use substring search bucket or table.",
+    "-s", "--search", is_flag=True, show_default=True, default=False, help="Use substring search bucket or table.",
 )
 @click.option("--table", help="The id or name of a Prism table to list all buckets.")
 @click.argument("bucket", required=False)
@@ -99,11 +90,7 @@ def buckets_get(ctx, bucket, table, isname, limit, offset, type_, search):
     help="Schema JSON file for the target table.",
 )
 @click.option(
-    "-o",
-    "--operation",
-    default="TruncateAndInsert",
-    show_default=True,
-    help="Operation to perform on the table.",
+    "-o", "--operation", default="TruncateAndInsert", show_default=True, help="Operation to perform on the table.",
 )
 @click.argument("bucket", required=False)
 @click.pass_context
@@ -120,11 +107,7 @@ def buckets_create(ctx, target_name, target_id, file, operation, bucket):
         sys.exit(1)
 
     bucket = p.buckets_create(
-        bucket_name=bucket,
-        target_id=target_id,
-        target_name=target_name,
-        schema=file,
-        operation=operation,
+        bucket_name=bucket, target_id=target_id, target_name=target_name, schema=file, operation=operation,
     )
 
     if bucket is not None:
@@ -136,19 +119,12 @@ def buckets_create(ctx, target_name, target_id, file, operation, bucket):
 
 @click.command("files")
 @click.option(
-    "-n",
-    "--target_name",
-    default=None,
-    help="Name of the table to associate with the bucket.",
+    "-n", "--target_name", default=None, help="Name of the table to associate with the bucket.",
 )
 @click.option("-i", "--target_id", default=None, help="Table ID to associate with the table.")
 @click.option("-f", "--file", default=None, help="Schema JSON file for the target table.")
 @click.option(
-    "-o",
-    "--operation",
-    default="TruncateAndInsert",
-    show_default=True,
-    help="Operation to perform on the table.",
+    "-o", "--operation", default="TruncateAndInsert", show_default=True, help="Operation to perform on the table.",
 )
 @click.option("-b", "--bucket", help="Bucket name to load files.", default=None)
 @click.option(
@@ -189,11 +165,7 @@ def buckets_files(ctx, target_name, target_id, file, operation, bucket, complete
 
 @click.command("complete")
 @click.option(
-    "-n",
-    "--isName",
-    is_flag=True,
-    default=False,
-    help="Flag to treat the bucket argument as a name.",
+    "-n", "--isName", is_flag=True, default=False, help="Flag to treat the bucket argument as a name.",
 )
 @click.argument("bucket", required=True)
 @click.pass_context
@@ -230,11 +202,7 @@ def buckets_complete(ctx, isname, bucket):
 
 @click.command("errorFile")
 @click.option(
-    "-n",
-    "--isName",
-    is_flag=True,
-    default=False,
-    help="Flag to treat the bucket argument as a name.",
+    "-n", "--isName", is_flag=True, default=False, help="Flag to treat the bucket argument as a name.",
 )
 @click.argument("bucket", required=True)
 @click.pass_context

prism/commands/dataChanges_commands.py

@@ -9,23 +9,13 @@
 
 @click.command("get")
 @click.option(
-    "-n",
-    "--isName",
-    default=False,
-    is_flag=True,
-    help="Flag to treat the dct argument as a name.",
+    "-n", "--isName", default=False, is_flag=True, help="Flag to treat the dct argument as a name.",
 )
 @click.option(
-    "-l",
-    "--limit",
-    default=-1,
-    help="The maximum number of object data entries included in the response.",
+    "-l", "--limit", default=-1, help="The maximum number of object data entries included in the response.",
 )
 @click.option(
-    "-o",
-    "--offset",
-    default=0,
-    help="The offset to the first object in a collection to include in the response.",
+    "-o", "--offset", default=0, help="The offset to the first object in a collection to include in the response.",
 )
 @click.option(
     "-t",
@@ -75,11 +65,7 @@ def dataChanges_get(ctx, isname, dct, limit, offset, type_, search):
 
 @click.command("validate")
 @click.option(
-    "-n",
-    "--isName",
-    default=False,
-    is_flag=True,
-    help="Flag to treat the dct argument as a name.",
+    "-n", "--isName", default=False, is_flag=True, help="Flag to treat the dct argument as a name.",
 )
 @click.option("-s", "--search", is_flag=True, help="Use contains search substring for --name.")
 @click.argument("dct", required=True)
@@ -120,11 +106,7 @@ def dataChanges_validate(ctx, isname, dct, search):
 
 @click.command("run")
 @click.option(
-    "-n",
-    "--isName",
-    default=False,
-    is_flag=True,
-    help="Flag to treat the dct argument as a name.",
+    "-n", "--isName", default=False, is_flag=True, help="Flag to treat the dct argument as a name.",
 )
 @click.argument("dct", required=True)
 @click.argument("fid", required=False)
@@ -164,18 +146,10 @@ def dataChanges_run(ctx, dct, fid, isname):
 
 @click.command("activities")
 @click.option(
-    "-n",
-    "--isName",
-    default=False,
-    is_flag=True,
-    help="Flag to treat the dct argument as a name.",
+    "-n", "--isName", default=False, is_flag=True, help="Flag to treat the dct argument as a name.",
 )
 @click.option(
-    "-s",
-    "--status",
-    is_flag=True,
-    default=False,
-    help="Return only the status of the activity.",
+    "-s", "--status", is_flag=True, default=False, help="Return only the status of the activity.",
 )
 @click.argument("dct", required=True)
 @click.argument("activityID", required=True)
@@ -216,25 +190,13 @@ def dataChanges_activities(ctx, dct, activityid, status, isname):
 
 @click.command("upload")
 @click.option(
-    "-n",
-    "--isName",
-    default=False,
-    is_flag=True,
-    help="Flag to treat the dct argument as a name.",
+    "-n", "--isName", default=False, is_flag=True, help="Flag to treat the dct argument as a name.",
 )
 @click.option(
-    "-w",
-    "--wait",
-    default=False,
-    is_flag=True,
-    help="Wait for the data change task to complete.",
+    "-w", "--wait", default=False, is_flag=True, help="Wait for the data change task to complete.",
 )
 @click.option(
-    "-v",
-    "--verbose",
-    default=False,
-    is_flag=True,
-    help="Display additional information.",
+    "-v", "--verbose", default=False, is_flag=True, help="Display additional information.",
 )
 @click.argument("dct", required=True)
 @click.argument("file", required=True, nargs=-1, type=click.Path(exists=True))

prism/commands/fileContainers_commands.py

@@ -40,10 +40,7 @@ def fileContainers_get(ctx, id):
 
 @click.command("load")
 @click.option(
-    "-i",
-    "--id",
-    default=None,
-    help="Target File container ID - defaults to a new container.",
+    "-i", "--id", default=None, help="Target File container ID - defaults to a new container.",
 )
 @click.argument("file", nargs=-1, type=click.Path(exists=True))
 @click.pass_context

prism/commands/tables_commands.py

@@ -10,11 +10,7 @@
 
 @click.command("get")
 @click.option(
-    "-n",
-    "--isName",
-    is_flag=True,
-    default=False,
-    help="Flag to treat the table argument as a name.",
+    "-n", "--isName", is_flag=True, default=False, help="Flag to treat the table argument as a name.",
 )
 @click.option(
     "-l",
@@ -39,17 +35,10 @@
     help="How much information returned for each table.",
 )
 @click.option(
-    "-c",
-    "--compact",
-    is_flag=True,
-    default=False,
-    help="Compact the table schema for use in edit (put) operations.",
+    "-c", "--compact", is_flag=True, default=False, help="Compact the table schema for use in edit (put) operations.",
 )
 @click.option(
-    "-s",
-    "--search",
-    is_flag=True,
-    help="Enable substring search of NAME in api name or display name.",
+    "-s", "--search", is_flag=True, help="Enable substring search of NAME in api name or display name.",
 )
 @click.argument("table", required=False)
 @click.pass_context
@@ -97,12 +86,7 @@ def tables_get(ctx, isname, table, limit, offset, type_, compact, search):
 @click.option("-n", "--table_name", help="Table name - overrides name from schema.")
 @click.option("-d", "--displayName", help="Specify a display name - defaults to name.")
 @click.option(
-    "-e",
-    "--enableForAnalysis",
-    type=bool,
-    is_flag=True,
-    default=None,
-    help="Enable this table for analytics.",
+    "-e", "--enableForAnalysis", type=bool, is_flag=True, default=None, help="Enable this table for analytics.",
 )
 @click.option("-s", "--sourceName", help="The API name of an existing table to copy.")
 @click.option("-w", "--sourceWID", help="The WID of an existing table to copy.")
@@ -164,11 +148,7 @@ def tables_create(ctx, table_name, displayname, enableforanalysis, sourcename, s
 
 @click.command("edit")
 @click.option(
-    "-t",
-    "--truncate",
-    is_flag=True,
-    default=False,
-    help="Truncate the table before updating.",
+    "-t", "--truncate", is_flag=True, default=False, help="Truncate the table before updating.",
 )
 @click.argument("file", required=True, type=click.Path(exists=True, dir_okay=False, readable=True))
 @click.pass_context
@@ -194,11 +174,7 @@ def tables_edit(ctx, file, truncate):
 
 @click.command("patch")
 @click.option(
-    "-n",
-    "--isName",
-    is_flag=True,
-    default=False,
-    help="Flag to treat the table argument as a name.",
+    "-n", "--isName", is_flag=True, default=False, help="Flag to treat the table argument as a name.",
 )
 @click.option(
     "--displayName",
@@ -222,10 +198,7 @@ def tables_edit(ctx, file, truncate):
     help="Set the documentation for an existing table.",
 )
 @click.option(
-    "--enableForAnalysis",
-    is_flag=False,
-    default=None,
-    type=click.Choice(["true", "false"], case_sensitive=False),
+    "--enableForAnalysis", is_flag=False, default=None, type=click.Choice(["true", "false"], case_sensitive=False),
 )
 @click.argument("table", required=True, type=str)
 @click.argument("file", required=False, type=click.Path(dir_okay=False))
@@ -334,11 +307,7 @@ def set_patch_value(attr, value):
 
 @click.command("upload")
 @click.option(
-    "-n",
-    "--isName",
-    is_flag=True,
-    default=False,
-    help="Flag to treat the table argument as a name.",
+    "-n", "--isName", is_flag=True, default=False, help="Flag to treat the table argument as a name.",
 )
 @click.option(
     "-o",
@@ -375,11 +344,7 @@ def tables_upload(ctx, table, isname, operation, file):
 
 @click.command("truncate")
 @click.option(
-    "-n",
-    "--isName",
-    is_flag=True,
-    default=False,
-    help="Flag to treat the table argument as a name.",
+    "-n", "--isName", is_flag=True, default=False, help="Flag to treat the table argument as a name.",
 )
 @click.argument("table", required=True)
 @click.pass_context