connectedhomeip icon indicating copy to clipboard operation
connectedhomeip copied to clipboard

[1.0] [SQA] [TC-SWTCH-2.2] Misleading data in MultiPressComplete Event.

Open Survensa opened this issue 2 years ago • 0 comments

Reproduction steps

  1. Use V1.0 commit as chip-tool
  2. Either Master or V1.0 commit as DUT
  3. After test Step 6a : Set up subscription to InitialPress, ShortRelease, MultiPressOngoing, MultiPressComplete events
  4. Subscribe the event : echo '{"Name":"MultiPressComplete","PreviousPosition":1,"TotalNumberOfPressesCounted":1}' > /tmp/chip_all_clusters_fifo_<PID>
  5. In test plan Step 6d : Operator releases switch Open the Interactive mode : ./chip-tool interactive start Run the subscription of the event : switch subscribe-event multi-press-complete 1 100 1 1

Bug prevalence

Whenever I do this

GitHub hash of the SDK that was being used

7c2353bb7af06303350796103867ed5aa9b4fd27

Platform

raspi

Platform Version(s)

RPI-4, 8GB-RAM variant

Type

Core SDK Crash

Anything else?

As per the spec MultiPressComplete Event Should have the data of the PreviousPosition,TotalNumberOfPressesCounted. While using V1.0 as chip-tool after "Operator releases switch" the data is not chnaged to PreviousPosition

NOTE : This issue is not observed while using Master commit as controller

Excepted outcome:

[1659695330.923244][3912:3918] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_003B Event 0x0000_0005 [1659695330.923264][3912:3918] CHIP:TOO: Event number: 45 [1659695330.923283][3912:3918] CHIP:TOO: Priority: Info [1659695330.923302][3912:3918] CHIP:TOO: Timestamp: 17754336 [1659695330.923326][3912:3918] CHIP:TOO: MultiPressComplete: { [1659695330.923346][3912:3918] CHIP:TOO: PrevioiusPosition: 1 [1659695330.923366][3912:3918] CHIP:TOO: TotalNumberOfPressesCounted: 1 [1659695330.923385][3912:3918] CHIP:TOO: }

Actual outcome:

[1669976114.571487][35475:35477] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_003B Event 0x0000_0006 [1669976114.571538][35475:35477] CHIP:TOO: Event number: 6 [1669976114.571561][35475:35477] CHIP:TOO: Priority: Info [1669976114.571584][35475:35477] CHIP:TOO: Timestamp: 18863395 [1669976114.571622][35475:35477] CHIP:TOO: MultiPressComplete: { [1669976114.571654][35475:35477] CHIP:TOO: NewPosition: 1 [1669976114.571687][35475:35477] CHIP:TOO: TotalNumberOfPressesCounted: 1 [1669976114.571714][35475:35477] CHIP:TOO: }

Referance : Spec : https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/master/src/app_clusters/Switch.adoc#multipresscomplete-event

Test Plan : https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/cluster/switch.adoc#tc-swtch-2-2-primary-functionality-with-server-as-dut

Log : TC-SWTCH-2.2.txt

Survensa avatar Dec 16 '22 12:12 Survensa