Improve error message in download when use invalid input (Azure#28992)

* add name and version for output

* add asset_name, asset_version, merge 2022-12-01 pr

* resolve comments

* add test and modify related codes

* refine code

* fix failed test

* fix comments

* fix condition._port_name

* fix test and some comments

* revert the modification of compute

* fix comments

* fix pylint error, add tests for CommandJob, Spark, Parallel, Sweep

* delete one test, will add in another pr

* modify error message

* improve error msg, add ut

* run black

---------

Co-authored-by: chenyangliao <chenyangliao@microsoft.com>

Author: lllilithyang

sdk/ml/azure-ai-ml/azure/ai/ml/operations/_job_operations.py

@@ -87,6 +87,7 @@
     PipelineChildJobError,
     ValidationErrorType,
     ValidationException,
+    UserErrorException,
 )
 from azure.ai.ml.operations._run_history_constants import RunHistoryConstants
 from azure.ai.ml.sweep import SweepJob
@@ -276,6 +277,8 @@ def get(self, name: str) -> Job:
         :rtype: Job
         :raise: ResourceNotFoundError if can't find a job matching provided name.
         """
+        if not isinstance(name, str):
+            raise UserErrorException(f"{name} is a invalid input for client.jobs.get().")
         job_object = self._get_job(name)
 
         if not _is_pipeline_child_job(job_object):

sdk/ml/azure-ai-ml/tests/job_common/unittests/test_job_operations.py

@@ -254,3 +254,8 @@ def test_job_create_skip_validation(self, mock_method, mock_job_operation: JobOp
             mock_thing.assert_not_called()
             mock_job_operation.create_or_update(job=job)
             mock_thing.assert_called_once()
+
+    def test_download_with_none(self, mock_job_operation: JobOperations) -> None:
+        with pytest.raises(Exception) as ex:
+            mock_job_operation.download(None)
+        assert "None is a invalid input for client.jobs.get()." in ex.value.message