added fix for authV2 parsing with testing, rerecorded all cassettes

Author: a0x1ab

src/authV2/HISTORY.rst

@@ -3,6 +3,10 @@
 Release History
 ===============
 
+1.0.1
+++++++
+* Fix excluded paths parsing in `az webapp auth update` command.
+
 1.0.0
 ++++++
 * Update module documentation.

src/authV2/azext_authV2/custom.py

@@ -160,10 +160,11 @@ def update_auth_settings_v2(cmd, resource_group_name, name, set_string=None, ena
         if "globalValidation" not in existing_auth.keys():
             existing_auth["globalValidation"] = {}
         try:
-            if isinstance(json.loads(excluded_paths), list):
-                excluded_paths_list = json.loads(excluded_paths)
+            parsed = json.loads(excluded_paths)
+            if isinstance(parsed, list):
+                excluded_paths_list = parsed
             else:
-                excluded_paths_list = excluded_paths.split(",")
+                excluded_paths_list = [parsed] if parsed else []
         except json.JSONDecodeError:
             excluded_paths_list = excluded_paths.split(",")
 

src/authV2/azext_authV2/tests/latest/recordings/test_authV2_auth.yaml

@@ -16,6 +16,7 @@
 class Authv2ScenarioTest(ScenarioTest):
 
     @ResourceGroupPreparer(name_prefix='cli_test_authV2')
+    @AllowLargeResponse()
     def test_authV2_clientsecret_param_combinations(self, resource_group):
         webapp_name = self.create_random_name('webapp-authentication-test', 40)
         plan_name = self.create_random_name('webapp-authentication-plan', 40)
@@ -29,13 +30,17 @@ def test_authV2_clientsecret_param_combinations(self, resource_group):
 
         # testing show command for newly created app and initial fields
         self.cmd('webapp auth show -g {} -n {}'.format(resource_group, webapp_name)).assert_with_checks([
-            JMESPathCheck('properties', {})
+            JMESPathCheck('properties.platform.enabled', False)
         ])
+        
+        # upgrade to v2 before using v2 commands
+        self.cmd('webapp auth config-version upgrade -g {} -n {}'.format(resource_group, webapp_name))
 
         # # update and verify
         self.cmd('webapp auth update -g {} -n {} --enabled true --runtime-version 1.2.8'
                 .format(resource_group, webapp_name)).assert_with_checks([
-                    JMESPathCheck('platform', "{'enabled': True, 'runtimeVersion': '1.2.8'}")
+                    JMESPathCheck('platform.enabled', True),
+                    JMESPathCheck('platform.runtimeVersion', '1.2.8')
         ])
 
         with self.assertRaisesRegex(ArgumentUsageError, 'Usage Error: --client-secret and --client-secret-setting-name cannot both be '
@@ -79,6 +84,7 @@ def test_authV2_clientsecret_param_combinations(self, resource_group):
                 .format(resource_group, webapp_name))
 
     @ResourceGroupPreparer(name_prefix='cli_test_authV2')
+    @AllowLargeResponse()
     def test_authV2_auth(self, resource_group):
         webapp_name = self.create_random_name('webapp-authentication-test', 40)
         plan_name = self.create_random_name('webapp-authentication-plan', 40)
@@ -92,17 +98,21 @@ def test_authV2_auth(self, resource_group):
 
         # testing show command for newly created app and initial fields
         self.cmd('webapp auth show -g {} -n {}'.format(resource_group, webapp_name)).assert_with_checks([
-            JMESPathCheck('properties', {})
+            JMESPathCheck('properties.platform.enabled', False)
         ])
+        
+        self.cmd('webapp auth config-version upgrade -g {} -n {}'.format(resource_group, webapp_name))
 
         # # update and verify
         self.cmd('webapp auth update -g {} -n {} --enabled true --runtime-version 1.2.8'
                 .format(resource_group, webapp_name)).assert_with_checks([
-                    JMESPathCheck('platform', "{'enabled': True, 'runtimeVersion': '1.2.8'}")
+                    JMESPathCheck('platform.enabled', True),
+                    JMESPathCheck('platform.runtimeVersion', '1.2.8')
         ])
 
 
     @ResourceGroupPreparer(name_prefix='cli_test_authV2')
+    @AllowLargeResponse()
     def test_authV2_authclassic(self, resource_group):
         webapp_name = self.create_random_name('webapp-authentication-test', 40)
         plan_name = self.create_random_name('webapp-authentication-plan', 40)

src/authV2/azext_authV2/tests/latest/recordings/test_authV2_authclassic.yaml

@@ -14,7 +14,7 @@
     from distutils import log as logger
     logger.warn("Wheel is not available, disabling bdist_wheel hook")
 
-VERSION = '1.0.0'
+VERSION = '1.0.1'
 
 # The full list of classifiers is available at
 # https://pypi.python.org/pypi?%3Aaction=list_classifiers