Raise usage error when updating profile does not receive any argument (#250)

Author: Juliya Smith

src/code42cli/cmds/profile.py

@@ -60,6 +60,7 @@ def username_option(required=False):
     is_flag=True,
     help="For development purposes, do not validate the SSL certificates of Code42 servers. "
     "This is not recommended, except for specific scenarios like testing.",
+    default=None,
 )
 
 
@@ -104,11 +105,19 @@ def create(name, server, username, password, disable_ssl_errors):
 def update(name, server, username, password, disable_ssl_errors):
     """Update an existing profile."""
     c42profile = cliprofile.get_profile(name)
+
+    if not server and not username and not password and disable_ssl_errors is None:
+        raise click.UsageError(
+            "Must provide at least one of `--username`, `--server`, `--password`, or "
+            "`--disable-ssl-errors` when updating a profile."
+        )
+
     cliprofile.update_profile(c42profile.name, server, username, disable_ssl_errors)
     if password:
         _set_pw(name, password)
     elif not c42profile.has_stored_password:
         _prompt_for_allow_password_set(c42profile.name)
+
     echo("Profile '{}' has been updated.".format(c42profile.name))
 
 

tests/cmds/test_profile.py

@@ -357,6 +357,22 @@ def test_update_profile_if_user_agrees_and_valid_connection_sets_password(
     )
 
 
+def test_update_profile_when_given_zero_args_prints_error_message(
+    runner, mock_cliprofile_namespace, profile
+):
+    name = "foo"
+    profile.name = name
+    profile.ignore_ssl_errors = False
+    mock_cliprofile_namespace.get_profile.return_value = profile
+    result = runner.invoke(cli, ["profile", "update"])
+    expected = (
+        "Must provide at least one of `--username`, `--server`, `--password`, "
+        "or `--disable-ssl-errors` when updating a profile."
+    )
+    assert "Profile 'foo' has been updated" not in result.output
+    assert expected in result.output
+
+
 def test_delete_profile_warns_if_deleting_default(runner, mock_cliprofile_namespace):
     mock_cliprofile_namespace.is_default_profile.return_value = True
     result = runner.invoke(cli, ["profile", "delete", "mockdefault"])