search

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.47k stars 2k forks source link

[CERT-TEST-FAILURE] - TC-DEM-1.1 - ClusterRevision value has to be updated for Device Energy Management Cluster #34297

Closed KishokG closed 2 months ago

KishokG commented 3 months ago

Feature Area

Other

Test Case

TC-DEM-1.1

Reproduction steps

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

Expected Outcome:

[1720690440.539] [50435:50437] [TOO] Endpoint: 1 Cluster: 0x0000_0098 Attribute 0x0000_FFFD DataVersion: 521028924 [1720690440.539] [50435:50437] [TOO] ClusterRevision: 4 [1720690440.540] [50435:50437] [EM] <<< [E:5099i S:17862 M:203971832 (Ack:69830139)] (S) Msg TX to 1:0000000000000001 [C5DD] [UDP:[fe80::e65f:1ff:fe0f:2878%eth0]:5540] --- Type 0000:10 (SecureChannel:StandaloneAck)

Actual Outcome:

[1720690440.539] [50435:50437] [TOO] Endpoint: 1 Cluster: 0x0000_0098 Attribute 0x0000_FFFD DataVersion: 521028924 [1720690440.539] [50435:50437] [TOO] ClusterRevision: 3 [1720690440.540] [50435:50437] [EM] <<< [E:5099i S:17862 M:203971832 (Ack:69830139)] (S) Msg TX to 1:0000000000000001 [C5DD] [UDP:[fe80::e65f:1ff:fe0f:2878%eth0]:5540] --- Type 0000:10 (SecureChannel:StandaloneAck)

Bug prevalence

Whenever I try to read ClusterRevision.

GitHub hash of the SDK that was being used

6e31453ed626768f9c74bc9e59ffae7a7137999f

Platform

raspi

Anything else?

In SPEC, the ClusterRevision value has been updated to 4. As per spec changes, SDK(energy-management-app) has to update the ClusterRevision value from 3 to 4.

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

cjandhyala commented 2 months ago

@KishokG is this still an issue ? Can we close this ticket. ?

KishokG commented 2 months ago

Yes @cjandhyala It's yet to be fixed

jamesharrow commented 2 months ago

Note that we have removed TC DEM 1.1 https://github.com/CHIP-Specifications/chip-test-plans/pull/4555 The cluster revision however has been fixed in the SDK in another SDK PR: https://github.com/project-chip/connectedhomeip/pull/35067