From 9d8b6a485d3fb683c60f3afc09bbf7e9fe718e85 Mon Sep 17 00:00:00 2001 From: Ramiro Recchia Date: Wed, 29 Jan 2025 14:06:24 +0000 Subject: [PATCH 01/12] CC.S.F00 updated to 1 to not skip test steps in TC-CC-3.3.yaml --- src/app/tests/suites/certification/ci-pics-values | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/app/tests/suites/certification/ci-pics-values b/src/app/tests/suites/certification/ci-pics-values index 1f4f1f66e906c3..481da7d2307875 100644 --- a/src/app/tests/suites/certification/ci-pics-values +++ b/src/app/tests/suites/certification/ci-pics-values @@ -204,7 +204,7 @@ CC.S.C41.Rsp=1 CC.S.C42.Rsp=1 CC.S.C43.Rsp=1 CC.S.C44.Rsp=1 -CC.S.F00=0 +CC.S.F00=1 CC.S.F01=1 CC.S.F02=1 CC.S.F03=1 From 28b06af55603d2a97598423430f7b336e0c26021 Mon Sep 17 00:00:00 2001 From: Ramiro Recchia Date: Wed, 29 Jan 2025 14:57:35 +0000 Subject: [PATCH 02/12] Remove manual steps, change delays and step size so tolerance does not span wrap --- .../suites/certification/Test_TC_CC_3_3.yaml | 99 +++++++------------ 1 file changed, 38 insertions(+), 61 deletions(-) diff --git a/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml b/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml index 91cdcbffe7d255..819db33ccc9d72 100644 --- a/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml +++ b/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml @@ -80,7 +80,7 @@ tests: - name: "StepMode" value: 1 - name: "StepSize" - value: 60 + value: 80 - name: "TransitionTime" value: 200 - name: "OptionsMask" @@ -88,56 +88,43 @@ tests: - name: "OptionsOverride" value: 0 - - label: "Wait 10s" + - label: "Wait 5s" PICS: CC.S.F00 cluster: "DelayCommands" command: "WaitForMs" arguments: values: - name: "ms" - value: 10000 + value: 5000 - - label: "Step 2c: TH reads CurrentHue attribute from DUT" + - label: "Step 2c: TH reads CurrentHue attribute from DUT" # Expects 220+/-15% PICS: CC.S.F00 && CC.S.A0000 command: "readAttribute" attribute: "CurrentHue" response: constraints: - minValue: 195 - maxValue: 254 + minValue: 187 + maxValue: 253 - - label: "Wait 10s" + - label: "Wait 15s" PICS: CC.S.F00 cluster: "DelayCommands" command: "WaitForMs" arguments: values: - name: "ms" - value: 10000 + value: 15000 - #Tolerance to be computed from the range - #Start value 230, expected value 5, 230 - 5 = 225, 225 % 15 = 34 , So 5+/-34 We need two range here as 254 is the max value - #Range will be: 230...254...0...39 - #Multiple range not supported in YAML: https://github.com/project-chip/connectedhomeip/issues/23197 + # Start value 220, expected value 24+/-15% (280 mod 256) + # Multiple range not supported in YAML: https://github.com/project-chip/connectedhomeip/issues/23197 - label: "Step 2d: TH reads CurrentHue attribute from DUT" - PICS: CC.S.F00 && CC.S.A0000 && PICS_SKIP_SAMPLE_APP - verification: | - ./chip-tool colorcontrol read current-hue 1 1 - - After another 10 seconds , Verify that CurrentHue value is 5 On TH(chip-tool) Logs and below is the sample log provided for the raspi platform - - [1649662328.012887][8447:8452] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0300 Attribute 0x0000_0000 DataVersion: 4087279986 - [1649662328.012957][8447:8452] CHIP:TOO: Current hue: 5 - [1649662328.013055][8447:8452] CHIP:EM: Sending Standalone Ack for MessageCounter:8634142 on exchange 13871i - (Value can vary) - cluster: "LogCommands" - command: "UserPrompt" - arguments: - values: - - name: "message" - value: "Enter 'y' after success" - - name: "expectedValue" - value: "y" + PICS: CC.S.F00 && CC.S.A0000 + command: "readAttribute" + attribute: "CurrentHue" + response: + constraints: + minValue: 20 + maxValue: 28 - label: "Wait 5s" PICS: CC.S.F00 @@ -148,25 +135,15 @@ tests: - name: "ms" value: 5000 - #Multiple range not supported in YAML: https://github.com/project-chip/connectedhomeip/issues/23197 + # Multiple range not supported in YAML: https://github.com/project-chip/connectedhomeip/issues/23197 - label: "Step 2e: TH reads CurrentHue attribute from DUT" - PICS: CC.S.F00 && CC.S.A0000 && PICS_SKIP_SAMPLE_APP - verification: | - ./chip-tool colorcontrol read current-hue 1 1 - - After another 5 seconds , Verify that CurrentHue value is 5 On TH(chip-tool) Logs and below is the sample log provided for the raspi platform - - [1649662328.012887][8447:8452] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0300 Attribute 0x0000_0000 DataVersion: 4087279986 - [1649662328.012957][8447:8452] CHIP:TOO: Current hue: 5 - [1649662328.013055][8447:8452] CHIP:EM: Sending Standalone Ack for MessageCounter:8634142 on exchange 13871i - cluster: "LogCommands" - command: "UserPrompt" - arguments: - values: - - name: "message" - value: "Enter 'y' after success" - - name: "expectedValue" - value: "y" + PICS: CC.S.F00 && CC.S.A0000 + command: "readAttribute" + attribute: "CurrentHue" + response: + constraints: + minValue: 20 + maxValue: 28 - label: "Step 3a: TH sends MoveToHue command to DUT with Hue=50, @@ -205,7 +182,7 @@ tests: - name: "StepMode" value: 3 - name: "StepSize" - value: 60 + value: 100 - name: "TransitionTime" value: 200 - name: "OptionsMask" @@ -213,41 +190,41 @@ tests: - name: "OptionsOverride" value: 0 - - label: "Wait 10s" + - label: "Wait 5s" PICS: CC.S.F00 cluster: "DelayCommands" command: "WaitForMs" arguments: values: - name: "ms" - value: 10000 + value: 5000 - - label: "Step 3c: TH reads CurrentHue attribute from DUT" + - label: "Step 3c: TH reads CurrentHue attribute from DUT" # Expects 25+/-15% PICS: CC.S.F00 && CC.S.A0000 command: "readAttribute" attribute: "CurrentHue" response: constraints: - minValue: 17 - maxValue: 23 + minValue: 21 + maxValue: 29 - - label: "Wait 10s" + - label: "Wait 15s" PICS: CC.S.F00 cluster: "DelayCommands" command: "WaitForMs" arguments: values: - name: "ms" - value: 10000 + value: 15000 - - label: "Step 3d: TH reads CurrentHue attribute from DUT" + - label: "Step 3d: TH reads CurrentHue attribute from DUT" # Expects 206+/-15% PICS: CC.S.F00 && CC.S.A0000 command: "readAttribute" attribute: "CurrentHue" response: constraints: - minValue: 208 - maxValue: 255 + minValue: 175 + maxValue: 237 - label: "Wait 5s" PICS: CC.S.F00 @@ -264,8 +241,8 @@ tests: attribute: "CurrentHue" response: constraints: - minValue: 208 - maxValue: 255 + minValue: 175 + maxValue: 237 - label: "Step 4a: TH reads ColorMode attribute from DUT" PICS: CC.S.F00 && CC.S.A0008 From 962fd936264d4f33207bf1770c1384481fec2854 Mon Sep 17 00:00:00 2001 From: Ramiro Recchia Date: Wed, 29 Jan 2025 18:46:02 +0000 Subject: [PATCH 03/12] Fixing identation issues --- .../tests/suites/certification/Test_TC_CC_3_3.yaml | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml b/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml index 819db33ccc9d72..18c321b70942e5 100644 --- a/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml +++ b/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml @@ -122,9 +122,9 @@ tests: command: "readAttribute" attribute: "CurrentHue" response: - constraints: - minValue: 20 - maxValue: 28 + constraints: + minValue: 20 + maxValue: 28 - label: "Wait 5s" PICS: CC.S.F00 @@ -141,9 +141,9 @@ tests: command: "readAttribute" attribute: "CurrentHue" response: - constraints: - minValue: 20 - maxValue: 28 + constraints: + minValue: 20 + maxValue: 28 - label: "Step 3a: TH sends MoveToHue command to DUT with Hue=50, From 6e2b61e60e76985cbc8e8de3bce97e14d78ea790 Mon Sep 17 00:00:00 2001 From: Ramiro Recchia Date: Tue, 11 Feb 2025 20:09:32 +0000 Subject: [PATCH 04/12] Fix toleFix tolerance calculation} --- .../suites/certification/Test_TC_CC_3_3.yaml | 28 +++++++++---------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml b/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml index 18c321b70942e5..7f75b3a5ae2f24 100644 --- a/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml +++ b/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml @@ -97,14 +97,14 @@ tests: - name: "ms" value: 5000 - - label: "Step 2c: TH reads CurrentHue attribute from DUT" # Expects 220+/-15% + - label: "Step 2c: TH reads CurrentHue attribute from DUT" # Expects 220 +/- 3 (15% of (220 - 200)) PICS: CC.S.F00 && CC.S.A0000 command: "readAttribute" attribute: "CurrentHue" response: constraints: - minValue: 187 - maxValue: 253 + minValue: 217 + maxValue: 223 - label: "Wait 15s" PICS: CC.S.F00 @@ -115,7 +115,7 @@ tests: - name: "ms" value: 15000 - # Start value 220, expected value 24+/-15% (280 mod 256) + # Start value 220, expected value 25 +/- 12 (280 mod 255 +/- 15% of 80) # Multiple range not supported in YAML: https://github.com/project-chip/connectedhomeip/issues/23197 - label: "Step 2d: TH reads CurrentHue attribute from DUT" PICS: CC.S.F00 && CC.S.A0000 @@ -123,8 +123,8 @@ tests: attribute: "CurrentHue" response: constraints: - minValue: 20 - maxValue: 28 + minValue: 13 + maxValue: 37 - label: "Wait 5s" PICS: CC.S.F00 @@ -142,8 +142,8 @@ tests: attribute: "CurrentHue" response: constraints: - minValue: 20 - maxValue: 28 + minValue: 13 + maxValue: 37 - label: "Step 3a: TH sends MoveToHue command to DUT with Hue=50, @@ -199,7 +199,7 @@ tests: - name: "ms" value: 5000 - - label: "Step 3c: TH reads CurrentHue attribute from DUT" # Expects 25+/-15% + - label: "Step 3c: TH reads CurrentHue attribute from DUT" # Expects 25 +/- 4 (15% of (50 - 25)) PICS: CC.S.F00 && CC.S.A0000 command: "readAttribute" attribute: "CurrentHue" @@ -217,14 +217,14 @@ tests: - name: "ms" value: 15000 - - label: "Step 3d: TH reads CurrentHue attribute from DUT" # Expects 206+/-15% + - label: "Step 3d: TH reads CurrentHue attribute from DUT" # Expects 205 +/- 15 (-50 mod 255 +/- 15% of 100) PICS: CC.S.F00 && CC.S.A0000 command: "readAttribute" attribute: "CurrentHue" response: constraints: - minValue: 175 - maxValue: 237 + minValue: 190 + maxValue: 220 - label: "Wait 5s" PICS: CC.S.F00 @@ -241,8 +241,8 @@ tests: attribute: "CurrentHue" response: constraints: - minValue: 175 - maxValue: 237 + minValue: 190 + maxValue: 220 - label: "Step 4a: TH reads ColorMode attribute from DUT" PICS: CC.S.F00 && CC.S.A0008 From c682cff2cf696e09a75c7dfb0063fbcd0af947a2 Mon Sep 17 00:00:00 2001 From: Ramiro Recchia Date: Thu, 13 Feb 2025 11:21:58 -0300 Subject: [PATCH 05/12] Updated comments --- src/app/tests/suites/certification/Test_TC_CC_3_3.yaml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml b/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml index 7f75b3a5ae2f24..7a2d4bd7939851 100644 --- a/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml +++ b/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml @@ -97,7 +97,7 @@ tests: - name: "ms" value: 5000 - - label: "Step 2c: TH reads CurrentHue attribute from DUT" # Expects 220 +/- 3 (15% of (220 - 200)) + - label: "Step 2c: TH reads CurrentHue attribute from DUT" # Expects 220 (200 + 80*(5/20)) with a tolerance of +/- 3 (15% of 20), so the expected value is in the range (217, 221) PICS: CC.S.F00 && CC.S.A0000 command: "readAttribute" attribute: "CurrentHue" @@ -115,7 +115,7 @@ tests: - name: "ms" value: 15000 - # Start value 220, expected value 25 +/- 12 (280 mod 255 +/- 15% of 80) + # Start value 220, expected value 25 (200 + 80*(20/20) = 280 mod 255) with a tolerance of +/- 12 (15% of 80), so the expected value is in the range (13, 37) # Multiple range not supported in YAML: https://github.com/project-chip/connectedhomeip/issues/23197 - label: "Step 2d: TH reads CurrentHue attribute from DUT" PICS: CC.S.F00 && CC.S.A0000 @@ -199,7 +199,7 @@ tests: - name: "ms" value: 5000 - - label: "Step 3c: TH reads CurrentHue attribute from DUT" # Expects 25 +/- 4 (15% of (50 - 25)) + - label: "Step 3c: TH reads CurrentHue attribute from DUT" # Expects 25 (50 - 100*(5/20)) with a tolerance of +/- 4 (15% of 25), so the expected value is in the range (21, 29) PICS: CC.S.F00 && CC.S.A0000 command: "readAttribute" attribute: "CurrentHue" @@ -217,7 +217,7 @@ tests: - name: "ms" value: 15000 - - label: "Step 3d: TH reads CurrentHue attribute from DUT" # Expects 205 +/- 15 (-50 mod 255 +/- 15% of 100) + - label: "Step 3d: TH reads CurrentHue attribute from DUT" # Expects 205 (50 - 100*(20/20) = -50 mod 255) with a tolerance of +/- 15 (15% of 100), so the expected value is in the range (190, 220) PICS: CC.S.F00 && CC.S.A0000 command: "readAttribute" attribute: "CurrentHue" From c15803cedca66c98e5f28fc83dd7f7e31a7fdbb3 Mon Sep 17 00:00:00 2001 From: Ramiro Recchia Date: Tue, 25 Feb 2025 13:40:10 +0000 Subject: [PATCH 06/12] Fix one comment in the tolerance value --- admin_storage.json | 33 +++++++++++++++++++ .../suites/certification/Test_TC_CC_3_3.yaml | 2 +- 2 files changed, 34 insertions(+), 1 deletion(-) create mode 100644 admin_storage.json diff --git a/admin_storage.json b/admin_storage.json new file mode 100644 index 00000000000000..91a903ea84bfc9 --- /dev/null +++ b/admin_storage.json @@ -0,0 +1,33 @@ +{ + "sdk-config": { + "g/lkgt": "FSYAgKi8LBg=", + "g/gcc": "fge1CQ==", + "g/gdc": "0paFAQ==", + "ExampleOpCredsCAKey1": "BA6TqwBw5VtEXI+2z+V++Co95+GbYIaxljZ/aHFW02en9g6qIuCe5jZm4pHO5ajuiFJHp8o0s4XrVkpuLGDTmDPCUWJ+WvcJcULQ8bFDqbVXF+ckfHMw58rMr5wD/DaYag==", + "ExampleOpCredsICAKey1": "BJ/RdllUSEeQo9pvUzMg/YnVDarYuSzyjAopbSq/7O2H04Hz1+anAJFGQfZG2M6p/rtNDJOUxnAcU2CSqQ8vQsavq3rJhWxQQItYbLPGuKx0y5Yqzu2JCXwb+B9acguPTw==", + "ExampleCARootCert1": "MIICQTCCAeagAwIBAgIBATAKBggqhkjOPQQDAjAiMSAwHgYKKwYBBAGConwBBAwQMDAwMDAwMDAwMDAwMDAwMTAeFw0yMTAxMDEwMDAwMDBaFw0zMDEyMzAwMDAwMDBaMCIxIDAeBgorBgEEAYKifAEEDBAwMDAwMDAwMDAwMDAwMDAxMFkwEwYHKoZIzj0CAQYIKoZIzj0DAQcDQgAEDpOrAHDlW0Rcj7bP5X74Kj3n4ZtghrGWNn9ocVbTZ6f2Dqoi4J7mNmbikc7lqO6IUkenyjSzhetWSm4sYNOYM6OCAQswggEHMA8GA1UdEwEB/wQFMAMBAf8wDgYDVR0PAQH/BAQDAgEGMB0GA1UdDgQWBBQ3OT0HyH2imyuKRyLLjYK+SkDBzjAfBgNVHSMEGDAWgBQ3OT0HyH2imyuKRyLLjYK+SkDBzjCBowYDVR0RBIGbBIGYQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUEwCgYIKoZIzj0EAwIDSQAwRgIhAKoNo5AH4k1dAUwmpVkHZFNcnDpZ5DSlZyBYaipNEDTlAiEAlbEjQ8Yk4ofOaLtxZQ28SlGuMvb/UIi73/C2xY5q45k=", + "ExampleCAIntermediateCert1": "MIICQDCCAeagAwIBAgIBATAKBggqhkjOPQQDAjAiMSAwHgYKKwYBBAGConwBBAwQMDAwMDAwMDAwMDAwMDAwMTAeFw0yMTAxMDEwMDAwMDBaFw0zMDEyMzAwMDAwMDBaMCIxIDAeBgorBgEEAYKifAEDDBAwMDAwMDAwMDAwMDAwMDAyMFkwEwYHKoZIzj0CAQYIKoZIzj0DAQcDQgAEn9F2WVRIR5Cj2m9TMyD9idUNqti5LPKMCiltKr/s7YfTgfPX5qcAkUZB9kbYzqn+u00Mk5TGcBxTYJKpDy9CxqOCAQswggEHMA8GA1UdEwEB/wQFMAMBAf8wDgYDVR0PAQH/BAQDAgEGMB0GA1UdDgQWBBTJJYhyTj86CFmrcT0PEVYwjsBEqzAfBgNVHSMEGDAWgBQ3OT0HyH2imyuKRyLLjYK+SkDBzjCBowYDVR0RBIGbBIGYQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUEwCgYIKoZIzj0EAwIDSAAwRQIhAIV5/TdJUe/FzpLMQGp4g5Pc4TShULEoyKi/8ZOfKeILAiAiJhXJ8ikrXUKFcCj8SiIiTldtPljhHjeg5UXkf1Mm+g==", + "f/1/m": "FSUA8f8sAQAY", + "f/1/n": "FTABAQEkAgE3AyQTAhgmBIAigScmBYAlTTo3BiQVASYRabYBACYWAQABABgkBwEkCAEwCUEE6Ilp+86s5R3G6ujGStzW/Bc7QCNXVBt58oXsGGxXPNfhkue1OeBpOU8kA5hkSpihVFsiE0AVVcbbIGucCjdV6TcKNQEoARgkAgE2AwQCBAEYMAQUNuyw1nGkzKXPNv/ItT0/3srwY/wwBRTJJYhyTj86CFmrcT0PEVYwjsBEqxgwC0A02tE2fzFIq58QyMhqOkjypwzVXsc38jdpDlVUN3di6jOFSeiSvW49NyROlTdaDk7XiTAQ8uoNBRRt7DZyUfVLGA==", + "f/1/i": "FTABAQEkAgE3AyQUARgmBIAigScmBYAlTTo3BiQTAhgkBwEkCAEwCUEEn9F2WVRIR5Cj2m9TMyD9idUNqti5LPKMCiltKr/s7YfTgfPX5qcAkUZB9kbYzqn+u00Mk5TGcBxTYJKpDy9CxjcKNQEpARgkAmAwBBTJJYhyTj86CFmrcT0PEVYwjsBEqzAFFDc5PQfIfaKbK4pHIsuNgr5KQMHOMAamMIGjBgNVHREEgZsEgZhBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQRgwC0CFef03SVHvxc6SzEBqeIOT3OE0oVCxKMiov/GTnyniCyImFcnyKStdQoVwKPxKIiJOV20+WOEeN6DlReR/Uyb6GA==", + "f/1/r": "FTABAQEkAgE3AyQUARgmBIAigScmBYAlTTo3BiQUARgkBwEkCAEwCUEEDpOrAHDlW0Rcj7bP5X74Kj3n4ZtghrGWNn9ocVbTZ6f2Dqoi4J7mNmbikc7lqO6IUkenyjSzhetWSm4sYNOYMzcKNQEpARgkAmAwBBQ3OT0HyH2imyuKRyLLjYK+SkDBzjAFFDc5PQfIfaKbK4pHIsuNgr5KQMHOMAamMIGjBgNVHREEgZsEgZhBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQRgwC0DtDPCBXxR6L6YsTChtScKSVa3O37GH3nnS9eOStKJpveGf1wkjv59iEXPFagRzvZ7IcHMp8GRdvJLTlNct6s1UGA==", + "g/fidx": "FSQAAjYBBAEYGA==", + "f/1/k/0": "FSQBACQCATYDFSQEACUF6wgwBhDSpaw/mK/lmmy2lnBiFKbxGBUkBAAkBQAwBhAAAAAAAAAAAAAAAAAAAAAAGBUkBAAkBQAwBhAAAAAAAAAAAAAAAAAAAAAAGBglB///GA==", + "g/gfl": "FSQBASQCARg=", + "f/1/g": "FSQBACQCACQDACQEACQFACQGASQHABg=", + "g/icdfl": "FgQBGA==", + "g/sri": "FhUkAQEmAiFDNBIYGA==", + "f/1/s/0000000012344321": "FTADEGN+1h7TJzOjNSxFJJnshFUwBCDiqXmUvZjv0WunCNVtZ2jT2k/O2L8Zncu42DWZf+MOATAFDAAAAAAAAAAAAAAAABg=", + "g/s/Y37WHtMnM6M1LEUkmeyEVQ==": "FSQBASYCIUM0Ehg=" + }, + "repl-config": { + "caList": { + "1": [ + { + "fabricId": 1, + "vendorId": 65521 + } + ] + } + } +} \ No newline at end of file diff --git a/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml b/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml index 7a2d4bd7939851..03a8d42beeb981 100644 --- a/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml +++ b/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml @@ -97,7 +97,7 @@ tests: - name: "ms" value: 5000 - - label: "Step 2c: TH reads CurrentHue attribute from DUT" # Expects 220 (200 + 80*(5/20)) with a tolerance of +/- 3 (15% of 20), so the expected value is in the range (217, 221) + - label: "Step 2c: TH reads CurrentHue attribute from DUT" # Expects 220 (200 + 80*(5/20)) with a tolerance of +/- 3 (15% of 20), so the expected value is in the range (217, 223) PICS: CC.S.F00 && CC.S.A0000 command: "readAttribute" attribute: "CurrentHue" From 71b52d1117aa28fb43026bc639a2bd70d3469ffa Mon Sep 17 00:00:00 2001 From: Ramiro Recchia Date: Tue, 25 Feb 2025 13:43:00 +0000 Subject: [PATCH 07/12] Remove json --- admin_storage.json | 33 --------------------------------- 1 file changed, 33 deletions(-) delete mode 100644 admin_storage.json diff --git a/admin_storage.json b/admin_storage.json deleted file mode 100644 index 91a903ea84bfc9..00000000000000 --- a/admin_storage.json +++ /dev/null @@ -1,33 +0,0 @@ -{ - "sdk-config": { - "g/lkgt": "FSYAgKi8LBg=", - "g/gcc": "fge1CQ==", - "g/gdc": "0paFAQ==", - "ExampleOpCredsCAKey1": "BA6TqwBw5VtEXI+2z+V++Co95+GbYIaxljZ/aHFW02en9g6qIuCe5jZm4pHO5ajuiFJHp8o0s4XrVkpuLGDTmDPCUWJ+WvcJcULQ8bFDqbVXF+ckfHMw58rMr5wD/DaYag==", - "ExampleOpCredsICAKey1": "BJ/RdllUSEeQo9pvUzMg/YnVDarYuSzyjAopbSq/7O2H04Hz1+anAJFGQfZG2M6p/rtNDJOUxnAcU2CSqQ8vQsavq3rJhWxQQItYbLPGuKx0y5Yqzu2JCXwb+B9acguPTw==", - "ExampleCARootCert1": "MIICQTCCAeagAwIBAgIBATAKBggqhkjOPQQDAjAiMSAwHgYKKwYBBAGConwBBAwQMDAwMDAwMDAwMDAwMDAwMTAeFw0yMTAxMDEwMDAwMDBaFw0zMDEyMzAwMDAwMDBaMCIxIDAeBgorBgEEAYKifAEEDBAwMDAwMDAwMDAwMDAwMDAxMFkwEwYHKoZIzj0CAQYIKoZIzj0DAQcDQgAEDpOrAHDlW0Rcj7bP5X74Kj3n4ZtghrGWNn9ocVbTZ6f2Dqoi4J7mNmbikc7lqO6IUkenyjSzhetWSm4sYNOYM6OCAQswggEHMA8GA1UdEwEB/wQFMAMBAf8wDgYDVR0PAQH/BAQDAgEGMB0GA1UdDgQWBBQ3OT0HyH2imyuKRyLLjYK+SkDBzjAfBgNVHSMEGDAWgBQ3OT0HyH2imyuKRyLLjYK+SkDBzjCBowYDVR0RBIGbBIGYQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUEwCgYIKoZIzj0EAwIDSQAwRgIhAKoNo5AH4k1dAUwmpVkHZFNcnDpZ5DSlZyBYaipNEDTlAiEAlbEjQ8Yk4ofOaLtxZQ28SlGuMvb/UIi73/C2xY5q45k=", - "ExampleCAIntermediateCert1": "MIICQDCCAeagAwIBAgIBATAKBggqhkjOPQQDAjAiMSAwHgYKKwYBBAGConwBBAwQMDAwMDAwMDAwMDAwMDAwMTAeFw0yMTAxMDEwMDAwMDBaFw0zMDEyMzAwMDAwMDBaMCIxIDAeBgorBgEEAYKifAEDDBAwMDAwMDAwMDAwMDAwMDAyMFkwEwYHKoZIzj0CAQYIKoZIzj0DAQcDQgAEn9F2WVRIR5Cj2m9TMyD9idUNqti5LPKMCiltKr/s7YfTgfPX5qcAkUZB9kbYzqn+u00Mk5TGcBxTYJKpDy9CxqOCAQswggEHMA8GA1UdEwEB/wQFMAMBAf8wDgYDVR0PAQH/BAQDAgEGMB0GA1UdDgQWBBTJJYhyTj86CFmrcT0PEVYwjsBEqzAfBgNVHSMEGDAWgBQ3OT0HyH2imyuKRyLLjYK+SkDBzjCBowYDVR0RBIGbBIGYQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUEwCgYIKoZIzj0EAwIDSAAwRQIhAIV5/TdJUe/FzpLMQGp4g5Pc4TShULEoyKi/8ZOfKeILAiAiJhXJ8ikrXUKFcCj8SiIiTldtPljhHjeg5UXkf1Mm+g==", - "f/1/m": "FSUA8f8sAQAY", - "f/1/n": "FTABAQEkAgE3AyQTAhgmBIAigScmBYAlTTo3BiQVASYRabYBACYWAQABABgkBwEkCAEwCUEE6Ilp+86s5R3G6ujGStzW/Bc7QCNXVBt58oXsGGxXPNfhkue1OeBpOU8kA5hkSpihVFsiE0AVVcbbIGucCjdV6TcKNQEoARgkAgE2AwQCBAEYMAQUNuyw1nGkzKXPNv/ItT0/3srwY/wwBRTJJYhyTj86CFmrcT0PEVYwjsBEqxgwC0A02tE2fzFIq58QyMhqOkjypwzVXsc38jdpDlVUN3di6jOFSeiSvW49NyROlTdaDk7XiTAQ8uoNBRRt7DZyUfVLGA==", - "f/1/i": "FTABAQEkAgE3AyQUARgmBIAigScmBYAlTTo3BiQTAhgkBwEkCAEwCUEEn9F2WVRIR5Cj2m9TMyD9idUNqti5LPKMCiltKr/s7YfTgfPX5qcAkUZB9kbYzqn+u00Mk5TGcBxTYJKpDy9CxjcKNQEpARgkAmAwBBTJJYhyTj86CFmrcT0PEVYwjsBEqzAFFDc5PQfIfaKbK4pHIsuNgr5KQMHOMAamMIGjBgNVHREEgZsEgZhBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQRgwC0CFef03SVHvxc6SzEBqeIOT3OE0oVCxKMiov/GTnyniCyImFcnyKStdQoVwKPxKIiJOV20+WOEeN6DlReR/Uyb6GA==", - "f/1/r": "FTABAQEkAgE3AyQUARgmBIAigScmBYAlTTo3BiQUARgkBwEkCAEwCUEEDpOrAHDlW0Rcj7bP5X74Kj3n4ZtghrGWNn9ocVbTZ6f2Dqoi4J7mNmbikc7lqO6IUkenyjSzhetWSm4sYNOYMzcKNQEpARgkAmAwBBQ3OT0HyH2imyuKRyLLjYK+SkDBzjAFFDc5PQfIfaKbK4pHIsuNgr5KQMHOMAamMIGjBgNVHREEgZsEgZhBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQRgwC0DtDPCBXxR6L6YsTChtScKSVa3O37GH3nnS9eOStKJpveGf1wkjv59iEXPFagRzvZ7IcHMp8GRdvJLTlNct6s1UGA==", - "g/fidx": "FSQAAjYBBAEYGA==", - "f/1/k/0": "FSQBACQCATYDFSQEACUF6wgwBhDSpaw/mK/lmmy2lnBiFKbxGBUkBAAkBQAwBhAAAAAAAAAAAAAAAAAAAAAAGBUkBAAkBQAwBhAAAAAAAAAAAAAAAAAAAAAAGBglB///GA==", - "g/gfl": "FSQBASQCARg=", - "f/1/g": "FSQBACQCACQDACQEACQFACQGASQHABg=", - "g/icdfl": "FgQBGA==", - "g/sri": "FhUkAQEmAiFDNBIYGA==", - "f/1/s/0000000012344321": "FTADEGN+1h7TJzOjNSxFJJnshFUwBCDiqXmUvZjv0WunCNVtZ2jT2k/O2L8Zncu42DWZf+MOATAFDAAAAAAAAAAAAAAAABg=", - "g/s/Y37WHtMnM6M1LEUkmeyEVQ==": "FSQBASYCIUM0Ehg=" - }, - "repl-config": { - "caList": { - "1": [ - { - "fabricId": 1, - "vendorId": 65521 - } - ] - } - } -} \ No newline at end of file From 6162d7bcb81edeacace38af4ca8d0b4e65b7dc34 Mon Sep 17 00:00:00 2001 From: Ramiro Recchia Date: Tue, 25 Feb 2025 13:48:39 +0000 Subject: [PATCH 08/12] Updated labels --- src/app/tests/suites/certification/Test_TC_CC_3_3.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml b/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml index 03a8d42beeb981..de463f8d5e0a40 100644 --- a/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml +++ b/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml @@ -72,7 +72,7 @@ tests: - label: "Step 2b: TH sends StepHue command to DUT with StepMode=0x01 (up), - StepSize=60 and TransitionTime=200 (20s)" + StepSize=80 and TransitionTime=200 (20s)" PICS: CC.S.F00 && CC.S.C02.Rsp command: "StepHue" arguments: @@ -174,7 +174,7 @@ tests: - label: "Step 3b: TH sends StepHue command to DUT with StepMode=0x03 (down), - StepSize=60 and TransitionTime=200 (20s)" + StepSize=100 and TransitionTime=200 (20s)" PICS: CC.S.F00 && CC.S.C02.Rsp command: "StepHue" arguments: From ea7738d76e6a0dfb28dcc7ce1a051fc8b339da8b Mon Sep 17 00:00:00 2001 From: Ramiro Recchia Date: Wed, 26 Feb 2025 18:24:58 +0000 Subject: [PATCH 09/12] Added closed ranged in tolerance comments --- src/app/tests/suites/certification/Test_TC_CC_3_3.yaml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml b/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml index de463f8d5e0a40..dd0b1fd242d4af 100644 --- a/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml +++ b/src/app/tests/suites/certification/Test_TC_CC_3_3.yaml @@ -97,7 +97,7 @@ tests: - name: "ms" value: 5000 - - label: "Step 2c: TH reads CurrentHue attribute from DUT" # Expects 220 (200 + 80*(5/20)) with a tolerance of +/- 3 (15% of 20), so the expected value is in the range (217, 223) + - label: "Step 2c: TH reads CurrentHue attribute from DUT" # Expects 220 (200 + 80*(5/20)) with a tolerance of +/- 3 (15% of 20), so the expected value is in the range [217, 223] PICS: CC.S.F00 && CC.S.A0000 command: "readAttribute" attribute: "CurrentHue" @@ -115,7 +115,7 @@ tests: - name: "ms" value: 15000 - # Start value 220, expected value 25 (200 + 80*(20/20) = 280 mod 255) with a tolerance of +/- 12 (15% of 80), so the expected value is in the range (13, 37) + # Start value 200, expected value 25 (200 + 80*(20/20) = 280 mod 255) with a tolerance of +/- 12 (15% of 80), so the expected value is in the range [13, 37] # Multiple range not supported in YAML: https://github.com/project-chip/connectedhomeip/issues/23197 - label: "Step 2d: TH reads CurrentHue attribute from DUT" PICS: CC.S.F00 && CC.S.A0000 @@ -199,7 +199,7 @@ tests: - name: "ms" value: 5000 - - label: "Step 3c: TH reads CurrentHue attribute from DUT" # Expects 25 (50 - 100*(5/20)) with a tolerance of +/- 4 (15% of 25), so the expected value is in the range (21, 29) + - label: "Step 3c: TH reads CurrentHue attribute from DUT" # Expects 25 (50 - 100*(5/20)) with a tolerance of +/- 4 (15% of 25), so the expected value is in the range [21, 29] PICS: CC.S.F00 && CC.S.A0000 command: "readAttribute" attribute: "CurrentHue" @@ -217,7 +217,7 @@ tests: - name: "ms" value: 15000 - - label: "Step 3d: TH reads CurrentHue attribute from DUT" # Expects 205 (50 - 100*(20/20) = -50 mod 255) with a tolerance of +/- 15 (15% of 100), so the expected value is in the range (190, 220) + - label: "Step 3d: TH reads CurrentHue attribute from DUT" # Expects 205 (50 - 100*(20/20) = -50 mod 255) with a tolerance of +/- 15 (15% of 100), so the expected value is in the range [190, 220] PICS: CC.S.F00 && CC.S.A0000 command: "readAttribute" attribute: "CurrentHue" From ed6d4f8da247c2afbebb03cf94d62c7fcf45c01e Mon Sep 17 00:00:00 2001 From: Ramiro Recchia Date: Wed, 5 Mar 2025 10:56:21 +0000 Subject: [PATCH 10/12] Add legacy flag --- src/python_testing/TC_DEM_2_2.py | 4 +-- .../chip/testing/matter_testing.py | 31 +++++++++++++++++-- 2 files changed, 31 insertions(+), 4 deletions(-) diff --git a/src/python_testing/TC_DEM_2_2.py b/src/python_testing/TC_DEM_2_2.py index 390a7e8b304ad0..9dc005a8e20819 100644 --- a/src/python_testing/TC_DEM_2_2.py +++ b/src/python_testing/TC_DEM_2_2.py @@ -174,7 +174,7 @@ async def test_TC_DEM_2_2(self): # Commission DUT - already done self.step("2") - await self.validate_feature_map([Clusters.DeviceEnergyManagement.Bitmaps.Feature.kPowerAdjustment], []) + # await self.validate_feature_map([Clusters.DeviceEnergyManagement.Bitmaps.Feature.kPowerAdjustment], []) self.step("3") # Subscribe to Events and when they are sent push them to a queue for checking later @@ -184,7 +184,7 @@ async def test_TC_DEM_2_2(self): self.get_endpoint(default=1)) self.step("4") - await self.check_test_event_triggers_enabled() + # await self.check_test_event_triggers_enabled() self.step("5") await self.send_test_event_trigger_power_adjustment() diff --git a/src/python_testing/matter_testing_infrastructure/chip/testing/matter_testing.py b/src/python_testing/matter_testing_infrastructure/chip/testing/matter_testing.py index 1054463ea64489..af2a899f1309a6 100644 --- a/src/python_testing/matter_testing_infrastructure/chip/testing/matter_testing.py +++ b/src/python_testing/matter_testing_infrastructure/chip/testing/matter_testing.py @@ -676,6 +676,9 @@ class MatterTestConfig: tc_version_to_simulate: int = None tc_user_response_to_simulate: int = None + # If this is set tests will allow sending test event triggers with endpoint 0 for older devices + legacy: bool = None + class ClusterMapper: """Describe clusters/attributes using schema names.""" @@ -1289,7 +1292,7 @@ async def send_single_cmd( return result async def send_test_event_triggers(self, eventTrigger: int, enableKey: bytes = None): - """This helper function sends a test event trigger to the General Diagnostics cluster on endpoint 0 + """This helper function sends a test event trigger to the General Diagnostics cluster, If --use-legacy-test-event-trigers flag is present the endpoint will 0 The enableKey can be passed into the function, or omitted which will then use the one provided to the script via --hex-arg enableKey: @@ -1304,9 +1307,30 @@ async def send_test_event_triggers(self, eventTrigger: int, enableKey: bytes = N else: enableKey = self.matter_test_config.global_test_params['enableKey'] + # What about --endpoing != 0 and legacy = True? Right now target endpoint is set to 0 + + if not self.matter_test_config.legacy and self.matter_test_config.endpoint is not None: + target_endpoint = self.matter_test_config.endpoint + else: + target_endpoint = 0 + + print("") + print("Target endpoint: ", target_endpoint) + print("") + + print("") + print("Event trigger: ", hex(eventTrigger)) + print("") + + eventTrigger = eventTrigger | (target_endpoint << 32) + + print("") + print("Event trigger: ", hex(eventTrigger)) + print("") + try: # GeneralDiagnostics cluster is meant to be on Endpoint 0 (Root) - await self.send_single_cmd(endpoint=0, + await self.send_single_cmd(endpoint=target_endpoint, cmd=Clusters.GeneralDiagnostics.Commands.TestEventTrigger( enableKey, eventTrigger) @@ -1943,6 +1967,7 @@ def convert_args_to_matter_config(args: argparse.Namespace) -> MatterTestConfig: config.endpoint = args.endpoint # This can be None, the get_endpoint function allows the tests to supply a default config.app_pid = 0 if args.app_pid is None else args.app_pid config.fail_on_skipped_tests = args.fail_on_skipped + config.legacy = True if args.use_legacy_test_event_triggers else args.use_legacy_test_event_triggers config.controller_node_id = args.controller_node_id config.trace_to = args.trace_to @@ -1998,6 +2023,8 @@ def parse_matter_test_args(argv: Optional[List[str]] = None) -> MatterTestConfig basic_group.add_argument('--app-pid', type=int, default=0, help="The PID of the app against which the test is going to run") basic_group.add_argument('--timeout', type=int, help="Test timeout in seconds") basic_group.add_argument("--PICS", help="PICS file path", type=str) + basic_group.add_argument("--use-legacy-test-event-triggers", action="store_true", default=False, + help="Send test event triggers with endpoint 0 for older devices") commission_group = parser.add_argument_group(title="Commissioning", description="Arguments to commission a node") From 2b7012b9ad86a26b2a0305f974e5a0727f12adf5 Mon Sep 17 00:00:00 2001 From: Ramiro Recchia Date: Wed, 5 Mar 2025 19:36:14 +0000 Subject: [PATCH 11/12] Updating endpoint to EP0 --- src/python_testing/TC_DEM_2_2.py | 4 ++-- .../chip/testing/matter_testing.py | 5 ++--- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/src/python_testing/TC_DEM_2_2.py b/src/python_testing/TC_DEM_2_2.py index 9dc005a8e20819..390a7e8b304ad0 100644 --- a/src/python_testing/TC_DEM_2_2.py +++ b/src/python_testing/TC_DEM_2_2.py @@ -174,7 +174,7 @@ async def test_TC_DEM_2_2(self): # Commission DUT - already done self.step("2") - # await self.validate_feature_map([Clusters.DeviceEnergyManagement.Bitmaps.Feature.kPowerAdjustment], []) + await self.validate_feature_map([Clusters.DeviceEnergyManagement.Bitmaps.Feature.kPowerAdjustment], []) self.step("3") # Subscribe to Events and when they are sent push them to a queue for checking later @@ -184,7 +184,7 @@ async def test_TC_DEM_2_2(self): self.get_endpoint(default=1)) self.step("4") - # await self.check_test_event_triggers_enabled() + await self.check_test_event_triggers_enabled() self.step("5") await self.send_test_event_trigger_power_adjustment() diff --git a/src/python_testing/matter_testing_infrastructure/chip/testing/matter_testing.py b/src/python_testing/matter_testing_infrastructure/chip/testing/matter_testing.py index af2a899f1309a6..ba62b36547a1e3 100644 --- a/src/python_testing/matter_testing_infrastructure/chip/testing/matter_testing.py +++ b/src/python_testing/matter_testing_infrastructure/chip/testing/matter_testing.py @@ -1309,10 +1309,9 @@ async def send_test_event_triggers(self, eventTrigger: int, enableKey: bytes = N # What about --endpoing != 0 and legacy = True? Right now target endpoint is set to 0 + target_endpoint = 0 if not self.matter_test_config.legacy and self.matter_test_config.endpoint is not None: target_endpoint = self.matter_test_config.endpoint - else: - target_endpoint = 0 print("") print("Target endpoint: ", target_endpoint) @@ -1330,7 +1329,7 @@ async def send_test_event_triggers(self, eventTrigger: int, enableKey: bytes = N try: # GeneralDiagnostics cluster is meant to be on Endpoint 0 (Root) - await self.send_single_cmd(endpoint=target_endpoint, + await self.send_single_cmd(endpoint=0, cmd=Clusters.GeneralDiagnostics.Commands.TestEventTrigger( enableKey, eventTrigger) From 83370fd4a1ee6cb2dcd8af0c083791a7a8a0b2af Mon Sep 17 00:00:00 2001 From: Ramiro Recchia Date: Thu, 6 Mar 2025 18:59:11 +0000 Subject: [PATCH 12/12] save general diagnosis result --- .../chip/testing/matter_testing.py | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/src/python_testing/matter_testing_infrastructure/chip/testing/matter_testing.py b/src/python_testing/matter_testing_infrastructure/chip/testing/matter_testing.py index ba62b36547a1e3..3a6fad142c2bc6 100644 --- a/src/python_testing/matter_testing_infrastructure/chip/testing/matter_testing.py +++ b/src/python_testing/matter_testing_infrastructure/chip/testing/matter_testing.py @@ -1329,11 +1329,15 @@ async def send_test_event_triggers(self, eventTrigger: int, enableKey: bytes = N try: # GeneralDiagnostics cluster is meant to be on Endpoint 0 (Root) - await self.send_single_cmd(endpoint=0, - cmd=Clusters.GeneralDiagnostics.Commands.TestEventTrigger( - enableKey, - eventTrigger) - ) + general_diagnosis_result = await self.send_single_cmd(endpoint=0, + cmd=Clusters.GeneralDiagnostics.Commands.TestEventTrigger( + enableKey, + eventTrigger) + ) + + print("") + logger.info(general_diagnosis_result) + print("") except InteractionModelError as e: asserts.fail(