[CERT-TEST-FAILURE] -TC-SWTCH-2.2 - Test failed

[Survensa]

Feature Area

Other

Test Case

TC-SWTCH-2.2

Reproduction steps

1. Bring up the DUT (all-clusters-app).

2. Pair the DUT using chip-tool through "onnetwork" transport.

3. Read NumberOfPositions attribute : ./chip-tool switch read number-of-positions 1 1

[1686289367.443749][30133:30135] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_003B Attribute 0x0000_0000 DataVersion: 2306185014
[1686289367.444066][30133:30135] CHIP:TOO:   NumberOfPositions: 2

4. Change the switch to first position : echo '{"Name":"SwitchLatched","NewPosition":0}' > /tmp/chip_all_clusters_fifo_<PID> (PID of DUT)

[1686289860.323236][30125:30127] CHIP:-: Received payload: "{"Name":"SwitchLatched","NewPosition":0}"
[1686289860.323565][30125:30125] CHIP:DMG: Endpoint 1, Cluster 0x0000_003B update version to 89759737
[1686289860.323602][30125:30125] CHIP:-: The latching switch is moved to a new position:0
[1686289860.323618][30125:30125] CHIP:ZCL: SwitchServer: OnSwitchLatch

5. Read CurrentPosition : ./chip-tool switch read current-position 1 1

[1646209289.746157][2617:2622] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_003B Attribute 0x0000_0001DataVersion: 1206711661
[1646209289.746228][2617:2622] CHIP:TOO:   CurrentPosition: 0

6. Change the switch to second position : echo '{"Name":"SwitchLatched","NewPosition":1}' > /tmp/chip_all_clusters_fifo_<PID> (PID of DUT)

[1659600080.279936][7312:7321] CHIP:-: Received payload: "{"Name":"SwitchLatched","NewPosition":1}"
[1659600080.280318][7312:7312] CHIP:DMG: Endpoint 1, Cluster 0x0000_003B update version to 1755409c
[1659600080.280376][7312:7312] CHIP:-: The latching switch is moved to a new position:1
[1659600080.280412][7312:7312] CHIP:ZCL: SwitchServer: OnSwitchLatch

7. Read CurrentPosition : ./chip-tool switch read current-position 1 1

[1646209289.746157][2617:2622] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_003B Attribute 0x0000_0001DataVersion: 1206711661
[1646209289.746228][2617:2622] CHIP:TOO:  CurrentPosition: 1

As per SDK NumberOfPositions value is 2, it should not support more than 2 positions but currently in SDK its able change position up to 255

8. Change the switch to greater than NumberOfPositions value(Value>2, Ex: 254) : echo '{"Name":"SwitchLatched","NewPosition":254}' > /tmp/chip_all_clusters_fifo_8057

[1691654274.324862][8057:8060] CHIP:-: Received payload: "{"Name":"SwitchLatched","NewPosition":254}"
[1691654274.325424][8057:8057] CHIP:DMG: Endpoint 1, Cluster 0x0000_003B update version to e39cff78
[1691654274.325516][8057:8057] CHIP:-: The latching switch is moved to a new position:254
[1691654274.325567][8057:8057] CHIP:ZCL: SwitchServer: OnSwitchLatch

9 Read CurrentPosition : ./chip-tool switch read current-position 1 1

[1691654896.228456][8330:8332] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_003B Attribute 0x0000_0001 DataVersion: 3818717048
[1691654896.228590][8330:8332] CHIP:TOO:   CurrentPosition: 254

Bug prevalence

Every Time

GitHub hash of the SDK that was being used

367a0c6

Platform

raspi

Anything else?

I believe that NumberOfPositions is the maximum position available in the device but the issue here is SDK is able to move the position beyond the NumberOfPositions limit with no error.

[tcarmelveilleux]

This is an issue with the debugging trigger implementation, which is an artificial switch implementation. It's not a cert blocker per-se since a real switch would not operate via a direct setting if position value.

@yufengwangca could you please update the code to respect the limit in the cluster?