Make contains/excludes constraints compile for lists of unsigned valu…

…es. (#19730)

Two changes here:

1) In chip_tests_iterate_expected_list, we were marking each individual value as
   an array (because "this.isArray" would be true in general when this helper is
   used).  That caused asTypedLiteral to fail to map it to an integer basic type
   and hence we were not adding with the proper suffixes, and that lead to
   signed-to-unsigned comparison errors.  This is the actual bugfix.

2) In asTypedLiteral, we really should be suffixing uint8_t with "U".  It's not
   clear why compilers don't complain about signed-to-unsigned compares for that
   type the way they do for uint16/32/64_t, but conceptually this is the right
   thing to do.

Fixes #19726

src/app/zap-templates/common/ClusterTestGeneration.js

@@ -1002,8 +1002,7 @@ function chip_tests_iterate_expected_list(values, options)
 {
   values = values.map(value => {
     return {
-      global: this.global, parent: this.parent, name: this.name, type: this.type, isArray: this.isArray,
-          isNullable: this.isNullable, value: value,
+      global: this.global, parent: this.parent, name: this.name, type: this.type, isArray: false, isNullable: false, value: value,
     }
   });
 

src/app/zap-templates/templates/app/helper.js

@@ -322,7 +322,7 @@ async function asTypedExpression(value, type)
   return `static_cast<${resultType}>(${value})`;
 }
 
-async function asTypedLiteral(value, type)
+async function asTypedLiteral(value, type, cookie)
 {
   const valueIsANumber = !isNaN(value);
   if (!valueIsANumber) {
@@ -335,6 +335,7 @@ async function asTypedLiteral(value, type)
     return value + 'L';
   case 'int64_t':
     return value + 'LL';
+  case 'uint8_t':
   case 'uint16_t':
     return value + 'U';
   case 'uint32_t':