Allowing chip-repl test to send invalid enum values (#24352)

* Proof of concept for allowing chip-repl test to send invalid enum values * Prototype after latest round of discussions * Quick cleanup * Moving to properly done version * Couple extra fixes * Remove unneeded global * Add lock for placeholder count * Restyle * add missing regen to cluster-enums.h after merging master * Quick fix * Quick Fix * Restyle
project-chip · Aug 1, 2023 · 1078332 · 1078332
1 parent c923d98
commit 1078332
Show file tree

Hide file tree

Showing 7 changed files with 1,550 additions and 438 deletions.
diff --git a/src/app/zap-templates/templates/app/cluster-enums.zapt b/src/app/zap-templates/templates/app/cluster-enums.zapt
@@ -24,6 +24,10 @@ enum class {{asType label}} : {{asUnderlyingZclType name}} {
 {{#zcl_enum_items}}
 k{{asUpperCamelCase label}} = {{asHex value 2}},
 {{/zcl_enum_items}}
+// All received enum values that are not listed above will be mapped
+// to kUnknownEnumValue. This is a helper enum value that should only
+// be used by code to process how it handles receiving and unknown
+// enum value. This specific should never be transmitted.
 kUnknownEnumValue = {{first_unused_enum_value mode="first_unused"}},
 };
 {{#if (isWeaklyTypedEnum label)}}

diff --git a/src/controller/python/BUILD.gn b/src/controller/python/BUILD.gn
@@ -209,6 +209,7 @@ chip_python_wheel_action("chip-core") {
         "chip/clusters/Attribute.py",
         "chip/clusters/Command.py",
         "chip/clusters/__init__.py",
+        "chip/clusters/enum.py",
         "chip/configuration/__init__.py",
         "chip/discovery/__init__.py",
         "chip/discovery/library_handle.py",
@@ -289,6 +290,7 @@ chip_python_wheel_action("chip-core") {
   }
 
   py_package_reqs = [
+    "aenum",
     "coloredlogs",
     "construct",
     "dacite",