Refactor

Author: abhaasgoyal

meorg_client/cli.py

@@ -218,10 +218,22 @@ def analysis_start(id: str):
 @click.argument("model_id")
 @click.argument("exp_id")
 @click.argument("name")
-def new_model_output(model_id: str, exp_id: str, name: str):
+def create_new_model_output(model_id: str, exp_id: str, name: str):
     """
     Create a new model output entity
 
+
+    Parameters
+    ----------
+    model_id : str
+        Model profile ID.
+
+    exp_id : str
+        Experiment ID.
+
+    name : str
+        New model output name
+
     Prints modeloutput ID of created object, and whether it already existed or not.
     """
     client = _get_client()
@@ -234,7 +246,7 @@ def new_model_output(model_id: str, exp_id: str, name: str):
         output_id = response.get("data").get("modeloutputId")
         created = response.get("data").get("created")
         click.echo(f"Model Output ID: {output_id}")
-        click.echo(f"Already created: {created}")
+        click.echo(f"Override (existing) Model Output ID: {created}")
     pass
 
 
@@ -334,7 +346,7 @@ def cli_output():
 cli_analysis.add_command(analysis_status)
 
 # Add output command
-cli_output.add_command(new_model_output)
+cli_output.add_command(create_new_model_output)
 
 # Add subparsers to the master
 cli.add_command(cli_endpoints)

meorg_client/tests/test_cli.py

@@ -61,7 +61,8 @@ def test_create_model_output(
 ):
     """Test Creation of Model output"""
     result = runner.invoke(
-        cli.new_model_output, [model_profile_id, experiment_id, model_output_name]
+        cli.create_new_model_output,
+        [model_profile_id, experiment_id, model_output_name],
     )
     assert result.exit_code == 0