Error handling

Author: RichardChen820

src/azure-cli/azure/cli/command_modules/appconfig/_kv_import_helpers.py

@@ -807,7 +807,7 @@ def __read_kv_from_kubernetes_configmap(
 
         return key_values
     except Exception as exception:
-        raise CLIError(
+        raise AzureInternalError(
             f"Failed to read key-values from ConfigMap '{configmap_name}' in namespace '{namespace}'.\n{str(exception)}"
         )
 

src/azure-cli/azure/cli/command_modules/appconfig/_validators.py

@@ -160,7 +160,7 @@ def validate_aks_cluster_name_or_id(cmd, namespace):
             elif namespace.connection_string:
                 config_store_name = get_store_name_from_connection_string(namespace.connection_string)
             else:
-                raise CLIError("Please provide App Configuration name or connection string for fetching the AKS cluster details. Alternatively, you can provide a valid ARM ID for the AKS cluster.")
+                raise ArgumentUsageError("Please provide App Configuration name or connection string for fetching the AKS cluster details. Alternatively, you can provide a valid ARM ID for the AKS cluster.")
 
             resource_group, _ = resolve_store_metadata(cmd, config_store_name)
             namespace.aks_cluster = {

src/azure-cli/azure/cli/command_modules/appconfig/tests/latest/test_appconfig_kv_import_export_commands.py

@@ -14,7 +14,7 @@
 from azure.cli.testsdk import (ResourceGroupPreparer, ScenarioTest, LiveScenarioTest)
 from azure.cli.command_modules.appconfig._constants import FeatureFlagConstants, KeyVaultConstants, ImportExportProfiles, AppServiceConstants
 from azure.cli.testsdk.scenario_tests import AllowLargeResponse
-from azure.cli.core.azclierror import MutuallyExclusiveArgumentError
+from azure.cli.core.azclierror import AzureInternalError, MutuallyExclusiveArgumentError
 from azure.cli.command_modules.appconfig.tests.latest._test_utils import create_config_store, CredentialResponseSanitizer, get_resource_name_prefix
 
 TEST_DIR = os.path.abspath(os.path.join(os.path.abspath(__file__), '..'))
@@ -1329,5 +1329,5 @@ def test_appconfig_import_from_kubernetes_configmap(self, resource_group, locati
         })
 
         # This should fail gracefully
-        with self.assertRaises(CLIError):
+        with self.assertRaises(AzureInternalError):
             self.cmd('appconfig kv import -n {config_store_name} -s aks --aks-cluster {aks_cluster_name} --configmap-name {non_existent_configmap} --configmap-namespace {namespace} --label {error_label} -y')