Remove command group aliases so the help messages are not duplicated

Author: BartoszCki

paperspace/cli/common.py

@@ -31,27 +31,7 @@ def jsonify_dicts(dict_):
 
 
 class ClickGroup(DYMMixin, HelpColorsGroup):
-    def group(self, *args, **kwargs):
-        aliases = []
-        _args = args
-        if args and isinstance(args[0], list):
-            # we have a list so create group aliases
-            aliases = args[0][1:]
-            _args = [args[0][0]] + list(args[1:])
-
-        if 'alias' in kwargs:
-            aliases.append(kwargs.pop('alias'))
-
-        def decorator(f):
-            cmd = click.group(*_args, **kwargs)(f)
-            self.add_command(cmd)
-            for alias in set(aliases):
-                alias_cmd = click.group(alias, **kwargs)(f)
-                self.add_command(alias_cmd)
-                alias_cmd.commands = cmd.commands
-            return cmd
-
-        return decorator
+    pass
 
 
 def prompt_for_secret(prompt):

paperspace/cli/deployments.py

@@ -9,7 +9,7 @@
 from paperspace.commands import deployments as deployments_commands
 
 
-@cli.group("deployments", alias='deployment', help="Manage deployments", cls=ClickGroup)
+@cli.group("deployments", help="Manage deployments", cls=ClickGroup)
 def deployments():
     pass
 

paperspace/cli/experiments.py

@@ -17,7 +17,7 @@
 )
 
 
-@cli.group(["experiments", "experiment"], help="Manage experiments", cls=ClickGroup)
+@cli.group("experiments", help="Manage experiments", cls=ClickGroup)
 def experiments():
     pass
 

paperspace/cli/hyperparameters.py

@@ -16,7 +16,7 @@ def add_use_docker_file_flag_if_used(ctx, param, value):
     return value
 
 
-@cli.group(["hyperparameters", "hyperparameter"], help="Manage hyperparameters", cls=ClickGroup)
+@cli.group("hyperparameters", help="Manage hyperparameters", cls=ClickGroup)
 def hyperparameters_group():
     pass
 

paperspace/cli/jobs.py

@@ -9,7 +9,7 @@
 from paperspace.commands import jobs as jobs_commands
 
 
-@cli.group(["jobs", "job"], help="Manage gradient jobs", cls=ClickGroup)
+@cli.group("jobs", help="Manage gradient jobs", cls=ClickGroup)
 def jobs_group():
     pass
 

paperspace/cli/machines.py

@@ -18,7 +18,7 @@
 )
 
 
-@cli.group(["machines", "machine"], help="Manage machines", cls=ClickGroup)
+@cli.group("machines", help="Manage machines", cls=ClickGroup)
 def machines_group():
     pass
 

paperspace/cli/models.py

@@ -6,7 +6,7 @@
 from paperspace.commands import models as models_commands
 
 
-@cli.group(["models", "model"], help="Manage models", cls=common.ClickGroup)
+@cli.group("models", help="Manage models", cls=common.ClickGroup)
 def models_group():
     pass
 

paperspace/cli/projects.py

@@ -4,7 +4,7 @@
 from paperspace.cli import common
 
 
-@cli.group(["projects", "project"], help="Manage projects", cls=common.ClickGroup)
+@cli.group("projects", help="Manage projects", cls=common.ClickGroup)
 def projects_group():
     pass
 

tests/functional/test_jobs.py

@@ -16,29 +16,6 @@ class TestJobs(object):
     EXPECTED_HEADERS_WITH_CHANGED_API_KEY["X-API-Key"] = "some_key"
 
 
-class TestJobsAlias(object):
-    expected_help = """Usage: cli %s [OPTIONS] COMMAND [ARGS]...
-
-  Manage gradient jobs
-
-Options:
-  --help  Show this message and exit.
-
-Commands:
-  artifacts  Manage jobs' artifacts
-  create     Create job
-  delete     Delete job
-  list       List jobs with optional filtering
-  logs       List job logs
-  stop       Stop running job
-"""
-    @pytest.mark.parametrize("group_name", ("jobs", "job"))
-    def test_alias_help(self, group_name):
-        cli_runner = CliRunner()
-        result = cli_runner.invoke(cli.cli, [group_name])
-        assert result.output == self.expected_help % group_name
-
-
 class TestListJobs(TestJobs):
     URL = "https://api.paperspace.io/jobs/getJobs/"
     BASIC_COMMAND = ["jobs", "list"]