[Tech Debt] standardising the error message for required parameter in cli command (#3827)

<!-- REMOVE IRRELEVANT COMMENTS BEFORE CREATING A PULL REQUEST -->
## Changes
<!-- Summary of your changes that are easy to understand. Add
screenshots when necessary -->

### Linked issues
<!-- DOC: Link issue with a keyword: close, closes, closed, fix, fixes,
fixed, resolve, resolves, resolved. See
https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword
-->

Resolves #2740 

### Functionality

- [ ] modified existing command: `databricks labs ucx ...`

### Tests
<!-- How is this tested? Please see the checklist below and also
describe any other relevant tests -->

- [ ] added unit tests

Author: HariGS-DB

src/databricks/labs/ucx/cli.py

@@ -255,7 +255,8 @@ def update_migration_progress(
 def repair_run(w: WorkspaceClient, step):
     """Repair Run the Failed Job"""
     if not step:
-        raise KeyError("You did not specify --step")
+        logger.error("--step is a required parameter")
+        return
     ctx = WorkspaceContext(w)
     logger.info(f"Repair Running {step} Job")
     ctx.deployed_workflows.repair_run(step)
@@ -313,10 +314,12 @@ def move(
     """move a uc table/tables from one schema to another schema in same or different catalog"""
     logger.info("Running move command")
     if from_catalog == "" or to_catalog == "":
-        logger.error("Please enter from_catalog and to_catalog details")
+        logger.error("--from_catalog and --to_catalog are required parameter")
         return
     if from_schema == "" or to_schema == "" or from_table == "":
-        logger.error("Please enter from_schema, to_schema and from_table (enter * for migrating all tables) details.")
+        logger.error(
+            "--from_schema, --to_schema and --from_table (enter * for migrating all tables) are required parameter."
+        )
         return
     if from_catalog == to_catalog and from_schema == to_schema:
         logger.error("please select a different schema or catalog to migrate to")
@@ -342,10 +345,12 @@ def alias(
 ):
     """move a uc table/tables from one schema to another schema in same or different catalog"""
     if from_catalog == "" or to_catalog == "":
-        logger.error("Please enter from_catalog and to_catalog details")
+        logger.error("--from_catalog and --to_catalog are required parameter")
         return
     if from_schema == "" or to_schema == "" or from_table == "":
-        logger.error("Please enter from_schema, to_schema and from_table (enter * for migrating all tables) details.")
+        logger.error(
+            "--from_schema, --to_schema and --from_table (enter * for migrating all tables) are required parameter."
+        )
         return
     if from_catalog == to_catalog and from_schema == to_schema:
         logger.error("please select a different schema or catalog to migrate to")

tests/unit/test_cli.py

@@ -414,9 +414,9 @@ def test_repair_run(ws):
     ws.jobs.list_runs.assert_called_once()
 
 
-def test_no_step_in_repair_run(ws):
-    with pytest.raises(KeyError):
-        repair_run(ws, "")
+def test_no_step_in_repair_run(ws, caplog):
+    repair_run(ws, "")
+    assert '--step is a required parameter' in caplog.messages
 
 
 def test_revert_migrated_tables(ws, caplog):
@@ -440,7 +440,7 @@ def test_move_no_catalog(ws, caplog):
     prompts = MockPrompts({})
     move(ws, prompts, "", "", "", "", "")
 
-    assert 'Please enter from_catalog and to_catalog details' in caplog.messages
+    assert '--from_catalog and --to_catalog are required parameter' in caplog.messages
 
 
 def test_move_same_schema(ws, caplog):
@@ -455,7 +455,7 @@ def test_move_no_schema(ws, caplog):
     move(ws, prompts, "SrcCat", "", "*", "TgtCat", "")
 
     assert (
-        'Please enter from_schema, to_schema and from_table (enter * for migrating all tables) details.'
+        '--from_schema, --to_schema and --from_table (enter * for migrating all tables) are required parameter.'
         in caplog.messages
     )
 
@@ -477,7 +477,7 @@ def test_move_aborted_via_prompt(ws):
 def test_alias_no_catalog(ws, caplog):
     alias(ws, "", "", "", "", "")
 
-    assert "Please enter from_catalog and to_catalog details" in caplog.messages
+    assert "--from_catalog and --to_catalog are required parameter" in caplog.messages
 
 
 def test_alias_same_schema(ws, caplog):
@@ -490,7 +490,7 @@ def test_alias_no_schema(ws, caplog):
     alias(ws, "SrcCat", "", "*", "TgtCat", "")
 
     assert (
-        'Please enter from_schema, to_schema and from_table (enter * for migrating all tables) details.'
+        '--from_schema, --to_schema and --from_table (enter * for migrating all tables) are required parameter.'
         in caplog.messages
     )