Make sure no field is using keyword string

asyncua/common/structures104.py

@@ -4,6 +4,7 @@ from datetime import datetime
 import uuid
 import logging
 import re
+import keyword
 from typing import Union, List, TYPE_CHECKING, Tuple, Optional, Any
 from dataclasses import dataclass, field
 
@@ -108,13 +109,14 @@ def clean_name(name):
     Remove characters that might be present in  OPC UA structures
     but cannot be part of of Python class names
     """
+    if keyword.iskeyword(name):
+        return name + "_"
     if name.isidentifier():
         return name
-    else:
-        newname = re.sub(r'\W+', '_', name)
-        newname = re.sub(r'^[0-9]+', r'_\g<0>', newname)
-        logger.warning("renamed %s to %s due to Python syntax", name, newname)
-        return newname
+    newname = re.sub(r'\W+', '_', name)
+    newname = re.sub(r'^[0-9]+', r'_\g<0>', newname)
+    logger.warning("renamed %s to %s due to Python syntax", name, newname)
+    return newname
 
 
 def get_default_value(uatype, enums=None):

tests/test_common.py

@@ -1124,12 +1124,12 @@ async def test_duplicated_browsenames_different_ns(opc):
         return
 
 
-async def test_custom_enum(opc):
+async def test_custom_enum_x(opc):
     idx = 4
     await new_enum(opc.opc, idx, "MyCustEnum", [
         "titi",
         "toto",
-        "tutu",
+        "None",
     ])
 
     await opc.opc.load_data_type_definitions()