{SQL} fix az sql server ad-admin update , az sql server ad-admin crea…

…te calls (#12541)

* Fixing az sql server ad-admin update and az sql server ad-admin create.
Adding test recording too.

* update the comments

* Revert update changes and fix it to have generic parameters. re-record test

* change aad create from properties to parameters

* remove spaces.

* Removing comment for issue #6011 as tests are recorded now.

* Fixing styling issues

Co-authored-by: Amol Agarwal <amagarwa@microsoft.com>

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

@@ -1100,7 +1100,7 @@ def _configure_security_policy_storage_params(arg_ctx):
     with self.argument_context('sql server ad-admin create') as c:
         # Create args that will be used to build up the ServerAzureADAdministrator object
         create_args_for_complex_type(
-            c, 'properties', ServerAzureADAdministrator, [
+            c, 'parameters', ServerAzureADAdministrator, [
                 'login',
                 'sid',
             ])

src/azure-cli/azure/cli/command_modules/sql/commands.py

@@ -420,7 +420,7 @@ def load_command_table(self, _):
         g.command('delete', 'delete')
         g.generic_update_command('update',
                                  custom_func_name='server_ad_admin_update',
-                                 setter_arg_name='properties')
+                                 setter_arg_name='parameters')
 
     server_keys_operations = CliCommandType(
         operations_tmpl='azure.mgmt.sql.operations#ServerKeysOperations.{}',

src/azure-cli/azure/cli/command_modules/sql/custom.py

@@ -2071,7 +2071,7 @@ def server_ad_admin_set(
     return client.create_or_update(
         server_name=server_name,
         resource_group_name=resource_group_name,
-        properties=kwargs)
+        parameters=kwargs)
 
 
 def server_ad_admin_update(
@@ -2090,7 +2090,6 @@ def server_ad_admin_update(
 
     return instance
 
-
 #####
 #           sql server firewall-rule
 #####