Temperature type (#1330)

* check if temperature is a signed type

Author: paulr34

src-electron/generator/helper-endpointconfig.js

@@ -651,15 +651,14 @@ function determineAttributeDefaultValue(
   typeSize,
   isNullable
 ) {
-  if (specifiedDefault !== null || !isNullable) {
+  if (specifiedDefault && (specifiedDefault !== null || !isNullable)) {
     return specifiedDefault
   }
 
   if (types.isString(type)) {
     // Handled elsewhere.
     return null
   }
-
   if (types.isSignedInteger(type)) {
     return '0x80' + '00'.repeat(typeSize - 1)
   }

src-electron/util/types.js

@@ -286,6 +286,7 @@ function isSignedInteger(type) {
     case 'int48s':
     case 'int56s':
     case 'int64s':
+    case 'temperature':
       return true
     default:
       return false

test/endpoint-config.test.js

@@ -216,7 +216,7 @@ test(
       '#define FIXED_PROFILE_IDS { 0x0107, 0x0104, 0x0104 }'
     )
     expect(epc).toContain('#define FIXED_ENDPOINT_TYPES { 0, 1, 2 }')
-    expect(epc).toContain('#define GENERATED_DEFAULTS_COUNT (12)')
+    expect(epc).toContain('#define GENERATED_DEFAULTS_COUNT (19)')
     expect(epc).toContain(
       `17, 'T', 'e', 's', 't', ' ', 'm', 'a', 'n', 'u', 'f', 'a', 'c', 't', 'u', 'r', 'e', 'r',`
     )

test/gen-matter-4.test.js

@@ -179,6 +179,9 @@ test(
       .attributeCount = 22, \\
       .clusterId = 0x00000028, \\
     },\\`)
+    expect(ept).toContain(
+      '  { 0x00000000, ZAP_TYPE(TEMPERATURE), 2, ZAP_ATTRIBUTE_MASK(NULLABLE), ZAP_SIMPLE_DEFAULT(0x8000) },'
+    )
   },
   testUtil.timeout.long()
 )

test/resource/matter-all-clusters.zap

@@ -2291,7 +2291,7 @@
             "0x0000 |        | server | 1 | 0 => GetWeeklyScheduleResponse"
           ],
           "attributes": [
-            "+ | 0x0000 |        | server | RAM |           |       |                      | 1 | 0 | 65344 | 0 => LocalTemperature [int16s]",
+            "+ | 0x0000 |        | server | RAM |           |       |                      | 1 | 0 | 65344 | 0 => LocalTemperature [temperature]",
             "+ | 0x0003 |        | server | RAM |           |       |               0x02BC | 1 | 0 | 65344 | 0 => AbsMinHeatSetpointLimit [int16s]",
             "+ | 0x0004 |        | server | RAM |           |       |               0x0BB8 | 1 | 0 | 65344 | 0 => AbsMaxHeatSetpointLimit [int16s]",
             "+ | 0x0005 |        | server | RAM |           |       |               0x0640 | 1 | 0 | 65344 | 0 => AbsMinCoolSetpointLimit [int16s]",

test/resource/old-matter/chip-types.xml

@@ -82,5 +82,6 @@ limitations under the License.
     <type id="0xFD" description="IPv6 Prefix"              name="ipv6pre"                     composite="true"/>
     <type id="0xFE" description="Hardware Address"         name="hwadr"                       composite="true"/>
     <type id="0xFF" description="Unknown"                  name="unknown"           size="0"                  />
+    <type id="0xD8" description="Temperature"              name="temperature"       size="2"  analog="true"     signed="true" />
   </atomic>
 </configurator>

zcl-builtin/matter/data-model/chip/chip-types.xml

@@ -81,5 +81,6 @@ limitations under the License.
     <type id="0xFD" description="IPv6 Prefix"              name="ipv6pre"                     composite="true"/>
     <type id="0xFE" description="Hardware Address"         name="hwadr"                       composite="true"/>
     <type id="0xFF" description="Unknown"                  name="unknown"           size="0"                  />
+    <type id="0xD8" description="Temperature"              name="temperature"       size="2"  analog="true"     signed="true" />
   </atomic>
 </configurator>

zcl-builtin/matter/data-model/chip/thermostat-cluster.xml

@@ -114,7 +114,7 @@ limitations under the License.
     <client tick="false" init="false">true</client>
     <server tick="false" init="false">true</server>
     <globalAttribute side="either" code="0xFFFD" value="5"/>
-    <attribute side="server" code="0x0000" define="LOCAL_TEMPERATURE" type="INT16S" min="0x954D" max="0x7FFF" writable="false" reportable="true" optional="false" isNullable="true">LocalTemperature</attribute>
+    <attribute side="server" code="0x0000" define="LOCAL_TEMPERATURE" type="TEMPERATURE" min="0x954D" max="0x7FFF" writable="false" reportable="true" optional="false" isNullable="true">LocalTemperature</attribute>
     <attribute side="server" code="0x0001" define="OUTDOOR_TEMPERATURE" type="INT16S" min="0x954D" max="0x7FFF" writable="false" optional="true" isNullable="true">OutdoorTemperature</attribute>
     <attribute side="server" code="0x0002" define="THERMOSTAT_OCCUPANCY" type="BITMAP8" min="0x00" max="0x01" writable="false" default="0x01" optional="true">Occupancy</attribute>
     <!-- OCCUPANCY -->