{ARM} az policy: Rewrite Azure Policy CRUD commands using auto-generation

src/azure-cli/azure/cli/command_modules/policyinsights/_completers.py

@@ -5,6 +5,7 @@
 
 from azure.cli.core.decorators import Completer
 from azure.cli.core.commands.client_factory import get_subscription_id
+from azure.cli.command_modules.resource._client_factory import _resource_policy_client_factory
 
 from ._client_factory import cf_policy_insights
 
@@ -33,3 +34,31 @@ def get_policy_metadata_completion_list(cmd, prefix, namespace, **kwargs):  # py
     query_options = QueryOptions(top=2000)
 
     return [metadata.name for metadata in client.list(query_options) if metadata.name.startswith(prefix)]
+
+
+@Completer
+def get_policy_completion_list(cmd, prefix, namespace, **kwargs):  # pylint: disable=unused-argument
+    policy_client = _resource_policy_client_factory(cmd.cli_ctx)
+    result = policy_client.policy_definitions.list()
+    return [i.name for i in result]
+
+
+@Completer
+def get_policy_set_completion_list(cmd, prefix, namespace, **kwargs):  # pylint: disable=unused-argument
+    policy_client = _resource_policy_client_factory(cmd.cli_ctx)
+    result = policy_client.policy_set_definitions.list()
+    return [i.name for i in result]
+
+
+@Completer
+def get_policy_assignment_completion_list(cmd, prefix, namespace, **kwargs):  # pylint: disable=unused-argument
+    policy_client = _resource_policy_client_factory(cmd.cli_ctx)
+    result = policy_client.policy_assignments.list()
+    return [i.name for i in result]
+
+
+@Completer
+def get_policy_exemption_completion_list(cmd, prefix, namespace, **kwargs):  # pylint: disable=unused-argument
+    policy_client = _resource_policy_client_factory(cmd.cli_ctx)
+    result = policy_client.policy_exemptions.list()
+    return [i.name for i in result]

src/azure-cli/azure/cli/command_modules/policyinsights/_params.py

@@ -7,13 +7,14 @@
     resource_group_name_type, get_enum_type)
 
 from azure.cli.command_modules.resource._completers import (
-    get_policy_set_completion_list, get_policy_completion_list,
-    get_policy_assignment_completion_list, get_providers_completion_list, get_resource_types_completion_list)
+    get_providers_completion_list, get_resource_types_completion_list)
 
 from ._validators import (validate_resource, validate_expand)
 from azure.cli.command_modules.resource._validators import validate_metadata
 
-from ._completers import get_policy_remediation_completion_list, get_policy_metadata_completion_list
+from ._completers import (
+    get_policy_set_completion_list, get_policy_completion_list, get_policy_assignment_completion_list,
+    get_policy_remediation_completion_list, get_policy_metadata_completion_list)
 
 from ._actions import AttestationEvidenceAddAction
 

src/azure-cli/azure/cli/command_modules/resource/_client_factory.py

@@ -115,18 +115,6 @@ def cf_feature_registrations(cli_ctx, _):
     return _resource_feature_client_factory(cli_ctx).subscription_feature_registrations
 
 
-def cf_policy_definitions(cli_ctx, _):
-    return _resource_policy_client_factory(cli_ctx).policy_definitions
-
-
-def cf_policy_set_definitions(cli_ctx, _):
-    return _resource_policy_client_factory(cli_ctx).policy_set_definitions
-
-
-def cf_policy_exemptions(cli_ctx, _):
-    return _resource_policy_client_factory(cli_ctx).policy_exemptions
-
-
 def cf_management_locks(cli_ctx, _):
     return _resource_lock_client_factory(cli_ctx).management_locks
 

src/azure-cli/azure/cli/command_modules/resource/_completers.py

@@ -5,36 +5,7 @@
 
 from azure.cli.core.decorators import Completer
 
-from azure.cli.command_modules.resource._client_factory import (
-    _resource_policy_client_factory, _resource_client_factory)
-
-
-@Completer
-def get_policy_completion_list(cmd, prefix, namespace, **kwargs):  # pylint: disable=unused-argument
-    policy_client = _resource_policy_client_factory(cmd.cli_ctx)
-    result = policy_client.policy_definitions.list()
-    return [i.name for i in result]
-
-
-@Completer
-def get_policy_set_completion_list(cmd, prefix, namespace, **kwargs):  # pylint: disable=unused-argument
-    policy_client = _resource_policy_client_factory(cmd.cli_ctx)
-    result = policy_client.policy_set_definitions.list()
-    return [i.name for i in result]
-
-
-@Completer
-def get_policy_assignment_completion_list(cmd, prefix, namespace, **kwargs):  # pylint: disable=unused-argument
-    policy_client = _resource_policy_client_factory(cmd.cli_ctx)
-    result = policy_client.policy_assignments.list()
-    return [i.name for i in result]
-
-
-@Completer
-def get_policy_exemption_completion_list(cmd, prefix, namespace, **kwargs):  # pylint: disable=unused-argument
-    policy_client = _resource_policy_client_factory(cmd.cli_ctx)
-    result = policy_client.policy_exemptions.list()
-    return [i.name for i in result]
+from azure.cli.command_modules.resource._client_factory import _resource_client_factory
 
 
 @Completer