Update command output field casing and include zones when null (Azure#3212)

Author: greenms

src/redisenterprise/HISTORY.rst

@@ -3,6 +3,11 @@
 Release History
 ===============
 
+0.1.1
+++++++
+* Renamed remaining snake_case command output fields to camelCase to be consistent with the REST API.
+* Listed the following cluster attribute as null in the command output when the cluster attribute is null: zones.
+
 0.1.0
 ++++++
 * Initial release.

src/redisenterprise/azext_redisenterprise/manual/custom.py

@@ -8,6 +8,7 @@
 
 from azure.cli.core.util import sdk_no_wait
 from azure.cli.core.azclierror import MutuallyExclusiveArgumentError
+from msrest.serialization import last_restapi_key_transformer
 
 
 def _get_database_client(cli_ctx):
@@ -17,7 +18,11 @@ def _get_database_client(cli_ctx):
 
 def _get_cluster_with_databases(cluster,
                                 databases):
-    result = cluster.as_dict()
+    result = cluster.as_dict(key_transformer=last_restapi_key_transformer)
+    # Restore select null cluster attributes that were removed by cluster.as_dict()
+    if cluster.zones is None:
+        result['zones'] = None
+
     result['databases'] = []
     for database in databases:
         result['databases'].append(database)

src/redisenterprise/azext_redisenterprise/manual/tests/latest/example_steps.py

@@ -14,10 +14,7 @@ def step_create(test, rg, checks=None):
         test.cmd('az redisenterprise create '
                  '--cluster-name "{cluster}" '
                  '--sku "EnterpriseFlash_F300" '
-                 '--capacity 3 '
                  '--tags tag1="value1" '
-                 '--zones "1" "2" "3" '
-                 '--minimum-tls-version "1.2" '
                  '--no-database '
                  '--resource-group "{rg}"',
                  checks=checks)

src/redisenterprise/azext_redisenterprise/manual/tests/latest/recordings/test_redisenterprise_scenario1.yaml

@@ -46,9 +46,9 @@ def call_scenario1(test, rg):
         test.check("sku.capacity", 4),
         test.check("tags.tag1", "value1"),
         test.check("zones", ["1", "2", "3"]),
-        test.check("minimum_tls_version", "1.2"),
-        test.check("provisioning_state", "Succeeded"),
-        test.check("resource_state", "Running"),
+        test.check("minimumTlsVersion", "1.2"),
+        test.check("provisioningState", "Succeeded"),
+        test.check("resourceState", "Running"),
         test.check("type", "Microsoft.Cache/redisEnterprise"),
         test.check("databases[0].name", "default"),
         test.check("databases[0].resourceGroup", "{rg}"),
@@ -114,7 +114,7 @@ def call_scenario2(test, rg):
         test.check("sku.name", "EnterpriseFlash_F300"),
         test.check("sku.capacity", 3),
         test.check("tags.tag1", "value1"),
-        test.check("zones", ["1", "2", "3"]),
+        test.check("zones", None),
         test.check("minimumTlsVersion", "1.2"),
         test.check("provisioningState", "Succeeded"),
         test.check("resourceState", "Running"),
@@ -127,10 +127,10 @@ def call_scenario2(test, rg):
         test.check("sku.name", "EnterpriseFlash_F300"),
         test.check("sku.capacity", 3),
         test.check("tags.tag1", "value1"),
-        test.check("zones", ["1", "2", "3"]),
-        test.check("minimum_tls_version", "1.2"),
-        test.check("provisioning_state", "Succeeded"),
-        test.check("resource_state", "Running"),
+        test.check("zones", None),
+        test.check("minimumTlsVersion", "1.2"),
+        test.check("provisioningState", "Succeeded"),
+        test.check("resourceState", "Running"),
         test.check("type", "Microsoft.Cache/redisEnterprise"),
         test.check("length(databases)", 0),
     ])

src/redisenterprise/azext_redisenterprise/manual/tests/latest/recordings/test_redisenterprise_scenario2.yaml

@@ -10,7 +10,7 @@
 from setuptools import setup, find_packages
 
 # HISTORY.rst entry.
-VERSION = '0.1.0'
+VERSION = '0.1.1'
 try:
     from azext_redisenterprise.manual.version import VERSION
 except ImportError: