project-chip / connectedhomeip

Matter (formerly Project CHIP) creates more connections between more objects, simplifying development for manufacturers and increasing compatibility for consumers, guided by the Connectivity Standards Alliance.
https://buildwithmatter.com
Apache License 2.0
7.45k stars 1.99k forks source link

[CERT-TEST-FAILURE] - TC-TSTAT-1.1 - ClusterRevision value has to be updated for Thermostat Cluster #34296

Closed KishokG closed 2 months ago

KishokG commented 3 months ago

Feature Area

Other

Test Case

TC-TSTAT-1.1

Reproduction steps

  1. Bring up the DUT with thermostat-app.
  2. Pair the DUT using chip-tool through "Onnetwork" transport.
  3. Run the command ./chip-tool thermostat read cluster-revision 1 1

Expected Outcome:

[1720690392.499] [50423:50425] [TOO] Endpoint: 1 Cluster: 0x0000_0201 Attribute 0x0000_FFFD DataVersion: 1877232262 [1720690392.499] [50423:50425] [TOO] ClusterRevision: 7 [1720690392.500] [50423:50425] [EM] <<< [E:36766i S:61287 M:188811862 (Ack:54281020)] (S) Msg TX to 1:0000000000000001 [6193] [UDP:[fe80::e65f:1ff:fe0f:2878%eth0]:5540] --- Type 0000:10 (SecureChannel:StandaloneAck)

Actual Outcome:

[1720690392.499] [50423:50425] [TOO] Endpoint: 1 Cluster: 0x0000_0201 Attribute 0x0000_FFFD DataVersion: 1877232262 [1720690392.499] [50423:50425] [TOO] ClusterRevision: 6 [1720690392.500] [50423:50425] [EM] <<< [E:36766i S:61287 M:188811862 (Ack:54281020)] (S) Msg TX to 1:0000000000000001 [6193] [UDP:[fe80::e65f:1ff:fe0f:2878%eth0]:5540] --- Type 0000:10 (SecureChannel:StandaloneAck)

Bug prevalence

Whenever I try to read ClusterRivision.

GitHub hash of the SDK that was being used

6e31453ed626768f9c74bc9e59ffae7a7137999f

Platform

raspi

Anything else?

In SPEC, the ClusterRevision value has been updated to 7. As per spec changes, SDK(thermostat-app) has to update the ClusterRevision value from 6 to 7.

Specification References: https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/master/src/app_clusters/Thermostat.adoc#revision-history

KishokG commented 2 months ago

Issue fixed