fixing alias test for CustomClusterConfigurationFileName readonly exception

Author: robertpatrick

core/src/main/resources/oracle/weblogic/deploy/aliases/category_modules/CoherenceClusterSystemResource.json

@@ -265,7 +265,7 @@
             },
             "attributes": {
                 "CustomClusterConfigurationFileLastUpdatedTimestamp": [ {"version": "[12.1.2,)", "wlst_mode": "both", "wlst_name": "CustomClusterConfigurationFileLastUpdatedTimestamp", "wlst_path": "WP001", "default_value": 0,      "wlst_type": "long",   "access": "${:IGNORED}" } ],
-                "CustomClusterConfigurationFileName":                 [ {"version": "[12.1.2,)", "wlst_mode": "both", "wlst_name": "CustomClusterConfigurationFileName",                 "wlst_path": "WP001", "default_value": null,   "wlst_type": "string", "set_method": "MBEAN.set_coherence_cluster_custom_config_file", "uses_path_tokens": "true" } ],
+                "CustomClusterConfigurationFileName":                 [ {"version": "[12.1.2,)", "wlst_mode": "both", "wlst_name": "CustomClusterConfigurationFileName",                 "wlst_path": "WP001", "default_value": null,   "wlst_type": "string", "set_method": "MBEAN.set_coherence_cluster_custom_config_file", "uses_path_tokens": "true", "comment": "This attribute is read-only in online mode.  We are working around that using the custom set method to invoke the MBean's importCustomClusterConfigurationFile() function to set the value" } ],
                 "Version":                                            [ {"version": "[12.1.2,)", "wlst_mode": "both", "wlst_name": "Version",                                            "wlst_path": "WP001", "default_value": null,   "wlst_type": "string"} ]
             },
             "wlst_attributes_path": "WP001",

integration-tests/alias-test/verify/src/test/python/aliastest/verify/utils.py

@@ -67,6 +67,12 @@
     ]
 }
 
+ONLINE_ATTRIBUTE_READONLY_EXCEPTION_MAP = {
+    "/CoherenceClusterSystemResource/CoherenceResource": [
+        "CustomClusterConfigurationFileName"
+    ]
+}
+
 OFFLINE_TEST_ANOMALIES_MAP = {
     '/Application': {
         'ModuleType': 'war',

integration-tests/alias-test/verify/src/test/python/aliastest/verify/verifier.py

@@ -121,7 +121,7 @@
     ERROR_ATTRIBUTE_ALIAS_NOT_FOUND:               'Attribute not found in aliases',
     ERROR_ATTRIBUTE_INCORRECT_CASE:                'Attribute case incorrect',
     ERROR_ATTRIBUTE_ALIAS_NOT_FOUND_IS_READONLY:   'Readonly attribute not found in aliases',
-    ERROR_ATTRIBUTE_READONLY:                      'Attribute is marked readwrite',
+    ERROR_ATTRIBUTE_READONLY:                      'Attribute is marked readwrite in the alias but read-only in WLST',
     ERROR_ATTRIBUTE_NOT_READONLY_VERSION:          'Attribute is marked readonly or is invalid version range',
     ERROR_ATTRIBUTE_NOT_READONLY:                  'Attribute is not marked readwrite',
     ERROR_ATTRIBUTE_WRONG_DEFAULT_VALUE:           'Attribute wrong default value',
@@ -629,6 +629,12 @@ def _is_generated_attribute_readonly(self, location, generated_attribute, genera
         _logger.entering(location.get_folder_path(), generated_attribute, str(generated_attribute_info),
                          str(alias_get_required_attribute_list), class_name=CLASS_NAME, method_name=_method_name)
 
+        if self._model_context.get_target_wlst_mode() == WlstModes.ONLINE:
+            if location.get_folder_path() in verify_utils.ONLINE_ATTRIBUTE_READONLY_EXCEPTION_MAP:
+                attribute_exceptions = verify_utils.ONLINE_ATTRIBUTE_READONLY_EXCEPTION_MAP[location.get_folder_path()]
+                if isinstance(attribute_exceptions, list) and generated_attribute in attribute_exceptions:
+                    return False
+
         if READ_TYPE not in generated_attribute_info and CMO_READ_TYPE not in generated_attribute_info:
             self._add_error(location, ERROR_FAILURE_ATTRIBUTE_UNEXPECTED,
                             message='Generated attribute has no read type', attribute=generated_attribute)